Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Update API packages #194

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Update API packages #194

merged 1 commit into from
Sep 9, 2024

Conversation

chouinar
Copy link
Collaborator

@chouinar chouinar commented Sep 9, 2024

Time to review: 1 mins

Changes proposed

Updated API packages

Context for reviewers

Mostly to upgrade flask-cors which had a vulnerability in releases before 5.0

@chouinar chouinar requested a review from jamesbursa as a code owner September 9, 2024 14:06
@github-actions github-actions bot added the api label Sep 9, 2024
Copy link
Collaborator

@coilysiren coilysiren left a comment

Choose a reason for hiding this comment

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

I figure you checked the changelog real quick?

@chouinar
Copy link
Collaborator Author

chouinar commented Sep 9, 2024

I figure you checked the changelog real quick?

Yes - minor changes to most packages. Pydantic and flask-cors are the only significant ones. Our use of Pydantic is pretty minimal outside of env variable management and I didn't see anything noteworthy for that. flask-cors could technically break something as it affects private networks and cors, but can't test that locally - if things break with that, just requires adjusting an env var.

@chouinar chouinar merged commit 7fd6e25 into main Sep 9, 2024
8 checks passed
@chouinar chouinar deleted the chouinar/package-updates-sep9 branch September 9, 2024 19:11
acouch pushed a commit that referenced this pull request Sep 18, 2024
Updated API packages

Mostly to upgrade flask-cors which had a vulnerability in releases
before 5.0
acouch pushed a commit that referenced this pull request Sep 18, 2024
Updated API packages

Mostly to upgrade flask-cors which had a vulnerability in releases
before 5.0
acouch pushed a commit to HHS/simpler-grants-gov that referenced this pull request Sep 18, 2024
Updated API packages

Mostly to upgrade flask-cors which had a vulnerability in releases
before 5.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants