Re: [scim] [INPUT REQUESTED] Re: Detailed error and status codes

[Phil Hunt <[e-mail]>]

Yes. :-)

Phil

> On Jun 25, 2014, at 0:25, Erik Wahlström <[e-mail]> wrote:
> 
> So that would make it look like the following right?
> 
> {
>   "schemas": ["urn:scim:api:messages:2.0:Error"],
>   "Error":
>     {
>       "type": "urn:scim:api:error:2.0:mutability",
>       "title": “Attribute is readOnly.",
>       "detail": “Attribute 'id' is readOnly and can’t be changed in a PATCH request.",
>       “status": 400  
>     }
> }
> 
> I replaced Errors with Error.
> Also replaced urn:scim:error:api:2.0:mutability with urn:scim:api:error:2.0:mutability
> 
> That works for me.
> / Erik
> 
> 
> 
>> On 25 Jun 2014, at 01:30, Phil Hunt <[e-mail]> wrote:
>> 
>> Note: July 04 is the LAST day for us to potentially submit updates for the IETF90 meeting.  This gives us about 1.5 weeks to close off the error issues and update the drafts.  If you have a moment please take a look and comment as soon as possible...
>> 
>> Julian thanks for the suggestion.  For the rest of the group’s benefit, the draft Julian referenced suggests the following attributes be returned in a standardized detailed error JSON body…
>> 
>>>    A problem details object MAY have the following members:
>>> 
>>>    o  "type" (string) - An absolute URI [RFC3986] that identifies the
>>>       problem type.  When dereferenced, it SHOULD provide human-readable
>>>       documentation for the problem type (e.g., using HTML
>>>       [W3C.REC-html401-19991224]).  When this member is not present, its
>>>       value is assumed to be "about:blank".
>>>    o  "title" (string) - A short, human-readable summary of the problem
>>>       type.  It SHOULD NOT change from occurrence to occurrence of the
>>>       problem, except for purposes of localisation.
>>>    o  "status" (number) - The HTTP status code ([RFC2616], Section 6)
>>>       generated by the origin server for this occurrence of the problem.
>>>    o  "detail" (string) - An human readable explanation specific to this
>>>       occurrence of the problem.
>>>    o  "instance" (string) - An absolute URI that identifies the specific
>>>       occurrence of the problem.  It may or may not yield further
>>>       information if dereferenced.
>> 
>> Here is the corresponding example error response:
>>>    HTTP/1.1 403 Forbidden
>>>    Content-Type: application/problem+json
>>>    Content-Language: en
>>> 
>>>    {
>>>     "type": "http://example.com/probs/out-of-credit",
>>>     "title": "You do not have enough credit.",
>>>     "detail": "Your current balance is 30, but that costs 50.",
>>>     "instance": "http://example.net/account/12345/msgs/abc",
>>>     "balance": 30,
>>>     "accounts": ["http://example.net/account/12345",
>>>                  "http://example.net/account/67890"]
>>>    }
>> 
>> 
>> Julian, what is the value of having “type” be a URI?  Why not just have a SCIM specific attribute (scimType) and use simple keywords? (asking mainly for the WG’s benefit). I guess the strong case is that by standardizing HTTP responses, client code gets simplified. However, using standard attributes creates the namespace conflict issue.  So we need URI based error codes.
>> 
>> The SCIM specs could adopt the “type”, “detail”, and “status”. We would register a namespace of urn:scim:error:apil:2.0 in the SCIM IANA section in addition to api:messages, schema:core, etc.  This would give detail error types of:
>>   urn:scim:error:api:2.0:mutability, urnscim:error:api:2.0:filter, etc.
>> 
>> The other items, like title, instance, etc could be optional or just omitted from SCIM spec. —> we would just indicate that clients should expect attributes within the JSON message other than those defined by the SCIM drafts.
>> 
>> IMPORTANT NOTE:  the format suggested by Julian allows for only one error at a time.  Erik had commented he prefers to return only one. I tend to agree. In the case of BULK requests, this is not really an issue since each BULK operation gets its own response block.
>> 
>> In the interests of time, any objections to using type (with URI based error codes), detail, status, per above and limiting to ONE per response?  
>> 
>> I know these are normative changes….however I’m pretty sure we can’t avoid changes in the final document clean up whether or not we adopt the proposed standardized error response.
>> 
>> Cheers,
>> 
>> Phil
>> 
>> @independentid
>> www.independentid.com
>> [e-mail]
>> 
>> 
>> 
>> On Jun 24, 2014, at 12:23 PM, Julian Reschke <[e-mail]> wrote:
>> 
>>>> On 2014-06-24 21:01, Phil Hunt wrote:
>>>> With regards to tickets 37, 46, 67, it looks like we will have some
>>>> normative changes required.  I am also expecting new SCIM error codes as
>>>> we look at the possible reasons a server might return Bad Request or
>>>> other HTTP status codes.
>>>> 
>>>> Currently, the example response from PATCH, we use “error” to indicate a
>>>> detailed error.
>>>> 
>>>> * "error":"mutability"*
>>>> 
>>>> >From Response Codes Section, no detail error is defined:
>>>> 
>>>> *HTTP/1.1 404 NOT FOUND*
>>>> 
>>>> {
>>>>   "schemas": ["urn:scim:api:messages:2.0:Error"],
>>>>   "Errors":[
>>>>     {
>>>>       *"description":"Resource 2819c223-7f76-453a-919d-413861904646 not found",
>>>>       "code":"404"*
>>>>     }
>>>>   ]
>>>> }
>>>> 
>>>> 
>>>> Doing a search around Twitter, AWS, and many others, I see that “code”
>>>> is often used to indicate a detailed error message (“mutability”) and
>>>> status is used to indicate HTTP status.
>>>> 
>>>> In an attempt to bring consistency across the API document, I am
>>>> proposing we use http_status and scim_code. This will help make clear
>>>> what we are referring to and allow existing implementations to co-exist
>>>> for a while (by having different names).
>>>> 
>>>> _Are there any objections to normalizing the spec around following
>>>> format and attributes?_
>>>> 
>>>> Proposed example:
>>>> 
>>>> *HTTP/1.1 400 Bad Request*
>>>> 
>>>>    Content-Type: application/scim+json;charset=UTF-8
>>>>    Cache-Control: no-store
>>>>    Pragma: no-cache
>>>> 
>>>>    {
>>>>      "schemas": ["urn:scim:api:messages:2.0:Error"],
>>>>      "Errors":[
>>>>        {
>>>>          *“scim_code":"mutability”,*
>>>> 
>>>> *          “http_status”:400,*
>>>>          "error_description":"Attribute 'id' is readOnly."
>>>>        },
>>>> 
>>>>         . . .
>>>> 
>>>>      ]
>>>> 
>>>>    }
>>>> ...
>>> 
>>> You may want to read <http://tools.ietf.org/html/draft-nottingham-http-problem-06>...
>>> 
>>> Best regards, Julian
>>> 
>>> _______________________________________________
>>> scim mailing list
>>> [e-mail]
>>> https://www.ietf.org/mailman/listinfo/scim
>> 
>> _______________________________________________
>> scim mailing list
>> [e-mail]
>> https://www.ietf.org/mailman/listinfo/scim
> 
> _______________________________________________
> scim mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/scim