Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: support jfif format #307

Closed
wants to merge 1 commit into from
Closed

feat: support jfif format #307

wants to merge 1 commit into from

Conversation

ximing
Copy link

@ximing ximing commented Apr 1, 2024

Details about the JFIF file format, please refer to https://docs.fileformat.com/image/jfif/

@broofa
Copy link
Owner

broofa commented Apr 1, 2024 •
edited
Loading

jshttp/mime-db#283

Edit: Better mime-db issue here: jshttp/mime-db#291

@broofa broofa closed this Apr 1, 2024
@broofa
Copy link
Owner

broofa commented Apr 1, 2024

tldr: This module depends on mime-db. mime-db only pulls definitions from IANA, NGINX, or Apache... none of which list jfif as a file extension. Probably the best course of action here will be to go through IANA to update the JPEG registration and/or file a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ximing @broofa