Skip to content

ssh-key: remove RSA MIN_KEY_SIZE#412

Merged
tarcieri merged 1 commit intoRustCrypto:masterfrom
WesleyAC:remove-min-key-size
Nov 8, 2025
Merged

ssh-key: remove RSA MIN_KEY_SIZE#412
tarcieri merged 1 commit intoRustCrypto:masterfrom
WesleyAC:remove-min-key-size

Conversation

@WesleyAC
Copy link
Contributor

@WesleyAC WesleyAC commented Oct 6, 2025

This will be implemented in the rsa crate instead of here.

See: #336

@WesleyAC WesleyAC force-pushed the remove-min-key-size branch 2 times, most recently from 2a73ff0 to 7580ba5 Compare October 6, 2025 20:28
This will be implemented in the `rsa` crate instead of here.

See: RustCrypto#336
@WesleyAC WesleyAC force-pushed the remove-min-key-size branch from 7580ba5 to eefa2fe Compare October 6, 2025 20:29
@tarcieri tarcieri changed the title Remove RSA MIN_KEY_SIZE. ssh-key: remove RSA MIN_KEY_SIZE Nov 8, 2025
@tarcieri tarcieri merged commit 7410bf7 into RustCrypto:master Nov 8, 2025
14 checks passed
zydou pushed a commit to zydou/arti that referenced this pull request Mar 16, 2026
Upstream `ssh-key` is missing some important features we need for
arti-relay:

  * a bug fix without which we can't convert deserialized RSA keys to
    their rsa counterparts: RustCrypto/SSH#318
  * @WesleyAC 's patch RustCrypto/SSH#412 for
    allowing insecure (1024 bits long) RSA keys (needed because the
    relay KS_relayid_rsa identity keys are 1024 bits long)

We plan to switch back to mainline `ssh-key` when `ssh-key 0.7.0` comes
out.

See the discussion in #2398 for more details.
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