jmapio / jmap

JSON Meta Application Protocol Specification (JMAP)
http://jmap.io
Apache License 2.0
1.25k stars 82 forks source link

New error codes #231

Closed ksmurchison closed 6 years ago

ksmurchison commented 6 years ago

After adding support for RFC 7807 error responses to Cyrus JMAP, it looks like we may want additional codes to indicate that the request exceeds a server limit (e.g. maxSizeRequest) and that a creationId is invalid. In both cases, a 'detail' element can contain additional info.

Also, for readability, should we hyphenate multi-word codes?
E.g. not-json, not-request, unknown-capability

neilj commented 6 years ago

Not sure what you mean by a "creationId is invalid", but that sounds like a method-level error.

I agree we probably want some kind of "exceeds limit" response when the whole HTTP request is rejected due to maxSizeRequest, maxCallsInRequest or maxConcurrentRequests constraints.