-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fixes #7252 - Uses gabriel-vasile/mimetype to support additional content types #7262
Conversation
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick and LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM. I've been fuzzing this library for a while and haven't found any crashes so this seems good to go.
Hm. Apparently I didn't push my rebase to the right origin... |
@gowthamgts thanks for the fix! |
Fixes #7252 - Uses gabriel-vasile/mimetype to support additional content types This commit was moved from ipfs/kubo@01e27f9
This fixes #7252 by replacing
http.DetectContentType()
withmimetype.Detect()
from https://github.com/gabriel-vasile/mimetype library.I've tested locally with the ODS files, mp3 files and html files. Please let me know if there's anything further that needs to be done!