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

"Warning" will be removed in next set of HTTP specs #125

Closed
reschke opened this issue Nov 5, 2019 · 12 comments
Closed

"Warning" will be removed in next set of HTTP specs #125

reschke opened this issue Nov 5, 2019 · 12 comments

Comments

@reschke
Copy link

reschke commented Nov 5, 2019

See https://greenbytes.de/tech/webdav/draft-ietf-httpbis-cache-06.html#header.warning.

@dret
Copy link
Owner

dret commented Nov 5, 2019

interesting. it will still exist and be used, right? i am not even sure how this is getting handled in the registry, you cannot really legislate existing header fields out of existence, right?
but given that it may not be recommended anymore: is there a similar mechanism you would suggest? we're trying to surface a "warning", i.e. something that can be ignored, but still may be interesting to look at.

@reschke
Copy link
Author

reschke commented Nov 5, 2019

you may want to bring up the use case; maybe we should reconsider the change wrt "warning"...

@dret
Copy link
Owner

dret commented Nov 5, 2019 via email

@andrecedik
Copy link
Collaborator

Thanks again for the feedback @reschke. What's the best place to get this sorted out? Back in the original issue, should we create a new one (in that repo) or is there another place that is better suited for discussing this?

@reschke
Copy link
Author

reschke commented Nov 5, 2019

....HTTP mailing list...

@dret
Copy link
Owner

dret commented Nov 6, 2019

thinking out loud: another possibility would be to say that if Warning is removed from the caching spec (which may make sense if it is not used in this context), we define and register it in our draft as a more general warning mechanism. the downside to this would be that we would depend on the caching spec being published.

@ioggstream
Copy link
Contributor

Afaik headers can be deprecated but not removed from the table. The removal imho would allow a re-registration.

We can say that warning is deprecated for caching purposes. We should even check if other specs references Warning.

Still I'm not sure if we can modify the syntax of an already existing header, but maybe it's possible

@royfielding
Copy link

royfielding commented Nov 6, 2019

In general, Warning for anything other than gateway issues (i.e., control information that cannot change the content because it doesn't own the content) is a bad design because nobody can determine when and how to implement it consistently. Even for caching, it turns out that nobody is interested in implementing or displaying a Warning about something that they had no control over, since all it does is provoke a user complaint to the entity that has no power or interest in fixing it.

In the API case, warnings belong in the response data format. They have to identify both the semantics of the warning and the extent of the response data to which it applies. Providing that in a header field is nearly impossible, and certainly not supported by the existing syntax of the Warning header field. It has been deprecated both because it looks like a response status code and because the information it supplies is not actionable by the recipient. Placing the warning directly within the data format can be both distinct from the status and actionable within the processing definition of that format.

@dret
Copy link
Owner

dret commented Nov 6, 2019 via email

@dret
Copy link
Owner

dret commented Nov 6, 2019 via email

@ioggstream
Copy link
Contributor

@andrecedik
Copy link
Collaborator

I'm closing this issue, since we've released a new version of the I-D, that now introduces a new header field.

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

5 participants