mkarg commented on issue #52: JOHNZON-281 JAX-RS Provider MUST throw NoContentException URL: https://github.com/apache/johnzon/pull/52#issuecomment-536211791 @rmannibucau Thanks for that! Unfortunately I gotta go away for few hours (hopefully will continue later this night). In fact, before I finish the code, three questions: * Once we catch the exception, the provider won't detect the zero-byte-stream when running ontop of other JSON-B implementations. Hence the provider is Johnzon-only. If this is intended, I will add a note in the JavaDocs. * Shall we really compare exception message text? That's pretty ugly. I'd prefer to actually link the code with JSON-P and catch `NothingToRead` class. * A MBR is still an MBR, hence a JAX-RS component, hence should comply with its rules. I could simply catch `NothingToRead` in that "other" MBR, too. This would be more clean than to say "this is not intended to be JAX-RS 2.1 compatible". Wdyt?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
