Patch for nonexistent user causes NPE - returns status 500 and logs exception with stack-trace

[ondrejzivotsky]

Updating nonexistent user causes NullPointerException. In the log is full stack-trace of NPE. I would expect HTTP 404 is returned (like in the case of the GET call).

Request:

> curl --location --request PATCH 'http://localhost:8080/scim/v2/Users/nonexistent-user-id' --header 'Content-Type: application/scim+json' \
--data '{
    "schemas":["urn:ietf:params:scim:api:messages:2.0:PatchOp"],
    "Operations":[
        {
      "op": "replace",
      "path": "active",
      "value": false
    }
    ]
}'

Response:

{
    "detail": "An internal error has occurred.",
    "schemas": [
        "urn:ietf:params:scim:api:messages:2.0:Error"
    ],
    "status": 500
}

Expected response:

{
  "detail" : "the 'User' resource with id 'nonexistent-user-id' does not exist",
  "schemas" : [ "urn:ietf:params:scim:api:messages:2.0:Error" ],
  "status" : 404
}