-
-
Notifications
You must be signed in to change notification settings - Fork 25
Update dependency express to v5 #187
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
5a6b865 to
549392f
Compare
74296ea to
2e960bd
Compare
2e960bd to
65790b0
Compare
65790b0 to
c307878
Compare
c3e781f to
a763b29
Compare
a763b29 to
90b6596
Compare
2a9c9c7 to
21bafe5
Compare
21bafe5 to
4fd5c36
Compare
fb1a85f to
5dbf614
Compare
5dbf614 to
2e02cd6
Compare
2e02cd6 to
f7a95e3
Compare
2ee7e95 to
cdd8d91
Compare
cdd8d91 to
5d89007
Compare
34d1195 to
14f8f3a
Compare
3fa6a25 to
31fa1b3
Compare
31fa1b3 to
80c4056
Compare
22bd231 to
06ee2bb
Compare
06ee2bb to
192185a
Compare
192185a to
ca21179
Compare
ca21179 to
e599ff5
Compare
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.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on February 3
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| "devDependencies": { | ||
| "c8": "10.1.3", | ||
| "express": "4.21.1", | ||
| "express": "5.2.1", |
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.
Express v5 wildcard route syntax breaks test code
Upgrading Express from v4 to v5 breaks existing test code in packages/http-stream/test/HttpStream.test.js. The test uses app.post('*', ...) which was valid in Express v4, but Express v5's path-to-regexp v8 requires a different syntax for wildcard routes (e.g., /*splat or /{*path}). This will cause a TypeError when the tests are run, as the * character is no longer treated as a wildcard in route definitions.
This PR contains the following updates:
4.21.2→5.2.14.21.1→5.2.14.17.21→5.0.6Release Notes
expressjs/express (express)
v5.2.1Compare Source
=======================
v5.2.0Compare Source
========================
body-parser@^2.2.1res.redirectwith undefined arguments, Express now emits a warning to help detect calls that pass undefined as the status or URL and make them easier to fix.v5.1.0Compare Source
========================
Uint8Arrayinres.send()res.sendFile()res.links()setprototypeofsafe-bufferutils-mergemethodsdepddebug@^4.4.0body-parser@^2.2.0router@^2.2.0content-type@^1.0.5finalhandler@^2.1.0qs@^6.14.0[email protected][email protected]v5.0.1Compare Source
==========
cookiesemver lock to address CVE-2024-47764v5.0.0Compare Source
=========================
path-is-absolutedependency - usepath.isAbsoluteinsteadres.status()accepts only integers, and input must be greater than 99 and less than 1000RangeError: Invalid status code: ${code}. Status code must be greater than 99 and less than 1000.for inputs outside this rangeTypeError: Invalid status code: ${code}. Status code must be an integer.for non integer inputsres.redirect('back')andres.location('back')is no longer a supported magic string, explicitly usereq.get('Referrer') || '/'.res.clearCookiewill ignore user providedmaxAgeandexpiresoptionsapplication/javascript=>text/javascriptv4.22.1Compare Source
v4.22.0Compare Source
Configuration
📅 Schedule: Branch creation - Monday through Friday ( * * * * 1-5 ) (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.
Note
Dependency upgrades
expressto5.2.1inexpress-test(dev),http-stream(dev), andprometheus-metrics(runtime)@types/expressto5.0.6inprometheus-metricsLockfile refresh (Express 5 tree)
body-parser@^2,finalhandler@^2,router@^2,send@^1.2,serve-static@^2.2,qs@^6.14,type-is@^2,debug@^4, and related updates; removes Express 4-era pinsScope/Risk
Written by Cursor Bugbot for commit e599ff5. This will update automatically on new commits. Configure here.