You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Justin Edelson (JIRA)" <ji...@apache.org> on 2016/12/08 14:52:00 UTC
[jira] [Updated] (SLING-6187) Provide a way for a POST request to
assert a set of required SlingPostProcessors
[ https://issues.apache.org/jira/browse/SLING-6187?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Justin Edelson updated SLING-6187:
----------------------------------
Attachment: SLING-6187.patch
I have thought some more about this and a don't see a better way of doing this. Any place outside of the request context could suffer from the same problem as the post processor itself -- it could be absent.
Proposed patch attached.
> Provide a way for a POST request to assert a set of required SlingPostProcessors
> --------------------------------------------------------------------------------
>
> Key: SLING-6187
> URL: https://issues.apache.org/jira/browse/SLING-6187
> Project: Sling
> Issue Type: Improvement
> Components: Servlets
> Reporter: Justin Edelson
> Assignee: Justin Edelson
> Fix For: Servlets Post 2.3.16
>
> Attachments: SLING-6187.patch
>
>
> I would like to add support for a new "special" request parameter understood by the Sling Post Servlet named {{:requiredPostProcessors}}. This parameter may contain a comma-delimited list of names (see below) which *must* be available *at the time the request is processed* in order for the request to be handled. Whether or not those processors _do_ anything or whether the request succeeds or not is a separate question; this is just a preflight check if you will.
> If any of the required SlingPostProcessors are not available, the request will fail with a 501 error.
> The name of a SlingPostProcessor will be defined by a newly defined service registration property {{postProcessor.name}} and default to the simple name of the SlingPostProcessor's implementation class if that property is not defined.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)