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

content-warning: add header considerations #132

Open
3 of 4 tasks
ioggstream opened this issue Apr 8, 2020 · 4 comments
Open
3 of 4 tasks

content-warning: add header considerations #132

ioggstream opened this issue Apr 8, 2020 · 4 comments

Comments

@ioggstream
Copy link
Contributor

ioggstream commented Apr 8, 2020

I expect

to have header considerations https://tools.ietf.org/html/rfc7231#section-8.3.1
in content-warning spec

Further questions

  • which is the behavior of Content-Warning in case of no-content responses eg. HEAD?

  • if the "warning" attribute is part of the representation, is that supposed to be returned even by subsequent GET requests? How does this impact with caching? See Relationship with cache: are warnings returned in subsequent requests? #138

  • This header SHOULD/NOT be cached/used in CONNECTION / mentioned in Vary

  • Intermediaries MUST NOT modify its value. (7fb08fa)

@andrecedik
Copy link
Collaborator

I've added a few things to address your points:

  • added section for cache considerations (2af5ba1)
  • added information about/for intermediaries (7fb08fa)

I'm still working on them, so for now this is just an FYI 😉

@ioggstream
Copy link
Contributor Author

Moved the cache/representation parte in another issue, as I think that's complex.

@andrecedik
Copy link
Collaborator

  • Added two sections explaining, which HTTP request methods are allowed in combination with the Content-Warning field and the embedded-warning (a419ba8)

@ioggstream
Copy link
Contributor Author

I left a comment on the above commit: a419ba8#r39174548

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