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

bump cryptography above 41.0.3 #20

Merged
merged 5 commits into from
Nov 13, 2023

Conversation

mnbf9rca
Copy link
Contributor

@mnbf9rca mnbf9rca commented Nov 6, 2023

resolves #19
adds additional tests (load_dotenv() is now tested)

Copy link
Collaborator

@nsnguyen nsnguyen left a comment

Choose a reason for hiding this comment

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

this lgtm, thanks @mnbf9rca !

@mnbf9rca mnbf9rca requested a review from nsnguyen November 8, 2023 07:25
@mnbf9rca
Copy link
Contributor Author

@nsnguyen i can't merge this PR as i dont have write access to the repo - would you mind merging it?

@nsnguyen nsnguyen merged commit 3757ec2 into dotenv-org:master Nov 13, 2023
@nsnguyen
Copy link
Collaborator

it's merged now @mnbf9rca . Thanks 👍

@mnbf9rca mnbf9rca deleted the patch/bump-cryptography branch December 5, 2023 08:20
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.

cryptography<41.0.3 includes vulnerable OpenSSL version
2 participants