Skip to content

docs: add send options and methods changes to Express 5 migration guide#2209

Closed
22BRS1317 wants to merge 1 commit intoexpressjs:gh-pagesfrom
22BRS1317:gh-pages
Closed

docs: add send options and methods changes to Express 5 migration guide#2209
22BRS1317 wants to merge 1 commit intoexpressjs:gh-pagesfrom
22BRS1317:gh-pages

Conversation

@22BRS1317
Copy link

Description

Fixes #1868

Added missing documentation to the Express 5 migration guide for
breaking changes from the send package that directly affect
Express users upgrading from v4 to v5.

Changes made

en/guide/migrating-5.md

  • Added new section res.sendFile() and express.static() options
    documenting removed options:

    • hidden → use dotfiles instead
    • from → use root instead
  • Added new section Removed send methods documenting methods
    no longer exported by the send package:

    • send.etag() → use etag option instead
    • send.index() → use index option instead
    • send.maxage() → use maxAge option instead
    • send.root() → use root option instead
  • Updated table of contents with links to both new sections

Motivation

Developers upgrading from Express 4 to Express 5 who relied on
these options and methods would encounter silent breakage with
no guidance in the migration guide. This PR fills that gap.

@22BRS1317 22BRS1317 requested a review from a team as a code owner March 16, 2026 19:19
@netlify
Copy link

netlify bot commented Mar 16, 2026

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 04a7431
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/69b857c6292a0e00084547bc
😎 Deploy Preview https://deploy-preview-2209--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 99 (🟢 up 3 from production)
Accessibility: 87 (no change from production)
Best Practices: 100 (no change from production)
SEO: 94 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@krzysdz
Copy link
Contributor

krzysdz commented Mar 17, 2026

I haven't reviewed it thoroughly yet, but I'm pretty sure that some of the changes to send that you've listed do not directly affect Express users.

@22BRS1317 22BRS1317 closed this Mar 18, 2026
@22BRS1317
Copy link
Author

I haven't reviewed it thoroughly yet, but I'm pretty sure that some of the changes to send that you've listed do not directly affect Express users.

Thank you for the feedback! You're right — after reviewing
this more carefully, I can see that send.etag(), send.root(),
send.index(), and send.maxage() were internal methods of the
send package used by Express internally, not methods that
Express users called directly in their applications.

I'll update the PR to remove that section and keep only the
changes that are directly relevant to Express users (the
res.sendFile() and express.static() options changes).

Sorry for the noise and thanks for the review!

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.

Add the changes in send to the migration guide

2 participants