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

net/http: use top-level font media type in sniffer #24524

Closed
agnivade opened this issue Mar 25, 2018 · 2 comments
Closed

net/http: use top-level font media type in sniffer #24524

agnivade opened this issue Mar 25, 2018 · 2 comments

Comments

@agnivade
Copy link
Contributor

@agnivade agnivade commented Mar 25, 2018

RFC 8081 declares a top level font media type for all types of fonts.

The same has been incorporated in whatwg/mimesniff#23. Should we move too ?

The changes would require updating the following mime types -

Obsolete New
application/font-ttf font/ttf
application/font-cff font/collection
application/font-off font/otf
application/font-sfnt font/sfnt
application/font-woff font/woff
application/font-woff2  font/woff2

/cc @bradfitz

@mvdan mvdan added the NeedsDecision label Mar 25, 2018
@bradfitz

This comment has been minimized.

Copy link
Contributor

@bradfitz bradfitz commented Mar 25, 2018

Sure. If it doesn't break any browser's font parsing, but from that other bug it sounds like it doesn't.

@gopherbot

This comment has been minimized.

Copy link

@gopherbot gopherbot commented Mar 25, 2018

Change https://golang.org/cl/102458 mentions this issue: net/http: use top-level font media type

@gopherbot gopherbot closed this in c0ce292 Mar 25, 2018
@golang golang locked and limited conversation to collaborators Mar 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.