Page MenuHomePhabricator

Passphrase can't distinguish between correct, mangled, and passphrase-encoded SSH keys
Closed, ResolvedPublic

Description

See PHI135. In that case, a user uploaded an SSH private key to passphrase which was missing a header and footer. This (a) didn't work and (b) appeared to be a passphrase key.

We could do a better job of trying to detect invalid private keys (missing header, passphrase-encrypted).

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley added a revision: Restricted Differential Revision.Nov 11 2019, 7:19 PM
epriestley added a revision: Restricted Differential Revision.Nov 13 2019, 6:17 PM
epriestley claimed this task.

I've marked D20905 as resolving this. This isn't really "resolved" completely, but T13454 has a better description of what the problems are and why they're difficult. Our behavior is, at least, substantially better than it was before.

epriestley added a commit: Restricted Diffusion Commit.Nov 13 2019, 6:22 PM
epriestley added a revision: Restricted Differential Revision.May 13 2020, 3:05 PM
epriestley added a commit: Restricted Diffusion Commit.May 13 2020, 3:14 PM