Fixes #45, #46 - Fix a few consistency issues#47
Conversation
…ithm The tokens should already have that as part of its name, if relevant.
| [[!PERMISSIONS-POLICY]] [[!CLIENT-HINTS]] | ||
|
|
||
| <li><p>Set <var>hintName</var> to "Sec-" concatenated with <var>hintName</var>. | ||
| <div class=issue>We need to figure out if we really want a `Sec-` prefix, and if so also exempt it from CORS.</div> |
There was a problem hiding this comment.
Should we file an issue related to the CORS question, assuming it's still an open issue?
There was a problem hiding this comment.
Might be good to open an issue to track this on our end as well though
|
This removes the If so, I can do something slightly different and add some language to that effect and special case the set of "legacy" tokens (Save-Data, DRP, Viewport-Width, Width, Device-Memory, Downlink, RTT, ECT). PTAL @yoavweiss, thanks! |
| [[!PERMISSIONS-POLICY]] [[!CLIENT-HINTS]] | ||
|
|
||
| <li><p>Set <var>hintName</var> to "Sec-" concatenated with <var>hintName</var>. | ||
| <div class=issue>We need to figure out if we really want a `Sec-` prefix, and if so also exempt it from CORS.</div> |
There was a problem hiding this comment.
Might be good to open an issue to track this on our end as well though
|
Thanks, @yoavweiss. I filed #49 so feel free to merge here when you have a chance. |
2 participants
Fixes a few spec inconsistencies, related to UA-CH tokens and prefixes in the Request processing algorithm.