Unable to get a connection, marshal dump needs a file as argument

Bug #808117 reported by Franck BRET
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Pushy
Fix Released
Critical
Andrew Wilkins

Bug Description

Evaluating pushy, i can't establish a connection to remote host. I've tried with pushy 0.5 and trunk version.

[code]
In [1]: import pushy

In [2]: con = pushy.connect("ssh:192.168.0.13", username="user", password="*****")
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/pushy/client.py", line 361, in __init__
    marshal.dump(packages, self.server.stdin, 1)
TypeError: marshal.dump() 2nd arg must be file
[/code]

Local host : Ubuntu
Python 2.7.1+ (r271:86832, Apr 11 2011, 18:13:53)
[GCC 4.5.2] on linux2

Remote host : Debian
Python 2.6.6 (r266:84292, Dec 27 2010, 00:02:40)
[GCC 4.4.5] on linux2

Am I doing something wrong ? How can I debug this ?

Thanks in advance, hope I can make it run has this module seems very interesting for me coupled with fabric.

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Franck, thank you for the bug report. At first glance it certainly looks like a bug, and not you doing something wrong. I am investigating now, and will update the defect later to let you know.

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Indeed it is a bug. It only occurs when using the Paramiko SSH library, since its "stdin" object is not a real file. This escaped testing since all my testing invoked only the native SSH library. I will rectify this in future testing.

I'll have a fix in the trunk shortly.

Changed in pushy:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → Andrew Wilkins (axwalk)
status: Confirmed → In Progress
Revision history for this message
Andrew Wilkins (axwalk) wrote :

Okay, I have committed a fix, and it's in the trunk. When you get a chance, please give it a go and see if it fixes the problem for you. I'll follow up with a bug fix release.
Thanks.

Changed in pushy:
status: In Progress → Fix Committed
Revision history for this message
Franck BRET (franckbret) wrote :

Hi Andrew,
Just tested it with last trunk revision and it works fine for me, no more errors.
Maybe you should fix 0.5 on pypi.
Don't hesitate to ask me if you need more testing.
Thanks a lot, I will now try it in depth ;-)

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Great, thanks Franck. I'll hopefully upload a new version to pypi and launchpad this evening. I hope you get some use out of Pushy.

Andrew Wilkins (axwalk)
Changed in pushy:
milestone: none → 0.5.1
Andrew Wilkins (axwalk)
Changed in pushy:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.