Clarification on non-mandatory error statuses, 429 made non-madatory#374
Clarification on non-mandatory error statuses, 429 made non-madatory#374rartych merged 5 commits intocamaraproject:mainfrom
Conversation
429 made non-mandatory
|
I like to propose a wording within the API design guideline in line with this comment Not include line 886 (indeed 501 is not a mandatory code in the way like 401, and 403 are) and after line 811 include specific section, in this way:
I think it respects the spirit targeted for this code and also refelcts the relevant guidelines derived from TSC. |
|
@PedroDiez I am fine with not mixing 501 with 401 and 403. I propose to extend the point starting in line 806 with the special considerations.
|
Perfect Rafal! |
Proposed changes have been applied to the PR |
What type of PR is this?
What this PR does / why we need it:
Clarification proposed in the issue discussion #355.
429 made non-madatory - #365
501 mandatory, if and only if an endpoint is optional for implementations
Which issue(s) this PR fixes:
Fixes #355 #365 #246
Does this PR introduce a breaking change?
Special notes for reviewers:
The discussion in #355 concludes to keep the error definitions in CAMARA_common.yaml.
Adding comments to each error schema (saying if it is mandatory) can be considered.
Changelog input
Additional documentation
This section can be blank.