joerghoh commented on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-servlets-post/pull/6#issuecomment-730638336
Hi @enapps-enorman I don't think that it makes sense to have this configurable. The HTTP standard gives clear recommendations what status code to use when, and I am just trying to fix current behavior, which I think is incorrect. And honestly I haven't payed much attention to backwards compatibility, because a 500 is always non-actionable from a client perspective. And regarding extensibility: In that case I would introduce new subtypes of PersistenceException (or change the Exception hierarchy used here completely) and map them in the SlingPostServlet to a different status code. But right now I don't see the need for it. ---------------------------------------------------------------- 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: [email protected]
