-
Notifications
You must be signed in to change notification settings - Fork 7
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
mixing formats and media types #183
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Tobias Looker <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not feeling confident with the change of MUST -> SHOULD for media type return. Maybe "MUST .. unless media type is known out-of-band"?
Co-authored-by: Paul Bastian <[email protected]>
|
||
In the successful response, the Status List Provider MUST use the following content-type: | ||
In the successful response, the Status List Provider SHOULD use the following content-type: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this work for you @paulbastian?
In the successful response, the Status List Provider SHOULD use the following content-type: | |
In the successful response, the Status List Provider MUST use the following content-type unless known out of band: |
closes #38
closes #168
closes #137
Rendered preview: https://drafts.oauth.net/draft-ietf-oauth-status-list/c2bo/mixing-formats/draft-ietf-oauth-status-list.html
This PR attempts to add a bit of language about mixing formats and also changes the content-type in the status list response to SHOULD and language around content type parsing that is similar to the language used in HTTP/1.1.
I am happy to remove the media types part if we want a bit more discussion, but when I started writing this, it felt like right path so I decided to include it as a proposal.