Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Delete back as a magic string #5933

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

blakeembrey
Copy link
Member

From #2692 (comment), removing "back" as a magic string to have people do this explicitly to ensure redirections are going where they expect.

@blakeembrey
Copy link
Member Author

One option is having the magic string “back” throw an error in 5.0 and remove the error in 6.0, instead of deprecate in 4 and delete in 5.

@blakeembrey blakeembrey merged commit bdd81f8 into expressjs:5.0 Sep 10, 2024
17 checks passed
@blakeembrey blakeembrey deleted the be/delete-redirect-back branch September 10, 2024 03:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants