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: DetectContentType does not detect application/x-bzip2 #32508
Comments
Change https://golang.org/cl/181420 mentions this issue: |
Thank you for filing this issue @coreydaley! So we get the signatures from https://mimesniff.spec.whatwg.org/#zip-based-mime-type so perhaps let's first file an issue on the Github issues https://github.com/whatwg/mimesniff and once that's agreed upon we'll LGTM your CL. Thank you. |
I am going to move this to the unreleased milestone since we are depending on browsers and the MIME working group to approve the signatures. |
@odeke-em, Unreleased is for code that's not included in Go releases. I changed to Unplanned. We can do this if the upstream spec is updated. |
Got it, thanks Brad!
…On Mon, Oct 14, 2019 at 1:42 PM Brad Fitzpatrick ***@***.***> wrote:
@odeke-em <https://github.com/odeke-em>, Unreleased is for code that's
not included in Go releases. I changed to Unplanned. We can do this if the
upstream spec is updated.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#32508?email_source=notifications&email_token=ABFL3V5DF64LIC2JHIJZQODQOTKUXA5CNFSM4HWLAZVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBGN3ZY#issuecomment-541908455>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABFL3VYA7TQU5DACSFQ3TL3QOTKUXANCNFSM4HWLAZVA>
.
|
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
extracted the first 512 bytes of a bzip2 file and ran net/http.DetectContentType on it
https://play.golang.org/p/u4WBPl0bsMS
What did you expect to see?
application/x-bzip2
What did you see instead?
application/octet-stream
The text was updated successfully, but these errors were encountered: