Looks reasonable at first glance, but the geotools-devel list is probably
the place to ask rather than the geoserver list.
Ian
On 30 August 2017 at 16:50, Watermeyer, Andreas <
[email protected]> wrote:
> 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/cfe895c1fe84c65904d1aa383496ed
> 2aff7afa42/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
>
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geoserver-users
>
--
Ian Turton
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
