Nice find!

Altho what'd be better is if you did something like $orig_algorithms = $this->supported_private_key_algorithms before the for loop and then did $this->supported_private_key_algorithms = $orig_algorithms after each $this->privatekey_login($username, $key) call (ie. same place you have your current code changes).

The reason this would be better than what you're currently doing is that $this->server_host_key_algorithms doesn't take NET_SSH2_MSG_EXT_INFO (RFC8308) into consideration.

I could make this change myself but I'll hold off for a few days in case you want to do it? Like if I do it your name won't be in the commit history. idk if that matters to you or not lol.

That said, if you want to make the code change vs me, don't use tabs to indent. In your change the line you added is preceded by one tab and then eight spaces. The tab should be replaced by four spaces (bringing the total number of spaces to twelve).

Thanks!

Hi, thank you for a quick response:

Altho what'd be better is if you did something like $orig_algorithms = $this->supported_private_key_algorithms before the for loop and then did $this->supported_private_key_algorithms = $orig_algorithms after each $this->privatekey_login($username, $key) call (ie. same place you have your current code changes).

The reason this would be better than what you're currently doing is that $this->server_host_key_algorithms doesn't take NET_SSH2_MSG_EXT_INFO (RFC8308) into consideration.

I could make this change myself but I'll hold off for a few days in case you want to do it? Like if I do it your name won't be in the commit history. idk if that matters to you or not lol.

I updated the patch according to your advice. I'm totally OK with not having my name in the commit history, so if this update is still not perfect, feel free to fix this on your own :-)

That said, if you want to make the code change vs me, don't use tabs to indent. In your change the line you added is preceded by one tab and then eight spaces. The tab should be replaced by four spaces (bringing the total number of spaces to twelve).

Sorry, I did a quick commit right in the GitHub web interface, now it's OK.

Looks good! I'll try to merge this this evening. More specifically, I'll cherry pick this to the 1.0 branch and then merge into the 2.0 branch, the 3.0 branch and then the master branch.

See 3b0fb1c

Thanks!