-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
use image/bmp #48
Comments
Closing at the issue appears to be in Rails, not this gem. |
I believe that this is a correct change. It appears that at one point |
I have released 3.2022.0105 with this fix. |
ghiculescu
added a commit
to ghiculescu/rails
that referenced
this issue
Apr 18, 2022
This is a follow up to rails#42227 (comment) The mime types database was incorrect regarding `image/bmp`. It has been fixed in mime-types/mime-types-data#48 and in discourse/mini_mime#45 Since a new version of `mini_mime` hasn't been cut yet, some of the tests in this PR look a bit off. But the core issue of not warning users if they use `image/bmp` is resolved.
This was referenced Apr 18, 2022
jyoun-godaddy
pushed a commit
to jyoun-godaddy/activestorage
that referenced
this issue
Jul 5, 2022
This is a follow up to rails/rails#42227 (comment) The mime types database was incorrect regarding `image/bmp`. It has been fixed in mime-types/mime-types-data#48 and in discourse/mini_mime#45 Since a new version of `mini_mime` hasn't been cut yet, some of the tests in this PR look a bit off. But the core issue of not warning users if they use `image/bmp` is resolved.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This gem prefers
image/x-bmp
(reference) butimage/bmp
should be used instead.image/bmp
is documented here: https://www.iana.org/assignments/media-types/image/bmpThe text was updated successfully, but these errors were encountered: