Hi all,

we are using the wfs-ng module as a WFS client. Unfortunately, for Transaction 
requests  WFS exception responses are dropped by the client and instead an 
empty IOException is thrown (TransactionResponseImpl.java:105, see 
https://github.com/geotools/geotools/blob/cfe895c1fe84c65904d1aa383496ed2aff7afa42/modules/unsupported/wfs-ng/src/main/java/org/geotools/data/wfs/internal/v1_x/TransactionResponseImpl.java#L105).

A better solutions is already implemented for the GetFeature case 
(AbstractGetFeatureResponseParserFactory#createResponse, see 
https://github.com/geotools/geotools/blob/cfe895c1fe84c65904d1aa383496ed2aff7afa42/modules/unsupported/wfs-ng/src/main/java/org/geotools/data/wfs/internal/parsers/AbstractGetFeatureResponseParserFactory.java#L108).

I'd like to propose a pull request, where this solution is also applied to the 
Transaction case.

One thing is however not so nice from my point of view: 
AbstractGetFeatureResponseParserFactory.parseException(...) creates an 
ExceptionReportType having a list of code+message pairs. This information is 
converted into a plain WFSException retaining a concatenated string field only. 
Suggestion: Introduce a subclass of WFSException which keeps this information 
separately to be easier accessible for consumers.

Before I start to work on it: 
- Are there better/alternative suggestions? Any comments are welcome.
 
 Best regards, 
 Andreas
 
 
     
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-users mailing list

Please make sure you read the following two resources before posting to this 
list:
- Earning your support instead of buying it, but Ian Turton: 
http://www.ianturton.com/talks/foss4g.html#/
- The GeoServer user list posting guidelines: 
http://geoserver.org/comm/userlist-guidelines.html

Geoserver-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-users

Reply via email to