-
Notifications
You must be signed in to change notification settings - Fork 1.5k
More robust is_supported_format suffix check #5634
Copy link
Copy link
Closed
Description
Is your feature request related to a problem? Please describe.
Sometimes the data I receive have uppercase suffixes (e.g., IMAGE.PNG), this could be recognized as the lowercase counterpart.
Describe the solution you'd like
Simply change this line
Line 1103 in 025c107
| if len(tokens) == 0 or all("." + s.lower() not in "".join(tokens) for s in suffixes): |
to something like
if len(tokens) == 0 or all("." + s.lower() not in "".join(map(str.lower, tokens)) for s in suffixes):If this is acceptable, I'm happy to open a PR.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels