SFTP Get - Error

[enriquemanuel]

When doing an SFTP GET the variable status is not defined.
in vendor/sinner/phpseclib-bundle/lib/Sinner/Phpseclib/Net/SFTP.php at line 1545

        // check the status from the NET_SFTP_STATUS case in the above switch after the file has been closed
        if ($status != NET_SFTP_STATUS_OK) {
            return false;
        }

After looking at the function, the $status is never defined.

I would assume that you want to see if the file has been closed or completed right?

If you remove this line, you are omitting the check from the function, but actually allows to pass it correctly after doing the sftp->get()

This would work correctly removing the status check, but I'm working on getting it fixed.

        $serverip='serverip';
        $pathtofile="/path/to/file/file.txt";

        //loading keyfile path
        $pathkey = $this->get('kernel')->locateResource("@EnriqueManuelOpsmartBundle/Resources/config/id_rsa_key");

        //connecting
        $sftp = new Net_SFTP($serverip);
        $key = new Crypt_RSA();
        $key->loadKey(file_get_contents($pathkey));
        $sftp->login('authorizeduser',$key);

        // sftp get to download the file
        $sftp->get($pathtofile,'localfile.txt');
        //disconnect
        $sftp->_disconnect(0);

[enriquemanuel]

This was an easy fix, you just need to do the same check that you are doing for all the other functions.
Add the following line

extract(unpack('Nstatus', $this->_string_shift($response, 4)));

before the

if ($status != NET_SFTP_STATUS_OK) {

Let me know if you want me to do a pull or you just want to change it.

[ashurai]

Still the status is here no where defined inside get function, i can fix this issue for one time but while any client or any one is taking composer update it will remove the local changes. So fix this problem or If you want then i can pull for this fix .

if ($status != NET_SFTP_STATUS_OK) {

Do the fixes as possible.