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

[security] set min version of jinja2, cryptography, python #229

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

ryantm
Copy link
Collaborator

@ryantm ryantm commented Feb 19, 2025

Why

  • There were Dependabot alerts for the indirect dependencies jinja2 and cryptography.

What changed

  • Set the min versions in pyproject.toml that resolve the security issue
  • Had to bump python min version too because cryptography didn't like 3.9

Test plan

  • CI passes

Rollout

Describe any procedures or requirements needed to roll this out safely (or check the box below)

  • This is fully backward and forward compatible

@ryantm ryantm requested a review from a team as a code owner February 19, 2025 22:45
@ryantm ryantm requested review from vlinkz and removed request for a team February 19, 2025 22:45
@ryantm ryantm enabled auto-merge (squash) February 19, 2025 22:45
Why
===
* There were Dependabot alerts for the indirect dependencies jinja2
and cryptography.

What changed
===
* Set the min versions in pyproject.toml that resolve the security
issue
* Had to bump python min version too because cryptography didn't like
3.9
* Update semaphore python version

Test plan
===
* CI passes
@ryantm ryantm force-pushed the rtm-02-19-min-deps-security branch from 147f9fc to 6c250f3 Compare February 19, 2025 22:46
@ryantm ryantm merged commit ba32313 into master Feb 19, 2025
6 checks passed
@ryantm ryantm deleted the rtm-02-19-min-deps-security branch February 19, 2025 23:48
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.

2 participants