docs: document removed options for res.sendFile() and express.static() in Express 5 migration guide by chakri11-git · Pull Request #2211 · expressjs/expressjs.com

chakri11-git · 2026-03-18T16:15:59Z

Description

Changes made

en/guide/migrating-5.md

Added new section documenting removed options for res.sendFile()
and express.static():
- hidden → use dotfiles option instead
- from → use root option instead
Added TOC entry linking to the new section

Motivation

Developers upgrading from Express 4 to Express 5 who used the hidden
or from options in res.sendFile() or express.static() would
encounter silent breakage with no guidance in the migration guide.
This PR documents those removed options so users know what to replace
them with.

Note

Previous PR #2209 was closed due to including internal send package
methods that were not part of the public Express API. This PR contains
only the user-facing option changes.

netlify · 2026-03-18T16:16:06Z

✅ Deploy Preview for expressjscom-preview ready!

Name	Link
🔨 Latest commit	`b29e7f6`
🔍 Latest deploy log	https://app.netlify.com/projects/expressjscom-preview/deploys/69d08f500031760008e5293a
😎 Deploy Preview	https://deploy-preview-2211--expressjscom-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 99 (🟢 up 3 from production) Accessibility: 100 (🟢 up 13 from production) Best Practices: 100 (no change from production) SEO: 100 (🟢 up 6 from production) PWA: 80 (🟢 up 50 from production) View the detailed breakdown and full score reports

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

krzysdz

It should be also mentioned for res.sendFile() that the default for dotfiles changed to "ignore". I think it could be merged with the "express.static dotfiles" section.

There's also the switch to mime-types (pillarjs/send#192), which affects res.sendFile() and express.static, but is mentioned (twice) in sections about res.sendfile() (lowercase f) and express.static.mime, so there's a (small) chance someone could miss it when looking only at the ToC. I'm not sure if this is something that really needs changing, but I'll leave it here for reviewers to decide.

By the way you don't have to close a PR and open a new one after pushing new commits. You could have kept #2209 and it would show the new changes automatically after updating source branch.

bjohansebas · 2026-03-22T16:20:41Z

Aside from @krzysdz’s comments, this needs to be released for the new documentation. Since I don’t have time to do documentation backports, could you update the reference to the new redesign branch in this same PR? Otherwise, I’ll do it when I start modifying the documentation.

chakri11-git · 2026-03-22T16:50:36Z

Hi! I've addressed all the feedback:

Removed the internal send.* methods section
Combined res.sendFile() and express.static() dotfiles
into a single section, explicitly mentioning the new default
value of "ignore"
Consolidated the duplicate MIME type documentation from
res.sendfile() and express.static.mime sections into one
dedicated section with updated TOC entry
Updated _includes/github-edit-btn.html to point to the
redesign branch as requested

Please let me know if any further changes are needed. Thank you!

krzysdz · 2026-03-22T19:02:23Z

+- HTML files (.html): now "text/html; charset=utf-8" instead of just "text/html"
+- XML files (.xml): now "application/xml" instead of "text/xml"
+- Font files (.woff): now "font/woff" instead of "application/font-woff"
+- SVG files (.svg): now "image/svg+xml" instead of "application/svg+xml"


Did this really change? I looked at mime@1.6.0 types database and I see image/svg+xml as the type for .svg and .svgz extensions.

Now I see that the duplicate sections weren't identical and one of them mentioned SVG. It turns out #1903 was vibecoded and is AI slop. Almost everything in this section is incorrect. Of all listed extensions only one has a different mime type - .js changed from application/javascript to text/javascript. HTML, CSS, JS and JSON also always had charset specified in Content-Type, so this didn't change either.

krzysdz · 2026-03-22T19:07:30Z

+ {{ site.github.repository_url }}/tree/redesign/_includes/api/en/{{page.version}}
 {% else %}
- {{ site.github.repository_url }}/edit/gh-pages/{{ page.path }}
+ {{ site.github.repository_url }}/edit/redesign/{{ page.path }}


I think @bjohansebas meant changing the PR base branch to redesign, not changing the edit button URL.

Thank you for the review!
PR base branch — I have changed the base branch of this PR
from gh-pages to redesign as requested.
SVG MIME type — After checking the Express 4 source code,
application/svg+xml was indeed the type used in Express 4,
so the change to image/svg+xml in Express 5 is accurate and
I have kept it in the documentation.
Edit button — Reverted _includes/github-edit-btn.html
back to gh-pages since the redesign branch is now the base
of the PR itself.
Please let me know if any further changes are needed. Thank you!

Yep, that’s what I meant.

I have done those changes

PR base branch — I have changed the base branch of this PR
from gh-pages to redesign as requested.
You did not, but @bjohansebas did that for you 30 min ago. You'll need to rebase your branch on redesign (git rebase redesign).
SVG MIME type — After checking the Express 4 source code,
application/svg+xml was indeed the type used in Express 4,
so the change to image/svg+xml in Express 5 is accurate and
I have kept it in the documentation.
Please don't blindly trust LLMs. This is not something done by Express directly, but mime (Express 4) or mime-types (and mime-db) (Express 5). I sent you a direct link to the mime type and file extension mapping from mime@1.6.0, where you can find what type is used for .svg in Express 4. You could also run a simple server to verify that Express 4 did not use application/svg+xml, but something else.

Thank you for the feedback and sorry for the incorrect SVG claim.
I verified using mime@1.6.0 directly:
node -e "const mime = require('mime'); console.log(mime.lookup('svg'));"
// output: image/svg+xml
This confirms SVG already returned "image/svg+xml" in Express 4 so
it was not a change. I have removed that line.
I have also rebased the branch on upstream/redesign — the PR now
shows 1 file changed with only the relevant changes to
legacy/en/guide/migrating-5.md.
Sorry for the confusion earlier and thank you for your patience!

bjohansebas · 2026-03-22T21:50:59Z

Note that I haven’t reviewed what this PR does, and I won’t do so until next weekend when I go on vacation to take the final steps of the redesign

bjohansebas · 2026-03-22T21:53:55Z

@22BRS1317 now do the rebase

…BRS1317/gh-pages

…ss.static() options

…press 5

…ndFile()

chakri11-git requested a review from a team as a code owner March 18, 2026 16:16

krzysdz reviewed Mar 19, 2026
View reviewed changes

krzysdz reviewed Mar 22, 2026
View reviewed changes

bjohansebas changed the base branch from gh-pages to redesignMarch 22, 2026 21:53

docs: document removed options for res.sendFile() and express.static()
2663ffd

chakri11-git force-pushed the gh-pages branch from 14b4860 to 2663ffdCompare March 22, 2026 22:43

chakri11-gitand others added 6 commits March 23, 2026 04:18

docs: remove incorrect SVG MIME type change
ae918dc

Merge branch 'redesign' into gh-pages
cfa9823

Merge branch 'redesign' of github.com:expressjs/expressjs.com into 22…
5ac1cbb
…BRS1317/gh-pages

docs: update migration guide with changes to res.sendFile() and expre…
987182f
…ss.static() options

docs: update migration guide with changes to MIME type handling in Ex…
90868eb
…press 5

docs: clarify MIME type changes affecting express.static() and res.se…
b29e7f6
…ndFile()

bjohansebas merged commit e75a587 into expressjs:redesignApr 4, 2026
9 of 10 checks passed

bjohansebas mentioned this pull request Apr 4, 2026
chore: redesign in Astro #2169
Merged

krzysdz mentioned this pull request May 27, 2026
Add the changes in send to the migration guide #1868
Closed

Uh oh!

Uh oh!

Conversation

chakri11-git commented Mar 18, 2026

Description

Changes made

Motivation

Note

Uh oh!

netlifyBot commented Mar 18, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for expressjscom-preview ready!

Uh oh!

krzysdz left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bjohansebas commented Mar 22, 2026

Uh oh!

chakri11-git commented Mar 22, 2026

Uh oh!

krzysdzMar 22, 2026

Choose a reason for hiding this comment

Uh oh!

krzysdzMar 23, 2026

Choose a reason for hiding this comment

Uh oh!

krzysdzMar 22, 2026

Choose a reason for hiding this comment

Uh oh!

chakri11-gitMar 22, 2026

Choose a reason for hiding this comment

Uh oh!

bjohansebasMar 22, 2026

Choose a reason for hiding this comment

Uh oh!

chakri11-gitMar 22, 2026

Choose a reason for hiding this comment

Uh oh!

krzysdzMar 22, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chakri11-gitMar 22, 2026

Choose a reason for hiding this comment

Uh oh!

bjohansebas commented Mar 22, 2026

Uh oh!

bjohansebas commented Mar 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

netlifyBot commented Mar 18, 2026•
edited
Loading

krzysdz left a comment •
edited
Loading

krzysdzMar 22, 2026•
edited
Loading