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

rsakey: add rsa-sha2-256/512 signature algorithms #70

Closed
wants to merge 1 commit into from
Closed

Conversation

ploxiln
Copy link
Owner

@ploxiln ploxiln commented Mar 16, 2020

This change adds support for the rsa-sha2-256 and rsa-sha2-512 signature
algorithms specified in RFC 8332. New RSA key type classes are added,
with the only difference compared to RSAKey that signing and verifying
blobs uses the SHA2-256 digest algorithm instead of the deprecated SHA-1.

port of paramiko#1643

@ploxiln ploxiln force-pushed the rsa_sha2 branch 4 times, most recently from c635060 to 2e9755a Compare March 17, 2020 00:48
This change adds support for the rsa-sha2-256 and rsa-sha2-512 signature
algorithms specified in RFC 8332. New RSA key type classes are added,
with the only difference compared to RSAKey that signing and verifying
blobs uses the SHA2-256 digest algorithm instead of the deprecated SHA-1.
@ploxiln
Copy link
Owner Author

ploxiln commented Mar 28, 2020

related flags for ssh-agent: paramiko#1644

@ploxiln ploxiln added the wip work in progress label Dec 20, 2021
@bitprophet
Copy link

Added (significantly more complete) support for this to paramiko's main branch just now - fyi.

@ploxiln
Copy link
Owner Author

ploxiln commented Dec 23, 2021

Indeed, I plan to discard this branch, and copy your work (with commit author attribution) in the coming weeks, thanks @bitprophet :)

@ploxiln
Copy link
Owner Author

ploxiln commented Feb 23, 2023

this branch/pr replaced by #149

@ploxiln ploxiln closed this Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wip work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants