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

Fix deserialization of unknown extensions #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wiktor-k
Copy link

OpenSSH 8.9 ssh client started sending EXTENSION commands. Unfortunately ssh-agent.rs didn't deserialize them correctly and even though these extensions were optional the agent didn't work.

This patch adjusts the deserialization of extension contents to properly capture all bytes and includes a test code observed in the wild.

Fixes #30.

OpenSSH 8.9 ssh client started sending EXTENSION commands. Unfortunately
ssh-agent.rs didn't deserialize them correctly and even though these
extensions were optional the agent didn't work.

This patch adjusts the deserialization of extension contents to properly
capture all bytes and includes a test code observed in the wild.

Fixes #30.
@d-e-s-o
Copy link
Contributor

d-e-s-o commented May 24, 2022

Thanks for working on a fix! I hope this pull request gets reviewed and merged soon, so that we can unblock people from going to OpenSSH 8.9.

@wiktor-k
Copy link
Author

TBH I wouldn't count on that. I've contacted the maintainers (getting e-mails from git commits) but haven't heard back. Since I needed this and a couple of other features I forked the repo and merged outstanding PRs there.

Just in case you're critically dependent on this here's the URL: https://github.com/wiktor-k/ssh-agent-lib

@d-e-s-o
Copy link
Contributor

d-e-s-o commented May 26, 2022

Yeah, I've found that fork after my comment. Will migrate over. Thanks for taking over!

Honestly, the least those guys could do is add a not-maintained badge to the repository and mark it as such in Cargo.toml.

@fmpomar
Copy link
Contributor

fmpomar commented May 26, 2022

Done. I just did exactly what you suggested. Please take into account that this project is not for profit and we have other concerns to address in our lives. Our intention is to eventually continue maintaining it, but currently it's impossible for us.
I hope you understand.

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.

SSH Agent doesn't work with OpenSSH 8.9
3 participants