Skip to content

Add JPEG 2000 mime types - #82

Merged
markwhitaker merged 4 commits into
markwhitaker:mainfrom
CraigN:main
Mar 13, 2026
Merged

Add JPEG 2000 mime types#82
markwhitaker merged 4 commits into
markwhitaker:mainfrom
CraigN:main

Conversation

@CraigN

@CraigN CraigN commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Add support for JPEG 2000 image and video mime types as per RFC3745.

@markwhitaker markwhitaker left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@CraigN Thanks for the PR! And apologies for missing it when you first submitted. Also thanks for your clean commit and for adding tests.

There's just one change I'd like to request: the existing constants are all named to match the MIME type value rather than being descriptive, so please can you rename these accordingly? For example, MimeType.Image.Jpeg2000 should be MimeType.Image.Jp2.

Many thanks.

@CraigN

CraigN commented Mar 12, 2026

Copy link
Copy Markdown
Contributor Author

Apologies for the delayed fix. I have pushed the required changes.

@CraigN
CraigN requested a review from markwhitaker March 12, 2026 10:36

@markwhitaker markwhitaker left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update, @CraigN. Merging this now. :shipit:

@markwhitaker
markwhitaker merged commit e9ef782 into markwhitaker:main Mar 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants