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

[Erik Wahlström <[e-mail]>]

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]<mailto:[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<http://tools.ietf.org/html/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<http://tools.ietf.org/html/draft-nottingham-http-problem-06#ref-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<http://tools.ietf.org/html/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<http://www.independentid.com/>
[e-mail]<mailto:[e-mail]>



On Jun 24, 2014, at 12:23 PM, Julian Reschke <[e-mail]<mailto:[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]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/scim

_______________________________________________
scim mailing list
[e-mail]<mailto:[e-mail]>
https://www.ietf.org/mailman/listinfo/scim