cryptography icon indicating copy to clipboard operation
cryptography copied to clipboard

pass or collect comments for ssh private keys

Open gzm55 opened this issue 2 years ago • 4 comments

close #8980

gzm55 avatar Jun 03 '23 12:06 gzm55

@alex could you plz review this pr?

gzm55 avatar Jun 09 '23 16:06 gzm55

It's on my TODO list for this weekend.

On Fri, Jun 9, 2023 at 12:53 PM James Z.M. Gao @.***> wrote:

@alex https://github.com/alex could you plz review this pr?

— Reply to this email directly, view it on GitHub https://github.com/pyca/cryptography/pull/9019#issuecomment-1584884169, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAAGBFDMIQJMTC3WV5KRYTXKNIJ3ANCNFSM6AAAAAAYZH4S6U . You are receiving this because you were mentioned.Message ID: @.***>

-- All that is necessary for evil to succeed is for good people to do nothing.

alex avatar Jun 09 '23 17:06 alex

IMO, this signature is too foreign to the rest of our API surface. Another (possibly bad) idea: should we have a separate loader load_ssh_key_with_comment that is documented as returning comment bytes (e.g. Tuple[PrivateKeyType, bytes])?

reaperhulk avatar Jun 17 '23 12:06 reaperhulk

@reaperhulk another api is ok, but most codes of the new api have to be copied directly from the original one

gzm55 avatar Jun 18 '23 05:06 gzm55

No activity here in over a year. The approach needs to be rethought, but we're willing to take a PR for comment parsing still. We recommend discussing the API on the issue first.

reaperhulk avatar Jul 20 '24 15:07 reaperhulk