Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2k
docs: remove deprecated res.redirect('back') from Express 3.x and 4.x API docs#2126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
netlifybot commentedDec 2, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview forexpressjscom-preview ready!
To edit notification comments on pull requests, go to yourNetlify project configuration. |
krzysdz commentedDec 2, 2025
Deprecated API should be marked as such (and it is), not removed from documentation if it still exists. It was removed from Express 5 docs, because it does not exist in Express 5. If you see somewhere in old code Your PR not only removes documentation for a feature that still exists (talking about Express 4), but also the recommendation how to replace it. |
vini4721 commentedDec 3, 2025
Thank you for the feedback@krzysdz! I understand the issue now. I need clarification: Issue#1834 mentions that
I did NOT find a separate 5.x API documentation file for Could you please point me to which specific file contains the Express 5 API reference that needs to be updated? I want to fix this correctly by only removing it from Express 5 docs while keeping it in 3.x/4.x docs where it's properly marked as deprecated. |
The deprecated feature should remain documented in Express 3.x and 4.xwhere it still exists. Only Express 5 removed this feature.
🚦 Lighthouse Results (Mobile & Desktop)
|
krzysdz commentedDec 3, 2025
The 5.x documentation for |
bjohansebas left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Hi, thanks for your contribution. This PR doesn’t seem to have any changes, unless GitHub is broken and not showing them to me.
If it’s as the title says, we don’t need to remove those references in the other APIs, as@krzysdz already mentioned.
Fixes#1834
Changes
res.redirect('back')example from Express 3.x API documentationres.redirect('back')example and deprecation notice from Express 4.x API documentationReason
The
res.redirect('back')method was deprecated in Express 4.x and completely removed in Express 5.x. These examples should no longer appear in the API documentation as they may confuse users.Related
'back'is no longer supported and recommend usingreq.get('Referrer') || '/'instead