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

Embedded Content-Warning can have member name other than warning #144

Open
subukris opened this issue Oct 20, 2020 · 0 comments
Open

Embedded Content-Warning can have member name other than warning #144

subukris opened this issue Oct 20, 2020 · 0 comments

Comments

@subukris
Copy link

Section 5 of the draft mandates the following:

A "warnings" member MUST encapsulate the warnings that will be returned to the client.

In some rare cases when warnings happens to be a member is the response schema, this may break the client because of the ambiguity.

Suggestion: If the member name is allowed to be flexible with warnings as the default, this problem can be overcome. Further we can add an additional optional token in the Content-Warning header to identify the name of the member which holds the embedded warnings.

For ex.

Content-Warning: "embedded-warning"; 1590190500; "_warnings"

For the cases where the warning is embedded into a response xml this could be an xpath.

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

No branches or pull requests

2 participants