-
Notifications
You must be signed in to change notification settings - Fork 10.9k
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
[FIX] File uploads for unknown file types but nothing is blocked #18263
[FIX] File uploads for unknown file types but nothing is blocked #18263
Conversation
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.
Thanks for the fix @20051231 🤗
This only works if the block list is empty. If another media type is blocked, files with unrecognized mime type still fail. |
Hello, Do you think this can be solved? If another media type is blocked, files with unrecognized mime type still fail. Thanks!! |
If a blocked mime is not recognized by the browser, the file still need to be blocked so I did not change the logic in this case. I think we might lookup the mime from the file extension in case of unrecognized mime. Or accept both mime and extension in the list? |
It is an option, if the browser does not recognize the mime, search if the file extension is in the whitelist. |
It seems files without mime-types still can not be uploaded. Is there a way to make it work? |
We are also having this problem. The issue is not solved, shouldn't it be open instead of merged? |
I believe it should be open as I have the same issue still. |
just to make sure, do you guys have both blacklist and whitelist settings blank and can still not upload files? if so, please open a new issue |
Proposed changes
I When the block list is empty, all file types should be allowed. But if the browser does not recognize the mime of a file type, the file is still rejected.
Issue(s)
#16836
How to test or reproduce
Screenshots
Types of changes
Checklist
Changelog
Further comments