CORS: Add data for wildcard support to certain headers #4577
Conversation
Elchi3
added
the
data:http
ddbeck
left a comment
ddbeck
left a comment
There was a problem hiding this comment.
Sorry for taking so long to get to this one, Florian. One set of corrections to make, but otherwise, this looks great. Thank you!
| "description": "Wildcard (<code>*</code>)", | ||
| "support": { | ||
| "chrome": { | ||
| "version_added": "63" |
There was a problem hiding this comment.
It looks like this one didn't make it to Chrome until 65
I suspect some of the other browsers here need updating as a result
There was a problem hiding this comment.
Woah, great catch! Astonishing how often the info on the Chrome bug tracker is just wrong.
There was a problem hiding this comment.
Yeah, it is funny how inconsistent it is. However, I wasn't clear here: only expose headers was in 65; the others did ship in 63.
There was a problem hiding this comment.
Thanks, should have read the change set more carefully. I've set those other two back to 63.
ddbeck
left a comment
ddbeck
left a comment
There was a problem hiding this comment.
Thanks, Florian! Looks great. 🎉
2 participants
For https://bugzilla.mozilla.org/show_bug.cgi?id=1309358#c29
These headers now accept a wildcard (
*) in certain situations:https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Access-Control-Expose-Headers
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Access-Control-Allow-Methods
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Access-Control-Allow-Headers
C.f. Fyrd/caniuse#4987