[
https://issues.apache.org/jira/browse/SLING-8974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17764586#comment-17764586
]
Robert Munteanu commented on SLING-8974:
----------------------------------------
Thinking about it some more, I think 400 would be a good compromise - 'bad
request' since the client should know that the node does not exist.
> Shows a 200 OK for a delete operation even if the node does not exist.
> ----------------------------------------------------------------------
>
> Key: SLING-8974
> URL: https://issues.apache.org/jira/browse/SLING-8974
> Project: Sling
> Issue Type: Bug
> Components: Servlets
> Reporter: Anisha Narang
> Priority: Major
>
> When you try any curl query for a 'delete' operation, it shows a 200 OK even
> even the node does not exist.
> curl query:
> {code:java}
> $ curl -F":operation=delete" http://slinghosturl.com/content/invalid_node
> <html>
> <head>
> <title>Content modified /content/invalid_node</title>
> </head>
> <body>
> <h1>Content modified /content/invalid_node</h1>
> <table>
> <tbody>
> <tr>
> <td>Status</td>
> <td><div id="Status">200</div></td>
> </tr>
> <tr>
> <td>Message</td>
> <td><div id="Message">OK</div></td>
> </tr>
> <tr>
> <td>Location</td>
> <td><a href="/invalid_node"
> id="Location">/invalid_node</a></td>
> </tr>
> <tr>
> <td>Parent Location</td>
> <td><a href="/content" id="ParentLocation">/content</a></td>
> </tr>
> <tr>
> <td>Path</td>
> <td><div id="Path">/content/invalid_node</div></td>
> </tr>
> <tr>
> <td>Referer</td>
> <td><div id="Referer"></div></td>
> </tr>
> <tr>
> <td>ChangeLog</td>
> <td><div
> id="ChangeLog"><pre>deleted("/content/invalid_node");<br/></pre></div></td>
> </tr>
> </tbody>
> </table>
> <p><a href="/invalid_node">Modified Resource</a></p>
> <p><a href="/content">Parent of Modified Resource</a></p>
> </body>
> {code}
> So, even though this node does not exist, there is a 200 OK response for the
> same which is not expected as per the documentation here ->
> [https://sling.apache.org/documentation/bundles/manipulating-content-the-slingpostservlet-servlets-post.html#content-removal]
> Expected result:
> The response should be 404 not found if the not does not exist.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
