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.

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

fix: bumps express and body-parser to their latest versions #13961

Closed
wants to merge 2 commits into from

Conversation

bu-michael
Copy link

@bu-michael bu-michael commented Sep 11, 2024

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

Fixes #13960 and #13959

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Fix security vulnerabilities

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@coveralls
Copy link

coveralls commented Sep 11, 2024

Pull Request Test Coverage Report for Build ffd339f1-ecfc-43e0-8c6e-04b3fbe194d2

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.213%

Totals Coverage Status
Change from base Build 0b2170fc-d850-443a-8e46-397cb7d6d6fd: 0.0%
Covered Lines: 6750
Relevant Lines: 7320

💛 - Coveralls

@Kenzo-Wada
Copy link

Kenzo-Wada commented Sep 12, 2024

When will this PR be merged?? I'd appreciate it if you could merge it soon since it relates to a vulnerability in our service..... ❤️

@thomashohn
Copy link

@bu-michael Could you bump express to 4.21.0?

@bu-michael
Copy link
Author

@bu-michael Could you bump express to 4.21.0?

For sure

@LucasGabrielBecker
Copy link

+1 waiting for this

1 similar comment
@brunoventurazup
Copy link

+1 waiting for this

@bu-michael
Copy link
Author

@thomashohn Bumped express to 4.21.0

@thomashohn
Copy link

@bu-michael - nice - unfortunately I'm no maintainer of this repo - but would like have have it merged ASAP too :)

@kamilmysliwiec
Copy link
Member

Fixed in #13971

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vulnerability in Express < 4.20.0
7 participants