Skip to content

fix: support loading .nii.gz files from local file system#824

Merged
PaulHax merged 1 commit intoKitware:mainfrom
PaulHax:fix-nii-gz
Nov 24, 2025
Merged

fix: support loading .nii.gz files from local file system#824
PaulHax merged 1 commit intoKitware:mainfrom
PaulHax:fix-nii-gz

Conversation

@PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Nov 24, 2025

Browsers provide application/gzip MIME type for .nii.gz files. The updateFileMimeType handler now checks if the browser-provided type is in our supported list before skipping. If the type is unsupported (like application/gzip), it continues to check the filename extension to correctly identify compound extensions like .nii.gz, etc.

@finetjul

Browsers provide application/gzip MIME type for .nii.gz files. The
updateFileMimeType handler now checks if the browser-provided type is
in our supported list before skipping. If the type is unsupported
(like application/gzip), it continues to check the filename extension
to correctly identify compound extensions like .nii.gz, .gipl.gz, etc.
@netlify
Copy link

netlify bot commented Nov 24, 2025

Deploy Preview for volview-dev ready!

Name Link
🔨 Latest commit 0c153b0
🔍 Latest deploy log https://app.netlify.com/projects/volview-dev/deploys/69246498e8039e00080f1782
😎 Deploy Preview https://deploy-preview-824--volview-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Member

@finetjul finetjul left a comment

Choose a reason for hiding this comment

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

It works. Thanks !

@PaulHax PaulHax added this pull request to the merge queue Nov 24, 2025
Merged via the queue into Kitware:main with commit fb8f39e Nov 24, 2025
7 checks passed
@PaulHax PaulHax deleted the fix-nii-gz branch November 24, 2025 14:32
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