Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revision of Chapter 6 Error Responses in API Design Guidelines #180

Closed
rartych opened this issue Apr 15, 2024 · 3 comments · Fixed by #213
Closed

Revision of Chapter 6 Error Responses in API Design Guidelines #180

rartych opened this issue Apr 15, 2024 · 3 comments · Fixed by #213
Labels
documentation Improvements or additions to documentation

Comments

@rartych
Copy link
Collaborator

rartych commented Apr 15, 2024

Problem description
There are a lot of questions and issues on Error Response format and content.
Currently the documentation lists the different error codes and cluster them into separate tables, depending on their nature: i) syntax exceptions, ii) service exceptions, and iii) server errors.
The tables do not contain HTTP status values which may introduce ambiguity. There are also differences between the list and the examples in CAMARA_common.yaml

There are also questions on the Error Response format returned by OpenId Connect flows.

Expected action
Update Chapter 6 to provide clear requirements on Error Response content.
Decide what should be changed and aligned in CAMARA_common.yaml.

Indicate when other standard formats can be used.

Additional context
Resolution of Issue #127 can be extension/exemplification of Chapter 6 requirements.

@rartych rartych added the documentation Improvements or additions to documentation label Apr 15, 2024
@PedroDiez
Copy link
Collaborator

Think also Issues:

Have relation with this

@rartych
Copy link
Collaborator Author

rartych commented Apr 23, 2024

I have prepared a table summarizing Chapter 6, CAMARA_common.yaml and QoD, Device Status and Location Verification error responses: https://wiki.camaraproject.org/display/CAM/Error+response+guidelines
Please note that the most of big differences is corrected by #174
I have also added table with proposals included in #127 and #145

We should think about changing chapter 6 by:

Please have a look @hdamker (QoD) @jlurien (Device Location) @bigludo7 (Device Status)

@bigludo7
Copy link
Collaborator

Thanks @rartych and fully support chapter 6 update in particular with #127 & #145

One quick comment: for Device Location & Device Status we did not require update as description is not "normative".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants