-
Notifications
You must be signed in to change notification settings - Fork 149
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
Pick the shortest reasonable Prefer / upgrade header #216
Comments
|
Is there any case under which "Prefer: https" might affect the caching policy? If so, then we should split it into a separate header due to limitations of Vary. If not, then +1 for "Prefer: https"... because clarity is good, and TLS is not the only secure transport option - e.g. QUIC? |
@mnot suggested that |
Also, I forgot about QUIC, et al, and discounted opportunistic encryption. Ugh. Is there a name that works? At all? |
If "https" is the only token advertised via Prefer, yes that's fine. However, the fact that its an existing header means that other values may be advertised as well, at which point "Prefer: x,y,z" forces "x,y,z" to become part of the cache key, evne if x and y have no business of being part of the cache key. I went through same exercise with CH, see: igrigorik/http-client-hints#14 |
:(
|
Sec: Y, Sec: 1, HTTPS: Y, HTTPS: 1? |
If it's going to be horrible horrible, |
HTTP: S may be many kinds of horrible, but it's also poetry :) |
I suspect HTTPS: 1 is more googleable though. |
@pde It is, and thanks! Searching "HTTPS: 1 header" led me to |
Luckily that |
We've gone from Prefer: representation=secure to Prefer: https to Prefer: tls. That's pretty good, 11 characters, 13 bytes on the wire, but should we go further?
An even shorter option which I'm stealing from #212 is Upgrade: 1, 10 characters. Or we could do Pref: tls, 8 chacters. Or go all the way to TLS: 1 or HS: 1 for "hypertext secure", 5 characters, 7 bytes on the wire?
There are some aesthetic / clarity arguments for stopping at some point along this line, and I feel I lack data for where to put the pin.
The text was updated successfully, but these errors were encountered: