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

Require cryptography >= 1.4, replace deprecated functions #285

Merged

Conversation

bluetech
Copy link
Contributor

@bluetech bluetech commented Aug 4, 2017

The signer/verifier interfaces are deprecated in cryptography==2.0. The
alternative sign/verify interfaces were added in cryptography==1.4.

@coveralls
Copy link

coveralls commented Aug 4, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 2b51474 on bluetech:replace-cryptography-deprecated-again into 5fbc484 on jpadilla:master.

@mark-adams
Copy link
Contributor

mark-adams commented Aug 25, 2017

This looks good but it needs to be added to the CHANGELOG before it can be merged.

The signer/verifier interfaces are deprecated in cryptography==2.0. The
alternative sign/verify interfaces were added in cryptography==1.4.
@bluetech bluetech force-pushed the replace-cryptography-deprecated-again branch from 2b51474 to ed27376 Compare August 25, 2017 18:29
@bluetech
Copy link
Contributor Author

@mark-adams I added CHANGELOG entries.

@coveralls
Copy link

coveralls commented Aug 25, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling ed27376 on bluetech:replace-cryptography-deprecated-again into 5fbc484 on jpadilla:master.

@mark-adams mark-adams merged commit 3def8d8 into jpadilla:master Aug 25, 2017
@mark-adams
Copy link
Contributor

@bluetech Thanks for the contribution!

bamarni added a commit to dcos/dcos-core-cli that referenced this pull request Aug 3, 2018
It started to trigger a warning after the cryptography bump.

cf. jpadilla/pyjwt#285

https://jira.mesosphere.com/browse/DCOS-40139
bamarni added a commit to dcos/dcos-core-cli that referenced this pull request Aug 3, 2018
It started to trigger a warning after the cryptography bump.

cf. jpadilla/pyjwt#285

https://jira.mesosphere.com/browse/DCOS-40139
bamarni added a commit to dcos/dcos-core-cli that referenced this pull request Aug 16, 2018
It started to trigger a warning after the cryptography bump.

cf. jpadilla/pyjwt#285

https://jira.mesosphere.com/browse/DCOS-40139
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.

None yet

3 participants