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

phpseclib3\Net\SSH2::keyboard_interactive_process(): Argument #1 must be of type array, string given #1836

Closed
simPod opened this issue Sep 4, 2022 · 4 comments

Comments

@simPod
Copy link
Contributor

simPod commented Sep 4, 2022

With v3.0.15 I'm getting new error:

phpseclib3\Net\SSH2::keyboard_interactive_process(): Argument #1 must be of type array, string given, called in /home/www/flop/10708/vendor/phpseclib/phpseclib/phpseclib/Net/SSH2.php on line 2396

image

@simPod simPod changed the title phpseclib3\Net\SSH2::keyboard_interactive_process(): Argument #1 must be of type array, string given phpseclib3\Net\SSH2::keyboard_interactive_process(): Argument #1 must be of type array, string given Sep 4, 2022
@frankdejonge
Copy link
Contributor

I encountered this as well, it was caught by the Flysystem test suite. I was able to reproduce this by supplying an invalid password:

$sftp = new SFTP($validHost, $validPort, 10);
var_dump($sftp->login($validUser, $notValidPassword));

In 3.0.14 the result is false, in 3.0.15 the result is an uncaught type-error.

@LordSimal
Copy link
Contributor

LordSimal commented Sep 5, 2022

Can also confirm that this error appears for login data which are not correct (under PHP 8.1)

@terrafrost
Copy link
Member

I've fixed this with 59e34b1 and made a new release (3.0.16).

Apologies!

I'll look into adding a unit test as time permits. I flew into London from Texas last night and will be flying out to Spain to do mountain climbing tomorrow.

@frankdejonge
Copy link
Contributor

Thanks for the speedy resolving @terrafrost 👍👌

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

No branches or pull requests

4 participants