lib: use req.socket over deprecated req.connection by bjohansebas · Pull Request #6705 · expressjs/express
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 22.6k
Comments
Merged
lib: use req.socket over deprecated req.connection#6705
bjohansebas merged 1 commit intomasterfrom
lib: use req.socket over deprecated req.connection#6705
bjohansebas merged 1 commit intomasterfrom
Conversation
Copy link
Member
bjohansebas
commented
Aug 13, 2025
bjohansebas
commented
req.connection was only deprecated in the documentation, but it’s possible that a runtime deprecation will be added since internally it calls socket, so it’s better to start migrating.
https://nodejs.org/dist/latest/docs/api/deprecations.html#DEP0133
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
bjohansebas
requested a review
from a team
Phillip9587 approved these changes Aug 13, 2025
UlisesGascon approved these changes Aug 13, 2025
bjohansebas
deleted the
connection
branch
UlisesGascon
mentioned this pull request
Merged
davidshen84
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants