Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bump express from 4.19.2 to 4.20.0 #122

Closed
wants to merge 1 commit into from
Closed

Bump express from 4.19.2 to 4.20.0 #122

wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 11, 2024

Bumps express from 4.19.2 to 4.20.0.

Release notes

Sourced from express's releases.

4.20.0

What's Changed

Important

  • IMPORTANT: The default depth level for parsing URL-encoded data is now 32 (previously was Infinity)
  • Remove link renderization in html while using res.redirect

Other Changes

New Contributors

Full Changelog: expressjs/express@4.19.1...4.20.0

Changelog

Sourced from express's changelog.

4.20.0 / 2024-09-10

  • deps: [email protected]
    • Remove link renderization in html while redirecting
  • deps: [email protected]
    • Remove link renderization in html while redirecting
  • deps: [email protected]
    • add depth option to customize the depth level in the parser
    • IMPORTANT: The default depth level for parsing URL-encoded data is now 32 (previously was Infinity)
  • Remove link renderization in html while using res.redirect
  • deps: [email protected]
    • Adds support for named matching groups in the routes using a regex
    • Adds backtracking protection to parameters without regexes defined
  • deps: encodeurl@~2.0.0
    • Removes encoding of \, |, and ^ to align better with URL spec
  • Deprecate passing options.maxAge and options.expires to res.clearCookie
    • Will be ignored in v5, clearCookie will set a cookie with an expires in the past to instruct clients to delete the cookie
Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

@dependabot
Bump express from 4.19.2 to 4.20.0
942f1f9 
Bumps [express](https://github.com/expressjs/express) from 4.19.2 to 4.20.0.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.19.2...4.20.0)

---
updated-dependencies:
- dependency-name: express
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Sep 11, 2024
@dependabot dependabot bot requested a review from pes10k as a code owner September 11, 2024 03:34
@dependabot Dependabot
Copy link
Contributor Author

dependabot bot commented on behalf of github Sep 11, 2024

Superseded by #124.

@dependabot dependabot bot closed this Sep 11, 2024
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/express-4.20.0 branch September 11, 2024 03:34
@github-actions GitHub Actions
Copy link

[puLL-Merge] - expressjs/[email protected]

Description

This pull request introduces significant improvements to the CI/CD pipeline, updates dependencies, adds new workflows, and contains various documentation and code updates for the Express.js repository. The main goals are to streamline the development process, enhance testing, and maintain the project with the latest best practices and dependencies.

Changes

Changes

.github/workflows/ci.yml

  • Changes to GitHub Actions workflows:
    • Limited branch triggers for push and pull_request.
    • Added a concurrency feature to cancel in-progress jobs when new jobs are triggered for the same PR or branch.
    • Split linting and testing into separate jobs for better clarity and efficiency.
    • Updated Node.js versions and testing strategies for both Ubuntu and Windows environments.
    • Reorganized the jobs for clearer steps and better dependency handling.
    • Added an io.js job to support older io.js versions.

.github/workflows/codeql.yml

  • New CodeQL Workflow:
    • Added for automatic code analysis and security scanning on push and pull_request events for the master branch.

.gitignore

  • Added npm-shrinkwrap.json to ignore list.

.npmrc

  • New configuration file for npm:
    • Added to disable package lock generation (package-lock=false).

Code-Of-Conduct.md

  • Fixed a link reference format.

Contributing.md

  • Updated guidelines on becoming a triager—removed specific request template in favor of a more community-focused approach.
  • Expanded the list of active project repositories and their respective captains.

History.md

  • Updated for version 4.20.0, recording changes in dependencies and deprecation notices.

Readme.md

  • Added a table of contents.
  • Updated badges and links.
  • Updated the list of both Technical Committee (TC) and Triagers, including emeriti members for both groups.

Release-Process.md

  • Added section on pre-release versions (alpha and beta).

Security.md

  • Added sections on pre-release versions and the Express threat model.

Triager-Guide.md

  • Added guidelines for moving issues labeled as question or discuss to GitHub Discussions.

appveyor.yml

  • Added support for Node.js version 22.0.

lib/response.js

  • Introduced deprecation notices for options.maxAge and options.expires in res.clearCookie.
  • Simplified res.location method and removed outdated urlParse usage.

package.json

  • Updated project version to 4.20.0.
  • Updated various dependencies to newer versions.
  • Enhanced test scripts to exclude additional directories.

Test Files

  • Updated tests to reflect changes in parsing and cookie handling.
  • Added tests for named capturing groups in regex.
  • Improved test handling for edge cases and encoding, especially with URLs in headers.

Security Hotspots

  • lib/response.js (Line 823, 906):
    • The changes introduce deprecated parameters in res.clearCookie which are set to be removed in Express 5.0. Ensure existing applications are updated accordingly.

Possible Issues

  • Compatibility:
    • The deprecation of options.maxAge and options.expires in res.clearCookie might affect applications that rely on these parameters.

Overall Assessment

The PR greatly improves the maintainability and security of the Express repository with comprehensive CI pipeline updates, dependency updates, and detailed documentation enhancements. Testing these changes thoroughly in different environments and ensuring backward compatibility will be essential.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pes10k pes10k Awaiting requested review from pes10k pes10k is a code owner

Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file puLL-Merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
0 participants