[merge] handle a SSHException

[Stefan (metze) Metzmacher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

John Arbash Meinel schrieb:
> Stefan (metze) Metzmacher wrote:
>> John Arbash Meinel schrieb:
>>>> I like the patch, but I think I've seen more of 'EOFError' than
>>>> SSHException. It might be because I'm running paramiko 1.5.2 instead of
>>>> 1.6.1.
>>>>
>>>> Actually, in my testing with 1.5.2 I get EOFError, and with paramiko
>>>> 1.6.1-dev (revno 371) I get the exception:
>>>>
>>>>   File "build/bdist.linux-i686/egg/paramiko/sftp_client.py", line 86, in
>>>> __init__
>>>> NameError: global name 'SSHException' is not defined
>> I've send that to the paramiko mailing list, but it didn't get through...
> 
> Robey, can you comment/at least patch this for 1.6.2. Do you have a
> bug-tracker for stuff like this?
> 
>> I've attached it to this mail to
>>
>>>> So I'd like to modify the patch to check for at least EOFError and
>>>> SSHException. I don't think we want to catch NameError.
>> ok, a new patch is attached.
>>
>> metze
> 
> +1 from me. I'd like to see this in 0.9, so I'll put it in my pending
> queue. And if we don't get any negative feedback I'll merge it in a day
> or so. (If we get positive, I'll merge it sooner).

ok thanks!

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)
Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org

iD8DBQFEyPXHm70gjA5TCD8RAuA3AJ42zHFtCQxdfS4YrjvdVi/fSSstHQCgyOmH
arM1I/eHezxiBJQ6Ca7f0yM=
=bosS
-----END PGP SIGNATURE-----