Eric Norman created SLING-10452:
-----------------------------------

             Summary: adjust HTTP status code for invalid :redirect value for 
modifyAce/deleteAce post request
                 Key: SLING-10452
                 URL: https://issues.apache.org/jira/browse/SLING-10452
             Project: Sling
          Issue Type: Improvement
            Reporter: Eric Norman
            Assignee: Eric Norman
             Fix For: JCR Jackrabbit Access Manager 3.0.10


When the modifyAce/deleteAce servlets receive an illegal or invalid :redirect 
parameter it should return a status code of 
[422|https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/422] instead of 
200 because the request was not fully successful.

Currently, the illegal :redirect parameter value is detected and a warning is 
logged.  The request continues to be processed without the redirect occurring.  
The client has no indication that something went wrong without reviewing the 
server logs.

For example:

Illegal redirect 

{{}}
{code:java}
curl -F principalId=myuser -F privilege@jcr:read=granted -F 
:redirect=https://sling.apache.org 
http://localhost:8080/test/node.modifyAce.html
{code}
 invalid redirect

{{}}
{code:java}
curl -F principalId=myuser -F privilege@jcr:read=granted -F :redirect=https:// 
http://localhost:8080/test/node.modifyAce.html{code}
{{}}

{{}}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to