CXF-8966 : Validation of nillable tags fails #1558
CXF-8966 : Validation of nillable tags fails #1558f2par0 wants to merge 1 commit intoapache:mainfrom
Conversation
1b8d4e0 to
39c98a0
Compare
|
@f2par0 thank you for attempt to fix the issue, I believe the issue is identified here FasterXML/woodstox#179 and it seems like the issue is not within CXF or Woodstox but MSV |
|
Yes, thanks for finding this issue. So this PR is more a workaround to avoid to call MSV writer. |
I would prefer the wait for fix to come from upstream if possible and not merge any workarounds |
|
FasterXML/woodstox#179 is a Woodstox issue after all - see FasterXML/woodstox#179 (comment) |
|
A fix for FasterXML/woodstox#179 is underway: FasterXML/woodstox#187 |
|
Why not change in general the approach to run the validation on reader-side instead of on a (dummy) writer during a additional copy process? I like the approach of these PR here to change..... I've tried it by myself here in a similar way. |
|
Sorry for the delay, FasterXML/woodstox#187 got stuck due to legal issues with the CLA. I hope those get resolved soonish. |
Change xml validation strategy to use the Woodstox validation when reading instead of having a reader and a writer to a nulloutput because this method is not able to handle the xsi:nil=true attributes.
Previous Algorithm :
Suggested algorithm :