enapps-enorman edited a comment on pull request #6: URL: https://github.com/apache/sling-org-apache-sling-servlets-post/pull/6#issuecomment-733247057
> 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? > Well, yes. The problem you are attempting to fix is the feature request that the proposed solution would solve. > 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. I have no problem with changing to a better default as long as it is well documented and the end users are given fair warning. ---------------------------------------------------------------- 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]
