Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upImplement MIME sniffing #14024
Implement MIME sniffing #14024
Comments
|
N.B. we should revert the changes to mime_sniffing_font_context.html from #13931 as part of this work. |
|
I’m taking a look at it right now and seeing what can be done to simplify and solidify the implementation. |
|
Does |
|
Looks like it, @KiChjang. Personally I’d like it decomposed down into smaller submodules. Then again that may just be bikeshedding on my part. Besides, “if it ain’t broke, don’t fix it”. |
|
I'm actually more inclined to make it conform more to the algorithms as defined by the spec, but I'm not sure how much effort is required to do so. |
|
It's supposed to already conform. This issue is specifically talking about integrating the MIME sniffing into the fetch network stack, now that the old network stack that supported it has been removed. |
|
This is blocking #14521 if we want to do it properly, since nosniff needs to compare the |
|
If integrating this into the fetch stack at all is even the right answer; it might be better to integrate it into the code that calls into the fetch stack instead. After all, the fetch stack doesn't interpret the bytes it returns. |
There's https://mimesniff.spec.whatwg.org/ , but it's not quite clear to me where it's invoked.