-
Notifications
You must be signed in to change notification settings - Fork 42
Description
Hello!
First of all I wanted to thank you for all the work in the library: it's a wonderful piece of software.
While having an an Okta app reviewed we got some feedback about the error responses we are sending, complaining about the status
property (HTTP status code) being an integer and not an string. It came to me as a surprise, but then I looked at the SCIM RFC and indeed, it specifies that the HTTP status code has to be a JSON string: see Section 3.12. of RFC 7644.
My question would be if this has been raised before (I didn't see any issues related) or if it was considered at some point and just decided to use an integer instead.
We can probably change the behavior by converting status
to string inside setAttribute
here, but I'm unsure if that will break consumers of the library. Do you know of a way we can "patch" this locally in an easy way?
Thank you very much!
Current behavior
{
"schemas": ["urn:ietf:params:scim:api:messages:2.0:Error"],
"detail":"Resource 2819c223-7f76-453a-919d-413861904646 not found",
"status": 404
}
Expected behavior
{
"schemas": ["urn:ietf:params:scim:api:messages:2.0:Error"],
"detail":"Resource 2819c223-7f76-453a-919d-413861904646 not found",
"status": "404"
}