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

Consider reserializing type attributes, not just parsing them. #123

Closed
wisniewskit opened this issue Mar 30, 2019 · 4 comments
Closed

Consider reserializing type attributes, not just parsing them. #123

wisniewskit opened this issue Mar 30, 2019 · 4 comments

Comments

@wisniewskit
Copy link

While working on Gecko's Fetch compliance, I noted that there are web platform tests which fail because they expect blob types to be passed through the MIME Sniffing standard's serialization. @annevk suggested that there may have been discussions on doing this reserialization, so I'm filing a bug to verify that option.

@inexorabletash
Copy link
Member

Additional context in #43

@annevk
Copy link
Member

annevk commented Apr 5, 2019

Aside from Blob, this also affects slice() and File.

I think what should happen is that we store a MIME type internally (I think @mkruisselbrink was planning on adding internal slots) and the type getter serializes it using https://mimesniff.spec.whatwg.org/#serialize-a-mime-type.

I looked into writing a patch for this briefly, but it would end up being quite the rewrite as these bits of the specification are quite dated.

@annevk
Copy link
Member

annevk commented May 13, 2019

Note that there's tests for this already at /mimesniff/mime-types/parsing.any.js so this is mainly about updating the specification as also discussed at #43 (comment).

@annevk
Copy link
Member

annevk commented Jan 9, 2020

Let's duplicate this into that issue as there's no real need to track it separately.

@annevk annevk closed this as completed Jan 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants