joerghoh commented on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-servlets-post/pull/6#issuecomment-733045651
Hi @enapps-enorman Regarding extensibility: I don't see any request to make the existing implementation more extensible regarding the handling of the return code, so I am bit hesitant to add a bunch of code, which we don't need (yet). Are you aware of features or feature requests which require this? And regarding backwards compatibility of the change: Yes, we could easily add a flag (via OSGI propertieis), which reverts to the old behavior and return a 500. But I would definitely change the default to a 405. ---------------------------------------------------------------- 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