Skip to content

6731-upgrade gunicorn 23.0.0 #6743

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

Merged
merged 1 commit into from
Apr 8, 2025
Merged

Conversation

cnlucas
Copy link
Member

@cnlucas cnlucas commented Apr 3, 2025

Summary (required)

Upgrades gunicorn to remediate snyk vuln

Breaking Changes
refuse requests where the uri field is empty
refuse requests with invalid CR/LR/NUL in heade field values
remove temporary --tolerate-dangerous-framing switch from 22.0

Required reviewers

1 dev

Impacted areas of the application

General components of the application that this PR will affect:

  • gunicorn servers

Related PRs

Related PRs against other branches:

fecgov/openFEC#6187

How to test

Running gunicorn locally has issues for me for CMS

Copy link
Contributor

@rfultz rfultz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't seem to break it!

@pkfec pkfec self-requested a review April 8, 2025 16:31
Copy link
Contributor

@pkfec pkfec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cms app works as expected. thanks @cnlucas

@pkfec pkfec merged commit 325405d into develop Apr 8, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this to ✅ Done in Website project Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[Snyk: High] gunicorn HTTP Request Smuggling (Due: 04/25/25)
3 participants