Skip to content

Conversation

achaljhawar
Copy link
Contributor

This PR improves the Express 5 migration guide by documenting an important change in MIME type handling that affects users migrating from Express 4 to Express 5.

Issue #1880

When migrating from Express 4 to Express 5, we discovered that JavaScript files served via res.sendFile() now return a Content-Type of "text/javascript" instead of the previous "application/javascript". This change occurs because Express 5 switched from its internal MIME type handling to the mime-types package.

Changes

  • Added explanatory notes to the express.static.mime section explaining that this change affects other methods like res.sendFile()
  • Added a specific note to the res.sendfile() section warning about potential Content-Type differences
  • Provided concrete examples of the JavaScript MIME type change to help users anticipate similar issues

These documentation improvements will help users understand and prepare for potential MIME type differences when upgrading to Express 5, even if they weren't directly using express.static.mime.

Developer's Certificate of Origin 1.1 is acknowledged and agreed to.

@achaljhawar achaljhawar requested review from a team as code owners May 14, 2025 20:47
Copy link

netlify bot commented May 14, 2025

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 2a0f9e9
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/685034fb2bb48100086eeb25
😎 Deploy Preview https://deploy-preview-1903--expressjscom-preview.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.

@achaljhawar
Copy link
Contributor Author

@bjohansebas I have added it back

@achaljhawar
Copy link
Contributor Author

@bjohansebas I have done the changes you asked me to do.

@achaljhawar achaljhawar requested a review from bjohansebas May 17, 2025 06:47
@ShubhamOulkar ShubhamOulkar merged commit 722fb07 into expressjs:gh-pages Jun 23, 2025
9 checks passed
@ShubhamOulkar
Copy link
Member

Thank you @achaljhawar

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.

3 participants