-
Notifications
You must be signed in to change notification settings - Fork 32
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
ssh-key: Add Certificate::decode_as
and expose KeyData::decode_as
#233
Conversation
Signed-off-by: Wiktor Kwapisiewicz <wiktor@metacode.biz>
This additional function is needed for SSH Agent Protocol where, based on the algorithm, we need to parse the `Certificate` or the `KeyData`. Without `decode_as` the `decode` function will greedily consume additional string from the reader. See: wiktor-k/ssh-agent-lib#83 Signed-off-by: Wiktor Kwapisiewicz <wiktor@metacode.biz>
See: RustCrypto/SSH#233 Signed-off-by: Wiktor Kwapisiewicz <wiktor@metacode.biz>
Hi @tarcieri, sorry to disturb but I'm wondering if you're planning on releasing Thanks for your time! |
Sure, I can cut some prereleases, although I'll need to cut prereleases of the dependencies first (which is something I planned on doing soon anyway) |
That's great to hear. Thanks! 🙇♂️ |
If you don't mind I'll ping you every now and then @tarcieri 👋 |
It might be awhile. There are dozens upon dozens of downstream dependencies to do prereleases of. |
Release PR here: #250 I had to temporarily remove the |
Thank you 🙏 |
This additional function is needed for SSH Agent Protocol where, based on
the algorithm, we need to parse the
Certificate
or theKeyData
.Without
decode_as
thedecode
function will greedily consume additionalstring from the reader.
This is similar to #211.
See: wiktor-k/ssh-agent-lib#83