~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/ssh.py

  • Committer: Ian Clatworthy
  • Date: 2007-12-11 02:07:30 UTC
  • mto: (3119.1.1 ianc-integration)
  • mto: This revision was merged to the branch mainline in revision 3120.
  • Revision ID: ian.clatworthy@internode.on.net-20071211020730-sdj4kj794dw0628e
make help topics more discoverable

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Robey Pointer <robey@lag.net>
 
1
# Copyright (C) 2005 Robey Pointer <robey@lag.net>
2
2
# Copyright (C) 2005, 2006, 2007 Canonical Ltd
3
3
#
4
4
# This program is free software; you can redistribute it and/or modify
13
13
#
14
14
# You should have received a copy of the GNU General Public License
15
15
# along with this program; if not, write to the Free Software
16
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
17
17
 
18
18
"""Foundation SSH support for SFTP and smart server."""
19
19
 
20
20
import errno
21
21
import getpass
22
 
import logging
23
22
import os
24
23
import socket
25
24
import subprocess
94
93
            try:
95
94
                vendor = self._ssh_vendors[vendor_name]
96
95
            except KeyError:
97
 
                vendor = self._get_vendor_from_path(vendor_name)
98
 
                if vendor is None:
99
 
                    raise errors.UnknownSSH(vendor_name)
100
 
                vendor.executable_path = vendor_name
 
96
                raise errors.UnknownSSH(vendor_name)
101
97
            return vendor
102
98
        return None
103
99
 
113
109
            stdout = stderr = ''
114
110
        return stdout + stderr
115
111
 
116
 
    def _get_vendor_by_version_string(self, version, progname):
 
112
    def _get_vendor_by_version_string(self, version, args):
117
113
        """Return the vendor or None based on output from the subprocess.
118
114
 
119
115
        :param version: The output of 'ssh -V' like command.
126
122
        elif 'SSH Secure Shell' in version:
127
123
            trace.mutter('ssh implementation is SSH Corp.')
128
124
            vendor = SSHCorpSubprocessVendor()
129
 
        elif 'lsh' in version:
130
 
            trace.mutter('ssh implementation is GNU lsh.')
131
 
            vendor = LSHSubprocessVendor()
132
 
        # As plink user prompts are not handled currently, don't auto-detect
133
 
        # it by inspection below, but keep this vendor detection for if a path
134
 
        # is given in BZR_SSH. See https://bugs.launchpad.net/bugs/414743
135
 
        elif 'plink' in version and progname == 'plink':
 
125
        elif 'plink' in version and args[0] == 'plink':
136
126
            # Checking if "plink" was the executed argument as Windows
137
 
            # sometimes reports 'ssh -V' incorrectly with 'plink' in its
 
127
            # sometimes reports 'ssh -V' incorrectly with 'plink' in it's
138
128
            # version.  See https://bugs.launchpad.net/bzr/+bug/107155
139
129
            trace.mutter("ssh implementation is Putty's plink.")
140
130
            vendor = PLinkSubprocessVendor()
142
132
 
143
133
    def _get_vendor_by_inspection(self):
144
134
        """Return the vendor or None by checking for known SSH implementations."""
145
 
        version = self._get_ssh_version_string(['ssh', '-V'])
146
 
        return self._get_vendor_by_version_string(version, "ssh")
147
 
 
148
 
    def _get_vendor_from_path(self, path):
149
 
        """Return the vendor or None using the program at the given path"""
150
 
        version = self._get_ssh_version_string([path, '-V'])
151
 
        return self._get_vendor_by_version_string(version, 
152
 
            os.path.splitext(os.path.basename(path))[0])
 
135
        # detection of plink vendor is disabled because of bug #107593
 
136
        # https://bugs.launchpad.net/bzr/+bug/107593
 
137
        # who want plink should explicitly enable it with BZR_SSH environment
 
138
        # variable.
 
139
        #~for args in (['ssh', '-V'], ['plink', '-V']):
 
140
        for args in (['ssh', '-V'],):
 
141
            version = self._get_ssh_version_string(args)
 
142
            vendor = self._get_vendor_by_version_string(version, args)
 
143
            if vendor is not None:
 
144
                return vendor
 
145
        return None
153
146
 
154
147
    def get_vendor(self, environment=None):
155
148
        """Find out what version of SSH is on the system.
176
169
register_ssh_vendor = _ssh_vendor_manager.register_vendor
177
170
 
178
171
 
179
 
def _ignore_signals():
 
172
def _ignore_sigint():
180
173
    # TODO: This should possibly ignore SIGHUP as well, but bzr currently
181
174
    # doesn't handle it itself.
182
175
    # <https://launchpad.net/products/bzr/+bug/41433/+index>
183
176
    import signal
184
177
    signal.signal(signal.SIGINT, signal.SIG_IGN)
185
 
    # GZ 2010-02-19: Perhaps make this check if breakin is installed instead
186
 
    if signal.getsignal(signal.SIGQUIT) != signal.SIG_DFL:
187
 
        signal.signal(signal.SIGQUIT, signal.SIG_IGN)
188
 
 
189
 
 
190
 
class SocketAsChannelAdapter(object):
 
178
 
 
179
 
 
180
class LoopbackSFTP(object):
191
181
    """Simple wrapper for a socket that pretends to be a paramiko Channel."""
192
182
 
193
183
    def __init__(self, sock):
194
184
        self.__socket = sock
195
185
 
196
 
    def get_name(self):
197
 
        return "bzr SocketAsChannelAdapter"
198
 
 
199
186
    def send(self, data):
200
187
        return self.__socket.send(data)
201
188
 
202
189
    def recv(self, n):
203
 
        try:
204
 
            return self.__socket.recv(n)
205
 
        except socket.error, e:
206
 
            if e.args[0] in (errno.EPIPE, errno.ECONNRESET, errno.ECONNABORTED,
207
 
                             errno.EBADF):
208
 
                # Connection has closed.  Paramiko expects an empty string in
209
 
                # this case, not an exception.
210
 
                return ''
211
 
            raise
 
190
        return self.__socket.recv(n)
212
191
 
213
192
    def recv_ready(self):
214
 
        # TODO: jam 20051215 this function is necessary to support the
215
 
        # pipelined() function. In reality, it probably should use
216
 
        # poll() or select() to actually return if there is data
217
 
        # available, otherwise we probably don't get any benefit
218
193
        return True
219
194
 
220
195
    def close(self):
226
201
 
227
202
    def connect_sftp(self, username, password, host, port):
228
203
        """Make an SSH connection, and return an SFTPClient.
229
 
 
 
204
        
230
205
        :param username: an ascii string
231
206
        :param password: an ascii string
232
207
        :param host: a host name as an ascii string
241
216
 
242
217
    def connect_ssh(self, username, password, host, port, command):
243
218
        """Make an SSH connection.
244
 
 
245
 
        :returns: an SSHConnection.
 
219
        
 
220
        :returns: something with a `close` method, and a `get_filelike_channels`
 
221
            method that returns a pair of (read, write) filelike objects.
246
222
        """
247
223
        raise NotImplementedError(self.connect_ssh)
248
224
 
266
242
            sock.connect((host, port))
267
243
        except socket.error, e:
268
244
            self._raise_connection_error(host, port=port, orig_error=e)
269
 
        return SFTPClient(SocketAsChannelAdapter(sock))
 
245
        return SFTPClient(LoopbackSFTP(sock))
270
246
 
271
247
register_ssh_vendor('loopback', LoopbackVendor())
272
248
 
273
249
 
 
250
class _ParamikoSSHConnection(object):
 
251
    def __init__(self, channel):
 
252
        self.channel = channel
 
253
 
 
254
    def get_filelike_channels(self):
 
255
        return self.channel.makefile('rb'), self.channel.makefile('wb')
 
256
 
 
257
    def close(self):
 
258
        return self.channel.close()
 
259
 
 
260
 
274
261
class ParamikoVendor(SSHVendor):
275
262
    """Vendor that uses paramiko."""
276
263
 
339
326
            self._raise_connection_error(host, port=port, orig_error=e,
340
327
                                         msg='Unable to invoke remote bzr')
341
328
 
342
 
_ssh_connection_errors = (EOFError, OSError, IOError, socket.error)
343
329
if paramiko is not None:
344
330
    vendor = ParamikoVendor()
345
331
    register_ssh_vendor('paramiko', vendor)
346
332
    register_ssh_vendor('none', vendor)
347
333
    register_default_ssh_vendor(vendor)
348
 
    _ssh_connection_errors += (paramiko.SSHException,)
 
334
    _sftp_connection_errors = (EOFError, paramiko.SSHException)
349
335
    del vendor
 
336
else:
 
337
    _sftp_connection_errors = (EOFError,)
350
338
 
351
339
 
352
340
class SubprocessVendor(SSHVendor):
353
341
    """Abstract base class for vendors that use pipes to a subprocess."""
354
342
 
355
343
    def _connect(self, argv):
356
 
        # Attempt to make a socketpair to use as stdin/stdout for the SSH
357
 
        # subprocess.  We prefer sockets to pipes because they support
358
 
        # non-blocking short reads, allowing us to optimistically read 64k (or
359
 
        # whatever) chunks.
360
 
        try:
361
 
            my_sock, subproc_sock = socket.socketpair()
362
 
            osutils.set_fd_cloexec(my_sock)
363
 
        except (AttributeError, socket.error):
364
 
            # This platform doesn't support socketpair(), so just use ordinary
365
 
            # pipes instead.
366
 
            stdin = stdout = subprocess.PIPE
367
 
            my_sock, subproc_sock = None, None
368
 
        else:
369
 
            stdin = stdout = subproc_sock
370
 
        proc = subprocess.Popen(argv, stdin=stdin, stdout=stdout,
 
344
        proc = subprocess.Popen(argv,
 
345
                                stdin=subprocess.PIPE,
 
346
                                stdout=subprocess.PIPE,
371
347
                                **os_specific_subprocess_params())
372
 
        if subproc_sock is not None:
373
 
            subproc_sock.close()
374
 
        return SSHSubprocessConnection(proc, sock=my_sock)
 
348
        return SSHSubprocess(proc)
375
349
 
376
350
    def connect_sftp(self, username, password, host, port):
377
351
        try:
378
352
            argv = self._get_vendor_specific_argv(username, host, port,
379
353
                                                  subsystem='sftp')
380
354
            sock = self._connect(argv)
381
 
            return SFTPClient(SocketAsChannelAdapter(sock))
382
 
        except _ssh_connection_errors, e:
 
355
            return SFTPClient(sock)
 
356
        except _sftp_connection_errors, e:
 
357
            self._raise_connection_error(host, port=port, orig_error=e)
 
358
        except (OSError, IOError), e:
 
359
            # If the machine is fast enough, ssh can actually exit
 
360
            # before we try and send it the sftp request, which
 
361
            # raises a Broken Pipe
 
362
            if e.errno not in (errno.EPIPE,):
 
363
                raise
383
364
            self._raise_connection_error(host, port=port, orig_error=e)
384
365
 
385
366
    def connect_ssh(self, username, password, host, port, command):
387
368
            argv = self._get_vendor_specific_argv(username, host, port,
388
369
                                                  command=command)
389
370
            return self._connect(argv)
390
 
        except _ssh_connection_errors, e:
 
371
        except (EOFError), e:
 
372
            self._raise_connection_error(host, port=port, orig_error=e)
 
373
        except (OSError, IOError), e:
 
374
            # If the machine is fast enough, ssh can actually exit
 
375
            # before we try and send it the sftp request, which
 
376
            # raises a Broken Pipe
 
377
            if e.errno not in (errno.EPIPE,):
 
378
                raise
391
379
            self._raise_connection_error(host, port=port, orig_error=e)
392
380
 
393
381
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
394
382
                                  command=None):
395
383
        """Returns the argument list to run the subprocess with.
396
 
 
 
384
        
397
385
        Exactly one of 'subsystem' and 'command' must be specified.
398
386
        """
399
387
        raise NotImplementedError(self._get_vendor_specific_argv)
402
390
class OpenSSHSubprocessVendor(SubprocessVendor):
403
391
    """SSH vendor that uses the 'ssh' executable from OpenSSH."""
404
392
 
405
 
    executable_path = 'ssh'
406
 
 
407
393
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
408
394
                                  command=None):
409
 
        args = [self.executable_path,
 
395
        assert subsystem is not None or command is not None, (
 
396
            'Must specify a command or subsystem')
 
397
        if subsystem is not None:
 
398
            assert command is None, (
 
399
                'subsystem and command are mutually exclusive')
 
400
        args = ['ssh',
410
401
                '-oForwardX11=no', '-oForwardAgent=no',
411
 
                '-oClearAllForwardings=yes',
 
402
                '-oClearAllForwardings=yes', '-oProtocol=2',
412
403
                '-oNoHostAuthenticationForLocalhost=yes']
413
404
        if port is not None:
414
405
            args.extend(['-p', str(port)])
426
417
class SSHCorpSubprocessVendor(SubprocessVendor):
427
418
    """SSH vendor that uses the 'ssh' executable from SSH Corporation."""
428
419
 
429
 
    executable_path = 'ssh'
430
 
 
431
420
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
432
421
                                  command=None):
433
 
        args = [self.executable_path, '-x']
 
422
        assert subsystem is not None or command is not None, (
 
423
            'Must specify a command or subsystem')
 
424
        if subsystem is not None:
 
425
            assert command is None, (
 
426
                'subsystem and command are mutually exclusive')
 
427
        args = ['ssh', '-x']
434
428
        if port is not None:
435
429
            args.extend(['-p', str(port)])
436
430
        if username is not None:
441
435
            args.extend([host] + command)
442
436
        return args
443
437
 
444
 
register_ssh_vendor('sshcorp', SSHCorpSubprocessVendor())
445
 
 
446
 
 
447
 
class LSHSubprocessVendor(SubprocessVendor):
448
 
    """SSH vendor that uses the 'lsh' executable from GNU"""
449
 
 
450
 
    executable_path = 'lsh'
451
 
 
452
 
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
453
 
                                  command=None):
454
 
        args = [self.executable_path]
455
 
        if port is not None:
456
 
            args.extend(['-p', str(port)])
457
 
        if username is not None:
458
 
            args.extend(['-l', username])
459
 
        if subsystem is not None:
460
 
            args.extend(['--subsystem', subsystem, host])
461
 
        else:
462
 
            args.extend([host] + command)
463
 
        return args
464
 
 
465
 
register_ssh_vendor('lsh', LSHSubprocessVendor())
 
438
register_ssh_vendor('ssh', SSHCorpSubprocessVendor())
466
439
 
467
440
 
468
441
class PLinkSubprocessVendor(SubprocessVendor):
469
442
    """SSH vendor that uses the 'plink' executable from Putty."""
470
443
 
471
 
    executable_path = 'plink'
472
 
 
473
444
    def _get_vendor_specific_argv(self, username, host, port, subsystem=None,
474
445
                                  command=None):
475
 
        args = [self.executable_path, '-x', '-a', '-ssh', '-2', '-batch']
 
446
        assert subsystem is not None or command is not None, (
 
447
            'Must specify a command or subsystem')
 
448
        if subsystem is not None:
 
449
            assert command is None, (
 
450
                'subsystem and command are mutually exclusive')
 
451
        args = ['plink', '-x', '-a', '-ssh', '-2']
476
452
        if port is not None:
477
453
            args.extend(['-P', str(port)])
478
454
        if username is not None:
487
463
 
488
464
 
489
465
def _paramiko_auth(username, password, host, port, paramiko_transport):
 
466
    # paramiko requires a username, but it might be none if nothing was supplied
 
467
    # use the local username, just in case.
 
468
    # We don't override username, because if we aren't using paramiko,
 
469
    # the username might be specified in ~/.ssh/config and we don't want to
 
470
    # force it to something else
 
471
    # Also, it would mess up the self.relpath() functionality
490
472
    auth = config.AuthenticationConfig()
491
 
    # paramiko requires a username, but it might be none if nothing was
492
 
    # supplied.  If so, use the local username.
493
473
    if username is None:
494
 
        username = auth.get_user('ssh', host, port=port,
495
 
                                 default=getpass.getuser())
 
474
        username = auth.get_user('ssh', host, port=port)
 
475
        if username is None:
 
476
            # Default to local user
 
477
            username = getpass.getuser()
 
478
 
496
479
    if _use_ssh_agent:
497
480
        agent = paramiko.Agent()
498
481
        for key in agent.get_keys():
510
493
    if _try_pkey_auth(paramiko_transport, paramiko.DSSKey, username, 'id_dsa'):
511
494
        return
512
495
 
513
 
    # If we have gotten this far, we are about to try for passwords, do an
514
 
    # auth_none check to see if it is even supported.
515
 
    supported_auth_types = []
516
 
    try:
517
 
        # Note that with paramiko <1.7.5 this logs an INFO message:
518
 
        #    Authentication type (none) not permitted.
519
 
        # So we explicitly disable the logging level for this action
520
 
        old_level = paramiko_transport.logger.level
521
 
        paramiko_transport.logger.setLevel(logging.WARNING)
522
 
        try:
523
 
            paramiko_transport.auth_none(username)
524
 
        finally:
525
 
            paramiko_transport.logger.setLevel(old_level)
526
 
    except paramiko.BadAuthenticationType, e:
527
 
        # Supported methods are in the exception
528
 
        supported_auth_types = e.allowed_types
529
 
    except paramiko.SSHException, e:
530
 
        # Don't know what happened, but just ignore it
531
 
        pass
532
 
    # We treat 'keyboard-interactive' and 'password' auth methods identically,
533
 
    # because Paramiko's auth_password method will automatically try
534
 
    # 'keyboard-interactive' auth (using the password as the response) if
535
 
    # 'password' auth is not available.  Apparently some Debian and Gentoo
536
 
    # OpenSSH servers require this.
537
 
    # XXX: It's possible for a server to require keyboard-interactive auth that
538
 
    # requires something other than a single password, but we currently don't
539
 
    # support that.
540
 
    if ('password' not in supported_auth_types and
541
 
        'keyboard-interactive' not in supported_auth_types):
542
 
        raise errors.ConnectionError('Unable to authenticate to SSH host as'
543
 
            '\n  %s@%s\nsupported auth types: %s'
544
 
            % (username, host, supported_auth_types))
545
 
 
546
496
    if password:
547
497
        try:
548
498
            paramiko_transport.auth_password(username, password)
552
502
 
553
503
    # give up and ask for a password
554
504
    password = auth.get_password('ssh', host, username, port=port)
555
 
    # get_password can still return None, which means we should not prompt
556
 
    if password is not None:
557
 
        try:
558
 
            paramiko_transport.auth_password(username, password)
559
 
        except paramiko.SSHException, e:
560
 
            raise errors.ConnectionError(
561
 
                'Unable to authenticate to SSH host as'
562
 
                '\n  %s@%s\n' % (username, host), e)
563
 
    else:
564
 
        raise errors.ConnectionError('Unable to authenticate to SSH host as'
565
 
                                     '  %s@%s' % (username, host))
 
505
    try:
 
506
        paramiko_transport.auth_password(username, password)
 
507
    except paramiko.SSHException, e:
 
508
        raise errors.ConnectionError(
 
509
            'Unable to authenticate to SSH host as %s@%s' % (username, host), e)
566
510
 
567
511
 
568
512
def _try_pkey_auth(paramiko_transport, pkey_class, username, filename):
630
574
def os_specific_subprocess_params():
631
575
    """Get O/S specific subprocess parameters."""
632
576
    if sys.platform == 'win32':
633
 
        # setting the process group and closing fds is not supported on
 
577
        # setting the process group and closing fds is not supported on 
634
578
        # win32
635
579
        return {}
636
580
    else:
637
 
        # We close fds other than the pipes as the child process does not need
 
581
        # We close fds other than the pipes as the child process does not need 
638
582
        # them to be open.
639
583
        #
640
584
        # We also set the child process to ignore SIGINT.  Normally the signal
642
586
        # this causes it to be seen only by bzr and not by ssh.  Python will
643
587
        # generate a KeyboardInterrupt in bzr, and we will then have a chance
644
588
        # to release locks or do other cleanup over ssh before the connection
645
 
        # goes away.
 
589
        # goes away.  
646
590
        # <https://launchpad.net/products/bzr/+bug/5987>
647
591
        #
648
592
        # Running it in a separate process group is not good because then it
649
593
        # can't get non-echoed input of a password or passphrase.
650
594
        # <https://launchpad.net/products/bzr/+bug/40508>
651
 
        return {'preexec_fn': _ignore_signals,
 
595
        return {'preexec_fn': _ignore_sigint,
652
596
                'close_fds': True,
653
597
                }
654
598
 
655
 
import weakref
656
 
_subproc_weakrefs = set()
657
 
 
658
 
def _close_ssh_proc(proc, sock):
659
 
    """Carefully close stdin/stdout and reap the SSH process.
660
 
 
661
 
    If the pipes are already closed and/or the process has already been
662
 
    wait()ed on, that's ok, and no error is raised.  The goal is to do our best
663
 
    to clean up (whether or not a clean up was already tried).
664
 
    """
665
 
    funcs = []
666
 
    for closeable in (proc.stdin, proc.stdout, sock):
667
 
        # We expect that either proc (a subprocess.Popen) will have stdin and
668
 
        # stdout streams to close, or that we will have been passed a socket to
669
 
        # close, with the option not in use being None.
670
 
        if closeable is not None:
671
 
            funcs.append(closeable.close)
672
 
    funcs.append(proc.wait)
673
 
    for func in funcs:
674
 
        try:
675
 
            func()
676
 
        except OSError:
677
 
            # It's ok for the pipe to already be closed, or the process to
678
 
            # already be finished.
679
 
            continue
680
 
 
681
 
 
682
 
class SSHConnection(object):
683
 
    """Abstract base class for SSH connections."""
684
 
 
685
 
    def get_sock_or_pipes(self):
686
 
        """Returns a (kind, io_object) pair.
687
 
 
688
 
        If kind == 'socket', then io_object is a socket.
689
 
 
690
 
        If kind == 'pipes', then io_object is a pair of file-like objects
691
 
        (read_from, write_to).
692
 
        """
693
 
        raise NotImplementedError(self.get_sock_or_pipes)
694
 
 
695
 
    def close(self):
696
 
        raise NotImplementedError(self.close)
697
 
 
698
 
 
699
 
class SSHSubprocessConnection(SSHConnection):
700
 
    """A connection to an ssh subprocess via pipes or a socket.
701
 
 
702
 
    This class is also socket-like enough to be used with
703
 
    SocketAsChannelAdapter (it has 'send' and 'recv' methods).
704
 
    """
705
 
 
706
 
    def __init__(self, proc, sock=None):
707
 
        """Constructor.
708
 
 
709
 
        :param proc: a subprocess.Popen
710
 
        :param sock: if proc.stdin/out is a socket from a socketpair, then sock
711
 
            should bzrlib's half of that socketpair.  If not passed, proc's
712
 
            stdin/out is assumed to be ordinary pipes.
713
 
        """
 
599
 
 
600
class SSHSubprocess(object):
 
601
    """A socket-like object that talks to an ssh subprocess via pipes."""
 
602
 
 
603
    def __init__(self, proc):
714
604
        self.proc = proc
715
 
        self._sock = sock
716
 
        # Add a weakref to proc that will attempt to do the same as self.close
717
 
        # to avoid leaving processes lingering indefinitely.
718
 
        def terminate(ref):
719
 
            _subproc_weakrefs.remove(ref)
720
 
            _close_ssh_proc(proc, sock)
721
 
        _subproc_weakrefs.add(weakref.ref(self, terminate))
722
605
 
723
606
    def send(self, data):
724
 
        if self._sock is not None:
725
 
            return self._sock.send(data)
726
 
        else:
727
 
            return os.write(self.proc.stdin.fileno(), data)
 
607
        return os.write(self.proc.stdin.fileno(), data)
 
608
 
 
609
    def recv_ready(self):
 
610
        # TODO: jam 20051215 this function is necessary to support the
 
611
        # pipelined() function. In reality, it probably should use
 
612
        # poll() or select() to actually return if there is data
 
613
        # available, otherwise we probably don't get any benefit
 
614
        return True
728
615
 
729
616
    def recv(self, count):
730
 
        if self._sock is not None:
731
 
            return self._sock.recv(count)
732
 
        else:
733
 
            return os.read(self.proc.stdout.fileno(), count)
734
 
 
735
 
    def close(self):
736
 
        _close_ssh_proc(self.proc, self._sock)
737
 
 
738
 
    def get_sock_or_pipes(self):
739
 
        if self._sock is not None:
740
 
            return 'socket', self._sock
741
 
        else:
742
 
            return 'pipes', (self.proc.stdout, self.proc.stdin)
743
 
 
744
 
 
745
 
class _ParamikoSSHConnection(SSHConnection):
746
 
    """An SSH connection via paramiko."""
747
 
 
748
 
    def __init__(self, channel):
749
 
        self.channel = channel
750
 
 
751
 
    def get_sock_or_pipes(self):
752
 
        return ('socket', self.channel)
753
 
 
754
 
    def close(self):
755
 
        return self.channel.close()
756
 
 
 
617
        return os.read(self.proc.stdout.fileno(), count)
 
618
 
 
619
    def close(self):
 
620
        self.proc.stdin.close()
 
621
        self.proc.stdout.close()
 
622
        self.proc.wait()
 
623
 
 
624
    def get_filelike_channels(self):
 
625
        return (self.proc.stdout, self.proc.stdin)
757
626