~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/sftp.py

  • Committer: Robert Collins
  • Date: 2009-07-07 04:32:13 UTC
  • mto: This revision was merged to the branch mainline in revision 4524.
  • Revision ID: robertc@robertcollins.net-20090707043213-4hjjhgr40iq7gk2d
More informative assertions in xml serialisation.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005 Robey Pointer <robey@lag.net>
2
 
# Copyright (C) 2005, 2006, 2007 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2007, 2008, 2009 Canonical Ltd
3
2
#
4
3
# This program is free software; you can redistribute it and/or modify
5
4
# it under the terms of the GNU General Public License as published by
13
12
#
14
13
# You should have received a copy of the GNU General Public License
15
14
# along with this program; if not, write to the Free Software
16
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
17
16
 
18
17
"""Implementation of Transport over SFTP, using paramiko."""
19
18
 
24
23
# suite.  Those formats all date back to 0.7; so we should be able to remove
25
24
# these methods when we officially drop support for those formats.
26
25
 
 
26
import bisect
27
27
import errno
 
28
import itertools
28
29
import os
29
30
import random
30
31
import select
34
35
import time
35
36
import urllib
36
37
import urlparse
37
 
import weakref
 
38
import warnings
38
39
 
39
40
from bzrlib import (
 
41
    config,
 
42
    debug,
40
43
    errors,
41
44
    urlutils,
42
45
    )
48
51
                           ParamikoNotPresent,
49
52
                           )
50
53
from bzrlib.osutils import pathjoin, fancy_rename, getcwd
 
54
from bzrlib.symbol_versioning import (
 
55
        deprecated_function,
 
56
        )
51
57
from bzrlib.trace import mutter, warning
52
58
from bzrlib.transport import (
 
59
    FileFileStream,
 
60
    _file_streams,
53
61
    local,
54
 
    register_urlparse_netloc_protocol,
55
62
    Server,
56
 
    split_url,
57
63
    ssh,
58
 
    Transport,
 
64
    ConnectedTransport,
59
65
    )
60
66
 
 
67
# Disable one particular warning that comes from paramiko in Python2.5; if
 
68
# this is emitted at the wrong time it tends to cause spurious test failures
 
69
# or at least noise in the test case::
 
70
#
 
71
# [1770/7639 in 86s, 1 known failures, 50 skipped, 2 missing features]
 
72
# test_permissions.TestSftpPermissions.test_new_files
 
73
# /var/lib/python-support/python2.5/paramiko/message.py:226: DeprecationWarning: integer argument expected, got float
 
74
#  self.packet.write(struct.pack('>I', n))
 
75
warnings.filterwarnings('ignore',
 
76
        'integer argument expected, got float',
 
77
        category=DeprecationWarning,
 
78
        module='paramiko.message')
 
79
 
61
80
try:
62
81
    import paramiko
63
82
except ImportError, e:
70
89
    from paramiko.sftp_file import SFTPFile
71
90
 
72
91
 
73
 
register_urlparse_netloc_protocol('sftp')
74
 
 
75
 
 
76
 
# This is a weakref dictionary, so that we can reuse connections
77
 
# that are still active. Long term, it might be nice to have some
78
 
# sort of expiration policy, such as disconnect if inactive for
79
 
# X seconds. But that requires a lot more fanciness.
80
 
_connected_hosts = weakref.WeakValueDictionary()
81
 
 
82
 
 
83
92
_paramiko_version = getattr(paramiko, '__version_info__', (0, 0, 0))
84
93
# don't use prefetch unless paramiko version >= 1.5.5 (there were bugs earlier)
85
94
_default_do_prefetch = (_paramiko_version >= (1, 5, 5))
86
95
 
87
96
 
88
 
def clear_connection_cache():
89
 
    """Remove all hosts from the SFTP connection cache.
90
 
 
91
 
    Primarily useful for test cases wanting to force garbage collection.
92
 
    """
93
 
    _connected_hosts.clear()
94
 
 
95
 
 
96
97
class SFTPLock(object):
97
98
    """This fakes a lock in a remote location.
98
 
    
 
99
 
99
100
    A present lock is indicated just by the existence of a file.  This
100
 
    doesn't work well on all transports and they are only used in 
 
101
    doesn't work well on all transports and they are only used in
101
102
    deprecated storage formats.
102
103
    """
103
 
    
 
104
 
104
105
    __slots__ = ['path', 'lock_path', 'lock_file', 'transport']
105
106
 
106
107
    def __init__(self, path, transport):
107
 
        assert isinstance(transport, SFTPTransport)
108
 
 
109
108
        self.lock_file = None
110
109
        self.path = path
111
110
        self.lock_path = path + '.write-lock'
135
134
            pass
136
135
 
137
136
 
138
 
class SFTPUrlHandling(Transport):
139
 
    """Mix-in that does common handling of SSH/SFTP URLs."""
140
 
 
141
 
    def __init__(self, base):
142
 
        self._parse_url(base)
143
 
        base = self._unparse_url(self._path)
144
 
        if base[-1] != '/':
145
 
            base += '/'
146
 
        super(SFTPUrlHandling, self).__init__(base)
147
 
 
148
 
    def _parse_url(self, url):
149
 
        (self._scheme,
150
 
         self._username, self._password,
151
 
         self._host, self._port, self._path) = self._split_url(url)
152
 
 
153
 
    def _unparse_url(self, path):
154
 
        """Return a URL for a path relative to this transport.
155
 
        """
156
 
        path = urllib.quote(path)
157
 
        # handle homedir paths
158
 
        if not path.startswith('/'):
159
 
            path = "/~/" + path
160
 
        netloc = urllib.quote(self._host)
161
 
        if self._username is not None:
162
 
            netloc = '%s@%s' % (urllib.quote(self._username), netloc)
163
 
        if self._port is not None:
164
 
            netloc = '%s:%d' % (netloc, self._port)
165
 
        return urlparse.urlunparse((self._scheme, netloc, path, '', '', ''))
166
 
 
167
 
    def _split_url(self, url):
168
 
        (scheme, username, password, host, port, path) = split_url(url)
169
 
        ## assert scheme == 'sftp'
170
 
 
171
 
        # the initial slash should be removed from the path, and treated
172
 
        # as a homedir relative path (the path begins with a double slash
173
 
        # if it is absolute).
174
 
        # see draft-ietf-secsh-scp-sftp-ssh-uri-03.txt
175
 
        # RBC 20060118 we are not using this as its too user hostile. instead
176
 
        # we are following lftp and using /~/foo to mean '~/foo'.
177
 
        # handle homedir paths
178
 
        if path.startswith('/~/'):
179
 
            path = path[3:]
180
 
        elif path == '/~':
181
 
            path = ''
182
 
        return (scheme, username, password, host, port, path)
183
 
 
184
 
    def abspath(self, relpath):
185
 
        """Return the full url to the given relative path.
186
 
        
187
 
        @param relpath: the relative path or path components
188
 
        @type relpath: str or list
189
 
        """
190
 
        return self._unparse_url(self._remote_path(relpath))
191
 
    
192
 
    def _remote_path(self, relpath):
193
 
        """Return the path to be passed along the sftp protocol for relpath.
194
 
        
195
 
        :param relpath: is a urlencoded string.
196
 
        """
197
 
        return self._combine_paths(self._path, relpath)
198
 
 
199
 
 
200
 
class SFTPTransport(SFTPUrlHandling):
 
137
class _SFTPReadvHelper(object):
 
138
    """A class to help with managing the state of a readv request."""
 
139
 
 
140
    # See _get_requests for an explanation.
 
141
    _max_request_size = 32768
 
142
 
 
143
    def __init__(self, original_offsets, relpath, _report_activity):
 
144
        """Create a new readv helper.
 
145
 
 
146
        :param original_offsets: The original requests given by the caller of
 
147
            readv()
 
148
        :param relpath: The name of the file (if known)
 
149
        :param _report_activity: A Transport._report_activity bound method,
 
150
            to be called as data arrives.
 
151
        """
 
152
        self.original_offsets = list(original_offsets)
 
153
        self.relpath = relpath
 
154
        self._report_activity = _report_activity
 
155
 
 
156
    def _get_requests(self):
 
157
        """Break up the offsets into individual requests over sftp.
 
158
 
 
159
        The SFTP spec only requires implementers to support 32kB requests. We
 
160
        could try something larger (openssh supports 64kB), but then we have to
 
161
        handle requests that fail.
 
162
        So instead, we just break up our maximum chunks into 32kB chunks, and
 
163
        asyncronously requests them.
 
164
        Newer versions of paramiko would do the chunking for us, but we want to
 
165
        start processing results right away, so we do it ourselves.
 
166
        """
 
167
        # TODO: Because we issue async requests, we don't 'fudge' any extra
 
168
        #       data.  I'm not 100% sure that is the best choice.
 
169
 
 
170
        # The first thing we do, is to collapse the individual requests as much
 
171
        # as possible, so we don't issues requests <32kB
 
172
        sorted_offsets = sorted(self.original_offsets)
 
173
        coalesced = list(ConnectedTransport._coalesce_offsets(sorted_offsets,
 
174
                                                        limit=0, fudge_factor=0))
 
175
        requests = []
 
176
        for c_offset in coalesced:
 
177
            start = c_offset.start
 
178
            size = c_offset.length
 
179
 
 
180
            # Break this up into 32kB requests
 
181
            while size > 0:
 
182
                next_size = min(size, self._max_request_size)
 
183
                requests.append((start, next_size))
 
184
                size -= next_size
 
185
                start += next_size
 
186
        if 'sftp' in debug.debug_flags:
 
187
            mutter('SFTP.readv(%s) %s offsets => %s coalesced => %s requests',
 
188
                self.relpath, len(sorted_offsets), len(coalesced),
 
189
                len(requests))
 
190
        return requests
 
191
 
 
192
    def request_and_yield_offsets(self, fp):
 
193
        """Request the data from the remote machine, yielding the results.
 
194
 
 
195
        :param fp: A Paramiko SFTPFile object that supports readv.
 
196
        :return: Yield the data requested by the original readv caller, one by
 
197
            one.
 
198
        """
 
199
        requests = self._get_requests()
 
200
        offset_iter = iter(self.original_offsets)
 
201
        cur_offset, cur_size = offset_iter.next()
 
202
        # paramiko .readv() yields strings that are in the order of the requests
 
203
        # So we track the current request to know where the next data is
 
204
        # being returned from.
 
205
        input_start = None
 
206
        last_end = None
 
207
        buffered_data = []
 
208
        buffered_len = 0
 
209
 
 
210
        # This is used to buffer chunks which we couldn't process yet
 
211
        # It is (start, end, data) tuples.
 
212
        data_chunks = []
 
213
        # Create an 'unlimited' data stream, so we stop based on requests,
 
214
        # rather than just because the data stream ended. This lets us detect
 
215
        # short readv.
 
216
        data_stream = itertools.chain(fp.readv(requests),
 
217
                                      itertools.repeat(None))
 
218
        for (start, length), data in itertools.izip(requests, data_stream):
 
219
            if data is None:
 
220
                if cur_coalesced is not None:
 
221
                    raise errors.ShortReadvError(self.relpath,
 
222
                        start, length, len(data))
 
223
            if len(data) != length:
 
224
                raise errors.ShortReadvError(self.relpath,
 
225
                    start, length, len(data))
 
226
            self._report_activity(length, 'read')
 
227
            if last_end is None:
 
228
                # This is the first request, just buffer it
 
229
                buffered_data = [data]
 
230
                buffered_len = length
 
231
                input_start = start
 
232
            elif start == last_end:
 
233
                # The data we are reading fits neatly on the previous
 
234
                # buffer, so this is all part of a larger coalesced range.
 
235
                buffered_data.append(data)
 
236
                buffered_len += length
 
237
            else:
 
238
                # We have an 'interrupt' in the data stream. So we know we are
 
239
                # at a request boundary.
 
240
                if buffered_len > 0:
 
241
                    # We haven't consumed the buffer so far, so put it into
 
242
                    # data_chunks, and continue.
 
243
                    buffered = ''.join(buffered_data)
 
244
                    data_chunks.append((input_start, buffered))
 
245
                input_start = start
 
246
                buffered_data = [data]
 
247
                buffered_len = length
 
248
            last_end = start + length
 
249
            if input_start == cur_offset and cur_size <= buffered_len:
 
250
                # Simplify the next steps a bit by transforming buffered_data
 
251
                # into a single string. We also have the nice property that
 
252
                # when there is only one string ''.join([x]) == x, so there is
 
253
                # no data copying.
 
254
                buffered = ''.join(buffered_data)
 
255
                # Clean out buffered data so that we keep memory
 
256
                # consumption low
 
257
                del buffered_data[:]
 
258
                buffered_offset = 0
 
259
                # TODO: We *could* also consider the case where cur_offset is in
 
260
                #       in the buffered range, even though it doesn't *start*
 
261
                #       the buffered range. But for packs we pretty much always
 
262
                #       read in order, so you won't get any extra data in the
 
263
                #       middle.
 
264
                while (input_start == cur_offset
 
265
                       and (buffered_offset + cur_size) <= buffered_len):
 
266
                    # We've buffered enough data to process this request, spit it
 
267
                    # out
 
268
                    cur_data = buffered[buffered_offset:buffered_offset + cur_size]
 
269
                    # move the direct pointer into our buffered data
 
270
                    buffered_offset += cur_size
 
271
                    # Move the start-of-buffer pointer
 
272
                    input_start += cur_size
 
273
                    # Yield the requested data
 
274
                    yield cur_offset, cur_data
 
275
                    cur_offset, cur_size = offset_iter.next()
 
276
                # at this point, we've consumed as much of buffered as we can,
 
277
                # so break off the portion that we consumed
 
278
                if buffered_offset == len(buffered_data):
 
279
                    # No tail to leave behind
 
280
                    buffered_data = []
 
281
                    buffered_len = 0
 
282
                else:
 
283
                    buffered = buffered[buffered_offset:]
 
284
                    buffered_data = [buffered]
 
285
                    buffered_len = len(buffered)
 
286
        if buffered_len:
 
287
            buffered = ''.join(buffered_data)
 
288
            del buffered_data[:]
 
289
            data_chunks.append((input_start, buffered))
 
290
        if data_chunks:
 
291
            if 'sftp' in debug.debug_flags:
 
292
                mutter('SFTP readv left with %d out-of-order bytes',
 
293
                    sum(map(lambda x: len(x[1]), data_chunks)))
 
294
            # We've processed all the readv data, at this point, anything we
 
295
            # couldn't process is in data_chunks. This doesn't happen often, so
 
296
            # this code path isn't optimized
 
297
            # We use an interesting process for data_chunks
 
298
            # Specifically if we have "bisect_left([(start, len, entries)],
 
299
            #                                       (qstart,)])
 
300
            # If start == qstart, then we get the specific node. Otherwise we
 
301
            # get the previous node
 
302
            while True:
 
303
                idx = bisect.bisect_left(data_chunks, (cur_offset,))
 
304
                if idx < len(data_chunks) and data_chunks[idx][0] == cur_offset:
 
305
                    # The data starts here
 
306
                    data = data_chunks[idx][1][:cur_size]
 
307
                elif idx > 0:
 
308
                    # The data is in a portion of a previous page
 
309
                    idx -= 1
 
310
                    sub_offset = cur_offset - data_chunks[idx][0]
 
311
                    data = data_chunks[idx][1]
 
312
                    data = data[sub_offset:sub_offset + cur_size]
 
313
                else:
 
314
                    # We are missing the page where the data should be found,
 
315
                    # something is wrong
 
316
                    data = ''
 
317
                if len(data) != cur_size:
 
318
                    raise AssertionError('We must have miscalulated.'
 
319
                        ' We expected %d bytes, but only found %d'
 
320
                        % (cur_size, len(data)))
 
321
                yield cur_offset, data
 
322
                cur_offset, cur_size = offset_iter.next()
 
323
 
 
324
 
 
325
class SFTPTransport(ConnectedTransport):
201
326
    """Transport implementation for SFTP access."""
202
327
 
203
328
    _do_prefetch = _default_do_prefetch
218
343
    # up the request itself, rather than us having to worry about it
219
344
    _max_request_size = 32768
220
345
 
221
 
    def __init__(self, base, clone_from=None):
222
 
        super(SFTPTransport, self).__init__(base)
223
 
        if clone_from is None:
224
 
            self._sftp_connect()
225
 
        else:
226
 
            # use the same ssh connection, etc
227
 
            self._sftp = clone_from._sftp
228
 
        # super saves 'self.base'
229
 
    
230
 
    def should_cache(self):
231
 
        """
232
 
        Return True if the data pulled across should be cached locally.
233
 
        """
234
 
        return True
235
 
 
236
 
    def clone(self, offset=None):
237
 
        """
238
 
        Return a new SFTPTransport with root at self.base + offset.
239
 
        We share the same SFTP session between such transports, because it's
240
 
        fairly expensive to set them up.
241
 
        """
242
 
        if offset is None:
243
 
            return SFTPTransport(self.base, self)
244
 
        else:
245
 
            return SFTPTransport(self.abspath(offset), self)
 
346
    def __init__(self, base, _from_transport=None):
 
347
        super(SFTPTransport, self).__init__(base,
 
348
                                            _from_transport=_from_transport)
246
349
 
247
350
    def _remote_path(self, relpath):
248
351
        """Return the path to be passed along the sftp protocol for relpath.
249
 
        
250
 
        relpath is a urlencoded string.
251
 
 
252
 
        :return: a path prefixed with / for regular abspath-based urls, or a
253
 
            path that does not begin with / for urls which begin with /~/.
254
 
        """
255
 
        # how does this work? 
256
 
        # it processes relpath with respect to 
257
 
        # our state:
258
 
        # firstly we create a path to evaluate: 
259
 
        # if relpath is an abspath or homedir path, its the entire thing
260
 
        # otherwise we join our base with relpath
261
 
        # then we eliminate all empty segments (double //'s) outside the first
262
 
        # two elements of the list. This avoids problems with trailing 
263
 
        # slashes, or other abnormalities.
264
 
        # finally we evaluate the entire path in a single pass
265
 
        # '.'s are stripped,
266
 
        # '..' result in popping the left most already 
267
 
        # processed path (which can never be empty because of the check for
268
 
        # abspath and homedir meaning that its not, or that we've used our
269
 
        # path. If the pop would pop the root, we ignore it.
270
 
 
271
 
        # Specific case examinations:
272
 
        # remove the special casefor ~: if the current root is ~/ popping of it
273
 
        # = / thus our seed for a ~ based path is ['', '~']
274
 
        # and if we end up with [''] then we had basically ('', '..') (which is
275
 
        # '/..' so we append '' if the length is one, and assert that the first
276
 
        # element is still ''. Lastly, if we end with ['', '~'] as a prefix for
277
 
        # the output, we've got a homedir path, so we strip that prefix before
278
 
        # '/' joining the resulting list.
279
 
        #
280
 
        # case one: '/' -> ['', ''] cannot shrink
281
 
        # case two: '/' + '../foo' -> ['', 'foo'] (take '', '', '..', 'foo')
282
 
        #           and pop the second '' for the '..', append 'foo'
283
 
        # case three: '/~/' -> ['', '~', ''] 
284
 
        # case four: '/~/' + '../foo' -> ['', '~', '', '..', 'foo'],
285
 
        #           and we want to get '/foo' - the empty path in the middle
286
 
        #           needs to be stripped, then normal path manipulation will 
287
 
        #           work.
288
 
        # case five: '/..' ['', '..'], we want ['', '']
289
 
        #            stripping '' outside the first two is ok
290
 
        #            ignore .. if its too high up
291
 
        #
292
 
        # lastly this code is possibly reusable by FTP, but not reusable by
293
 
        # local paths: ~ is resolvable correctly, nor by HTTP or the smart
294
 
        # server: ~ is resolved remotely.
295
 
        # 
296
 
        # however, a version of this that acts on self.base is possible to be
297
 
        # written which manipulates the URL in canonical form, and would be
298
 
        # reusable for all transports, if a flag for allowing ~/ at all was
299
 
        # provided.
300
 
        assert isinstance(relpath, basestring)
301
 
        relpath = urlutils.unescape(relpath)
302
 
 
303
 
        # case 1)
304
 
        if relpath.startswith('/'):
305
 
            # abspath - normal split is fine.
306
 
            current_path = relpath.split('/')
307
 
        elif relpath.startswith('~/'):
308
 
            # root is homedir based: normal split and prefix '' to remote the
309
 
            # special case
310
 
            current_path = [''].extend(relpath.split('/'))
 
352
 
 
353
        :param relpath: is a urlencoded string.
 
354
        """
 
355
        relative = urlutils.unescape(relpath).encode('utf-8')
 
356
        remote_path = self._combine_paths(self._path, relative)
 
357
        # the initial slash should be removed from the path, and treated as a
 
358
        # homedir relative path (the path begins with a double slash if it is
 
359
        # absolute).  see draft-ietf-secsh-scp-sftp-ssh-uri-03.txt
 
360
        # RBC 20060118 we are not using this as its too user hostile. instead
 
361
        # we are following lftp and using /~/foo to mean '~/foo'
 
362
        # vila--20070602 and leave absolute paths begin with a single slash.
 
363
        if remote_path.startswith('/~/'):
 
364
            remote_path = remote_path[3:]
 
365
        elif remote_path == '/~':
 
366
            remote_path = ''
 
367
        return remote_path
 
368
 
 
369
    def _create_connection(self, credentials=None):
 
370
        """Create a new connection with the provided credentials.
 
371
 
 
372
        :param credentials: The credentials needed to establish the connection.
 
373
 
 
374
        :return: The created connection and its associated credentials.
 
375
 
 
376
        The credentials are only the password as it may have been entered
 
377
        interactively by the user and may be different from the one provided
 
378
        in base url at transport creation time.
 
379
        """
 
380
        if credentials is None:
 
381
            password = self._password
311
382
        else:
312
 
            # root is from the current directory:
313
 
            if self._path.startswith('/'):
314
 
                # abspath, take the regular split
315
 
                current_path = []
316
 
            else:
317
 
                # homedir based, add the '', '~' not present in self._path
318
 
                current_path = ['', '~']
319
 
            # add our current dir
320
 
            current_path.extend(self._path.split('/'))
321
 
            # add the users relpath
322
 
            current_path.extend(relpath.split('/'))
323
 
        # strip '' segments that are not in the first one - the leading /.
324
 
        to_process = current_path[:1]
325
 
        for segment in current_path[1:]:
326
 
            if segment != '':
327
 
                to_process.append(segment)
328
 
 
329
 
        # process '.' and '..' segments into output_path.
330
 
        output_path = []
331
 
        for segment in to_process:
332
 
            if segment == '..':
333
 
                # directory pop. Remove a directory 
334
 
                # as long as we are not at the root
335
 
                if len(output_path) > 1:
336
 
                    output_path.pop()
337
 
                # else: pass
338
 
                # cannot pop beyond the root, so do nothing
339
 
            elif segment == '.':
340
 
                continue # strip the '.' from the output.
341
 
            else:
342
 
                # this will append '' to output_path for the root elements,
343
 
                # which is appropriate: its why we strip '' in the first pass.
344
 
                output_path.append(segment)
345
 
 
346
 
        # check output special cases:
347
 
        if output_path == ['']:
348
 
            # [''] -> ['', '']
349
 
            output_path = ['', '']
350
 
        elif output_path[:2] == ['', '~']:
351
 
            # ['', '~', ...] -> ...
352
 
            output_path = output_path[2:]
353
 
        path = '/'.join(output_path)
354
 
        return path
355
 
 
356
 
    def relpath(self, abspath):
357
 
        scheme, username, password, host, port, path = self._split_url(abspath)
358
 
        error = []
359
 
        if (username != self._username):
360
 
            error.append('username mismatch')
361
 
        if (host != self._host):
362
 
            error.append('host mismatch')
363
 
        if (port != self._port):
364
 
            error.append('port mismatch')
365
 
        if (not path.startswith(self._path)):
366
 
            error.append('path mismatch')
367
 
        if error:
368
 
            extra = ': ' + ', '.join(error)
369
 
            raise PathNotChild(abspath, self.base, extra=extra)
370
 
        pl = len(self._path)
371
 
        return path[pl:].strip('/')
 
383
            password = credentials
 
384
 
 
385
        vendor = ssh._get_ssh_vendor()
 
386
        user = self._user
 
387
        if user is None:
 
388
            auth = config.AuthenticationConfig()
 
389
            user = auth.get_user('ssh', self._host, self._port)
 
390
        connection = vendor.connect_sftp(self._user, password,
 
391
                                         self._host, self._port)
 
392
        return connection, (user, password)
 
393
 
 
394
    def _get_sftp(self):
 
395
        """Ensures that a connection is established"""
 
396
        connection = self._get_connection()
 
397
        if connection is None:
 
398
            # First connection ever
 
399
            connection, credentials = self._create_connection()
 
400
            self._set_connection(connection, credentials)
 
401
        return connection
372
402
 
373
403
    def has(self, relpath):
374
404
        """
375
405
        Does the target location exist?
376
406
        """
377
407
        try:
378
 
            self._sftp.stat(self._remote_path(relpath))
 
408
            self._get_sftp().stat(self._remote_path(relpath))
 
409
            # stat result is about 20 bytes, let's say
 
410
            self._report_activity(20, 'read')
379
411
            return True
380
412
        except IOError:
381
413
            return False
382
414
 
383
415
    def get(self, relpath):
384
 
        """
385
 
        Get the file at the given relative path.
 
416
        """Get the file at the given relative path.
386
417
 
387
418
        :param relpath: The relative path to the file
388
419
        """
389
420
        try:
 
421
            # FIXME: by returning the file directly, we don't pass this
 
422
            # through to report_activity.  We could try wrapping the object
 
423
            # before it's returned.  For readv and get_bytes it's handled in
 
424
            # the higher-level function.
 
425
            # -- mbp 20090126
390
426
            path = self._remote_path(relpath)
391
 
            f = self._sftp.file(path, mode='rb')
 
427
            f = self._get_sftp().file(path, mode='rb')
392
428
            if self._do_prefetch and (getattr(f, 'prefetch', None) is not None):
393
429
                f.prefetch()
394
430
            return f
396
432
            self._translate_io_exception(e, path, ': error retrieving',
397
433
                failure_exc=errors.ReadError)
398
434
 
399
 
    def readv(self, relpath, offsets):
 
435
    def get_bytes(self, relpath):
 
436
        # reimplement this here so that we can report how many bytes came back
 
437
        f = self.get(relpath)
 
438
        try:
 
439
            bytes = f.read()
 
440
            self._report_activity(len(bytes), 'read')
 
441
            return bytes
 
442
        finally:
 
443
            f.close()
 
444
 
 
445
    def _readv(self, relpath, offsets):
400
446
        """See Transport.readv()"""
401
447
        # We overload the default readv() because we want to use a file
402
448
        # that does not have prefetch enabled.
406
452
 
407
453
        try:
408
454
            path = self._remote_path(relpath)
409
 
            fp = self._sftp.file(path, mode='rb')
 
455
            fp = self._get_sftp().file(path, mode='rb')
410
456
            readv = getattr(fp, 'readv', None)
411
457
            if readv:
412
458
                return self._sftp_readv(fp, offsets, relpath)
413
 
            mutter('seek and read %s offsets', len(offsets))
 
459
            if 'sftp' in debug.debug_flags:
 
460
                mutter('seek and read %s offsets', len(offsets))
414
461
            return self._seek_and_read(fp, offsets, relpath)
415
462
        except (IOError, paramiko.SSHException), e:
416
463
            self._translate_io_exception(e, path, ': error retrieving')
417
464
 
418
 
    def _sftp_readv(self, fp, offsets, relpath='<unknown>'):
 
465
    def recommended_page_size(self):
 
466
        """See Transport.recommended_page_size().
 
467
 
 
468
        For SFTP we suggest a large page size to reduce the overhead
 
469
        introduced by latency.
 
470
        """
 
471
        return 64 * 1024
 
472
 
 
473
    def _sftp_readv(self, fp, offsets, relpath):
419
474
        """Use the readv() member of fp to do async readv.
420
475
 
421
 
        And then read them using paramiko.readv(). paramiko.readv()
 
476
        Then read them using paramiko.readv(). paramiko.readv()
422
477
        does not support ranges > 64K, so it caps the request size, and
423
 
        just reads until it gets all the stuff it wants
 
478
        just reads until it gets all the stuff it wants.
424
479
        """
425
 
        offsets = list(offsets)
426
 
        sorted_offsets = sorted(offsets)
427
 
 
428
 
        # The algorithm works as follows:
429
 
        # 1) Coalesce nearby reads into a single chunk
430
 
        #    This generates a list of combined regions, the total size
431
 
        #    and the size of the sub regions. This coalescing step is limited
432
 
        #    in the number of nearby chunks to combine, and is allowed to
433
 
        #    skip small breaks in the requests. Limiting it makes sure that
434
 
        #    we can start yielding some data earlier, and skipping means we
435
 
        #    make fewer requests. (Beneficial even when using async)
436
 
        # 2) Break up this combined regions into chunks that are smaller
437
 
        #    than 64KiB. Technically the limit is 65536, but we are a
438
 
        #    little bit conservative. This is because sftp has a maximum
439
 
        #    return chunk size of 64KiB (max size of an unsigned short)
440
 
        # 3) Issue a readv() to paramiko to create an async request for
441
 
        #    all of this data
442
 
        # 4) Read in the data as it comes back, until we've read one
443
 
        #    continuous section as determined in step 1
444
 
        # 5) Break up the full sections into hunks for the original requested
445
 
        #    offsets. And put them in a cache
446
 
        # 6) Check if the next request is in the cache, and if it is, remove
447
 
        #    it from the cache, and yield its data. Continue until no more
448
 
        #    entries are in the cache.
449
 
        # 7) loop back to step 4 until all data has been read
450
 
        #
451
 
        # TODO: jam 20060725 This could be optimized one step further, by
452
 
        #       attempting to yield whatever data we have read, even before
453
 
        #       the first coallesced section has been fully processed.
454
 
 
455
 
        # When coalescing for use with readv(), we don't really need to
456
 
        # use any fudge factor, because the requests are made asynchronously
457
 
        coalesced = list(self._coalesce_offsets(sorted_offsets,
458
 
                               limit=self._max_readv_combine,
459
 
                               fudge_factor=0,
460
 
                               ))
461
 
        requests = []
462
 
        for c_offset in coalesced:
463
 
            start = c_offset.start
464
 
            size = c_offset.length
465
 
 
466
 
            # We need to break this up into multiple requests
467
 
            while size > 0:
468
 
                next_size = min(size, self._max_request_size)
469
 
                requests.append((start, next_size))
470
 
                size -= next_size
471
 
                start += next_size
472
 
 
473
 
        mutter('SFTP.readv() %s offsets => %s coalesced => %s requests',
474
 
                len(offsets), len(coalesced), len(requests))
475
 
 
476
 
        # Queue the current read until we have read the full coalesced section
477
 
        cur_data = []
478
 
        cur_data_len = 0
479
 
        cur_coalesced_stack = iter(coalesced)
480
 
        cur_coalesced = cur_coalesced_stack.next()
481
 
 
482
 
        # Cache the results, but only until they have been fulfilled
483
 
        data_map = {}
484
 
        # turn the list of offsets into a stack
485
 
        offset_stack = iter(offsets)
486
 
        cur_offset_and_size = offset_stack.next()
487
 
 
488
 
        for data in fp.readv(requests):
489
 
            cur_data += data
490
 
            cur_data_len += len(data)
491
 
 
492
 
            if cur_data_len < cur_coalesced.length:
493
 
                continue
494
 
            assert cur_data_len == cur_coalesced.length, \
495
 
                "Somehow we read too much: %s != %s" % (cur_data_len,
496
 
                                                        cur_coalesced.length)
497
 
            all_data = ''.join(cur_data)
498
 
            cur_data = []
499
 
            cur_data_len = 0
500
 
 
501
 
            for suboffset, subsize in cur_coalesced.ranges:
502
 
                key = (cur_coalesced.start+suboffset, subsize)
503
 
                data_map[key] = all_data[suboffset:suboffset+subsize]
504
 
 
505
 
            # Now that we've read some data, see if we can yield anything back
506
 
            while cur_offset_and_size in data_map:
507
 
                this_data = data_map.pop(cur_offset_and_size)
508
 
                yield cur_offset_and_size[0], this_data
509
 
                cur_offset_and_size = offset_stack.next()
510
 
 
511
 
            # We read a coalesced entry, so mark it as done
512
 
            cur_coalesced = None
513
 
            # Now that we've read all of the data for this coalesced section
514
 
            # on to the next
515
 
            cur_coalesced = cur_coalesced_stack.next()
516
 
 
517
 
        if cur_coalesced is not None:
518
 
            raise errors.ShortReadvError(relpath, cur_coalesced.start,
519
 
                cur_coalesced.length, len(data))
 
480
        helper = _SFTPReadvHelper(offsets, relpath, self._report_activity)
 
481
        return helper.request_and_yield_offsets(fp)
520
482
 
521
483
    def put_file(self, relpath, f, mode=None):
522
484
        """
527
489
        :param mode: The final mode for the file
528
490
        """
529
491
        final_path = self._remote_path(relpath)
530
 
        self._put(final_path, f, mode=mode)
 
492
        return self._put(final_path, f, mode=mode)
531
493
 
532
494
    def _put(self, abspath, f, mode=None):
533
495
        """Helper function so both put() and copy_abspaths can reuse the code"""
538
500
        try:
539
501
            try:
540
502
                fout.set_pipelined(True)
541
 
                self._pump(f, fout)
 
503
                length = self._pump(f, fout)
542
504
            except (IOError, paramiko.SSHException), e:
543
505
                self._translate_io_exception(e, tmp_abspath)
544
506
            # XXX: This doesn't truly help like we would like it to.
547
509
            #      sticky bit. So it is probably best to stop chmodding, and
548
510
            #      just tell users that they need to set the umask correctly.
549
511
            #      The attr.st_mode = mode, in _sftp_open_exclusive
550
 
            #      will handle when the user wants the final mode to be more 
551
 
            #      restrictive. And then we avoid a round trip. Unless 
 
512
            #      will handle when the user wants the final mode to be more
 
513
            #      restrictive. And then we avoid a round trip. Unless
552
514
            #      paramiko decides to expose an async chmod()
553
515
 
554
516
            # This is designed to chmod() right before we close.
555
 
            # Because we set_pipelined() earlier, theoretically we might 
 
517
            # Because we set_pipelined() earlier, theoretically we might
556
518
            # avoid the round trip for fout.close()
557
519
            if mode is not None:
558
 
                self._sftp.chmod(tmp_abspath, mode)
 
520
                self._get_sftp().chmod(tmp_abspath, mode)
559
521
            fout.close()
560
522
            closed = True
561
523
            self._rename_and_overwrite(tmp_abspath, abspath)
 
524
            return length
562
525
        except Exception, e:
563
526
            # If we fail, try to clean up the temporary file
564
527
            # before we throw the exception
570
533
            try:
571
534
                if not closed:
572
535
                    fout.close()
573
 
                self._sftp.remove(tmp_abspath)
 
536
                self._get_sftp().remove(tmp_abspath)
574
537
            except:
575
538
                # raise the saved except
576
539
                raise e
591
554
            fout = None
592
555
            try:
593
556
                try:
594
 
                    fout = self._sftp.file(abspath, mode='wb')
 
557
                    fout = self._get_sftp().file(abspath, mode='wb')
595
558
                    fout.set_pipelined(True)
596
559
                    writer(fout)
597
560
                except (paramiko.SSHException, IOError), e:
599
562
                                                 ': unable to open')
600
563
 
601
564
                # This is designed to chmod() right before we close.
602
 
                # Because we set_pipelined() earlier, theoretically we might 
 
565
                # Because we set_pipelined() earlier, theoretically we might
603
566
                # avoid the round trip for fout.close()
604
567
                if mode is not None:
605
 
                    self._sftp.chmod(abspath, mode)
 
568
                    self._get_sftp().chmod(abspath, mode)
606
569
            finally:
607
570
                if fout is not None:
608
571
                    fout.close()
656
619
 
657
620
    def iter_files_recursive(self):
658
621
        """Walk the relative paths of all files in this transport."""
 
622
        # progress is handled by list_dir
659
623
        queue = list(self.list_dir('.'))
660
624
        while queue:
661
625
            relpath = queue.pop(0)
672
636
        else:
673
637
            local_mode = mode
674
638
        try:
675
 
            self._sftp.mkdir(abspath, local_mode)
 
639
            self._report_activity(len(abspath), 'write')
 
640
            self._get_sftp().mkdir(abspath, local_mode)
 
641
            self._report_activity(1, 'read')
676
642
            if mode is not None:
677
 
                self._sftp.chmod(abspath, mode=mode)
 
643
                # chmod a dir through sftp will erase any sgid bit set
 
644
                # on the server side.  So, if the bit mode are already
 
645
                # set, avoid the chmod.  If the mode is not fine but
 
646
                # the sgid bit is set, report a warning to the user
 
647
                # with the umask fix.
 
648
                stat = self._get_sftp().lstat(abspath)
 
649
                mode = mode & 0777 # can't set special bits anyway
 
650
                if mode != stat.st_mode & 0777:
 
651
                    if stat.st_mode & 06000:
 
652
                        warning('About to chmod %s over sftp, which will result'
 
653
                                ' in its suid or sgid bits being cleared.  If'
 
654
                                ' you want to preserve those bits, change your '
 
655
                                ' environment on the server to use umask 0%03o.'
 
656
                                % (abspath, 0777 - mode))
 
657
                    self._get_sftp().chmod(abspath, mode=mode)
678
658
        except (paramiko.SSHException, IOError), e:
679
659
            self._translate_io_exception(e, abspath, ': unable to mkdir',
680
660
                failure_exc=FileExists)
683
663
        """Create a directory at the given path."""
684
664
        self._mkdir(self._remote_path(relpath), mode=mode)
685
665
 
 
666
    def open_write_stream(self, relpath, mode=None):
 
667
        """See Transport.open_write_stream."""
 
668
        # initialise the file to zero-length
 
669
        # this is three round trips, but we don't use this
 
670
        # api more than once per write_group at the moment so
 
671
        # it is a tolerable overhead. Better would be to truncate
 
672
        # the file after opening. RBC 20070805
 
673
        self.put_bytes_non_atomic(relpath, "", mode)
 
674
        abspath = self._remote_path(relpath)
 
675
        # TODO: jam 20060816 paramiko doesn't publicly expose a way to
 
676
        #       set the file mode at create time. If it does, use it.
 
677
        #       But for now, we just chmod later anyway.
 
678
        handle = None
 
679
        try:
 
680
            handle = self._get_sftp().file(abspath, mode='wb')
 
681
            handle.set_pipelined(True)
 
682
        except (paramiko.SSHException, IOError), e:
 
683
            self._translate_io_exception(e, abspath,
 
684
                                         ': unable to open')
 
685
        _file_streams[self.abspath(relpath)] = handle
 
686
        return FileFileStream(self, relpath, handle)
 
687
 
686
688
    def _translate_io_exception(self, e, path, more_info='',
687
689
                                failure_exc=PathError):
688
690
        """Translate a paramiko or IOError into a friendlier exception.
694
696
        :param failure_exc: Paramiko has the super fun ability to raise completely
695
697
                           opaque errors that just set "e.args = ('Failure',)" with
696
698
                           no more information.
697
 
                           If this parameter is set, it defines the exception 
 
699
                           If this parameter is set, it defines the exception
698
700
                           to raise in these cases.
699
701
        """
700
702
        # paramiko seems to generate detailless errors.
703
705
            if (e.args == ('No such file or directory',) or
704
706
                e.args == ('No such file',)):
705
707
                raise NoSuchFile(path, str(e) + more_info)
706
 
            if (e.args == ('mkdir failed',)):
 
708
            if (e.args == ('mkdir failed',) or
 
709
                e.args[0].startswith('syserr: File exists')):
707
710
                raise FileExists(path, str(e) + more_info)
708
711
            # strange but true, for the paramiko server.
709
712
            if (e.args == ('Failure',)):
710
713
                raise failure_exc(path, str(e) + more_info)
 
714
            # Can be something like args = ('Directory not empty:
 
715
            # '/srv/bazaar.launchpad.net/blah...: '
 
716
            # [Errno 39] Directory not empty',)
 
717
            if (e.args[0].startswith('Directory not empty: ')
 
718
                or getattr(e, 'errno', None) == errno.ENOTEMPTY):
 
719
                raise errors.DirectoryNotEmpty(path, str(e))
711
720
            mutter('Raising exception with args %s', e.args)
712
721
        if getattr(e, 'errno', None) is not None:
713
722
            mutter('Raising exception with errno %s', e.errno)
720
729
        """
721
730
        try:
722
731
            path = self._remote_path(relpath)
723
 
            fout = self._sftp.file(path, 'ab')
 
732
            fout = self._get_sftp().file(path, 'ab')
724
733
            if mode is not None:
725
 
                self._sftp.chmod(path, mode)
 
734
                self._get_sftp().chmod(path, mode)
726
735
            result = fout.tell()
727
736
            self._pump(f, fout)
728
737
            return result
732
741
    def rename(self, rel_from, rel_to):
733
742
        """Rename without special overwriting"""
734
743
        try:
735
 
            self._sftp.rename(self._remote_path(rel_from),
 
744
            self._get_sftp().rename(self._remote_path(rel_from),
736
745
                              self._remote_path(rel_to))
737
746
        except (IOError, paramiko.SSHException), e:
738
747
            self._translate_io_exception(e, rel_from,
740
749
 
741
750
    def _rename_and_overwrite(self, abs_from, abs_to):
742
751
        """Do a fancy rename on the remote server.
743
 
        
 
752
 
744
753
        Using the implementation provided by osutils.
745
754
        """
746
755
        try:
 
756
            sftp = self._get_sftp()
747
757
            fancy_rename(abs_from, abs_to,
748
 
                    rename_func=self._sftp.rename,
749
 
                    unlink_func=self._sftp.remove)
 
758
                         rename_func=sftp.rename,
 
759
                         unlink_func=sftp.remove)
750
760
        except (IOError, paramiko.SSHException), e:
751
 
            self._translate_io_exception(e, abs_from, ': unable to rename to %r' % (abs_to))
 
761
            self._translate_io_exception(e, abs_from,
 
762
                                         ': unable to rename to %r' % (abs_to))
752
763
 
753
764
    def move(self, rel_from, rel_to):
754
765
        """Move the item at rel_from to the location at rel_to"""
760
771
        """Delete the item at relpath"""
761
772
        path = self._remote_path(relpath)
762
773
        try:
763
 
            self._sftp.remove(path)
 
774
            self._get_sftp().remove(path)
764
775
        except (IOError, paramiko.SSHException), e:
765
776
            self._translate_io_exception(e, path, ': unable to delete')
766
 
            
 
777
 
 
778
    def external_url(self):
 
779
        """See bzrlib.transport.Transport.external_url."""
 
780
        # the external path for SFTP is the base
 
781
        return self.base
 
782
 
767
783
    def listable(self):
768
784
        """Return True if this store supports listing."""
769
785
        return True
778
794
        # -- David Allouche 2006-08-11
779
795
        path = self._remote_path(relpath)
780
796
        try:
781
 
            entries = self._sftp.listdir(path)
 
797
            entries = self._get_sftp().listdir(path)
 
798
            self._report_activity(sum(map(len, entries)), 'read')
782
799
        except (IOError, paramiko.SSHException), e:
783
800
            self._translate_io_exception(e, path, ': failed to list_dir')
784
801
        return [urlutils.escape(entry) for entry in entries]
787
804
        """See Transport.rmdir."""
788
805
        path = self._remote_path(relpath)
789
806
        try:
790
 
            return self._sftp.rmdir(path)
 
807
            return self._get_sftp().rmdir(path)
791
808
        except (IOError, paramiko.SSHException), e:
792
809
            self._translate_io_exception(e, path, ': failed to rmdir')
793
810
 
795
812
        """Return the stat information for a file."""
796
813
        path = self._remote_path(relpath)
797
814
        try:
798
 
            return self._sftp.stat(path)
 
815
            return self._get_sftp().stat(path)
799
816
        except (IOError, paramiko.SSHException), e:
800
817
            self._translate_io_exception(e, path, ': unable to stat')
801
818
 
825
842
        # that we have taken the lock.
826
843
        return SFTPLock(relpath, self)
827
844
 
828
 
    def _sftp_connect(self):
829
 
        """Connect to the remote sftp server.
830
 
        After this, self._sftp should have a valid connection (or
831
 
        we raise an TransportError 'could not connect').
832
 
 
833
 
        TODO: Raise a more reasonable ConnectionFailed exception
834
 
        """
835
 
        self._sftp = _sftp_connect(self._host, self._port, self._username,
836
 
                self._password)
837
 
 
838
845
    def _sftp_open_exclusive(self, abspath, mode=None):
839
846
        """Open a remote path exclusively.
840
847
 
851
858
        """
852
859
        # TODO: jam 20060816 Paramiko >= 1.6.2 (probably earlier) supports
853
860
        #       using the 'x' flag to indicate SFTP_FLAG_EXCL.
854
 
        #       However, there is no way to set the permission mode at open 
 
861
        #       However, there is no way to set the permission mode at open
855
862
        #       time using the sftp_client.file() functionality.
856
 
        path = self._sftp._adjust_cwd(abspath)
 
863
        path = self._get_sftp()._adjust_cwd(abspath)
857
864
        # mutter('sftp abspath %s => %s', abspath, path)
858
865
        attr = SFTPAttributes()
859
866
        if mode is not None:
860
867
            attr.st_mode = mode
861
 
        omode = (SFTP_FLAG_WRITE | SFTP_FLAG_CREATE 
 
868
        omode = (SFTP_FLAG_WRITE | SFTP_FLAG_CREATE
862
869
                | SFTP_FLAG_TRUNC | SFTP_FLAG_EXCL)
863
870
        try:
864
 
            t, msg = self._sftp._request(CMD_OPEN, path, omode, attr)
 
871
            t, msg = self._get_sftp()._request(CMD_OPEN, path, omode, attr)
865
872
            if t != CMD_HANDLE:
866
873
                raise TransportError('Expected an SFTP handle')
867
874
            handle = msg.get_string()
868
 
            return SFTPFile(self._sftp, handle, 'wb', -1)
 
875
            return SFTPFile(self._get_sftp(), handle, 'wb', -1)
869
876
        except (paramiko.SSHException, IOError), e:
870
877
            self._translate_io_exception(e, abspath, ': unable to open',
871
878
                failure_exc=FileExists)
942
949
                # probably a failed test; unit test thread will log the
943
950
                # failure/error
944
951
                sys.excepthook(*sys.exc_info())
945
 
                warning('Exception from within unit test server thread: %r' % 
 
952
                warning('Exception from within unit test server thread: %r' %
946
953
                        x)
947
954
 
948
955
 
959
966
 
960
967
    Not all methods are implemented, this is deliberate as this class is not a
961
968
    replacement for the builtin sockets layer. fileno is not implemented to
962
 
    prevent the proxy being bypassed. 
 
969
    prevent the proxy being bypassed.
963
970
    """
964
971
 
965
972
    simulated_time = 0
967
974
        "close", "getpeername", "getsockname", "getsockopt", "gettimeout",
968
975
        "setblocking", "setsockopt", "settimeout", "shutdown"])
969
976
 
970
 
    def __init__(self, sock, latency, bandwidth=1.0, 
 
977
    def __init__(self, sock, latency, bandwidth=1.0,
971
978
                 really_sleep=True):
972
 
        """ 
 
979
        """
973
980
        :param bandwith: simulated bandwith (MegaBit)
974
981
        :param really_sleep: If set to false, the SocketDelay will just
975
982
        increase a counter, instead of calling time.sleep. This is useful for
978
985
        self.sock = sock
979
986
        self.latency = latency
980
987
        self.really_sleep = really_sleep
981
 
        self.time_per_byte = 1 / (bandwidth / 8.0 * 1024 * 1024) 
 
988
        self.time_per_byte = 1 / (bandwidth / 8.0 * 1024 * 1024)
982
989
        self.new_roundtrip = False
983
990
 
984
991
    def sleep(self, s):
1046
1053
 
1047
1054
    def _run_server_entry(self, sock):
1048
1055
        """Entry point for all implementations of _run_server.
1049
 
        
 
1056
 
1050
1057
        If self.add_latency is > 0.000001 then sock is given a latency adding
1051
1058
        decorator.
1052
1059
        """
1069
1076
        event = threading.Event()
1070
1077
        ssh_server.start_server(event, server)
1071
1078
        event.wait(5.0)
1072
 
    
 
1079
 
1073
1080
    def setUp(self, backing_server=None):
1074
1081
        # XXX: TODO: make sftpserver back onto backing_server rather than local
1075
1082
        # disk.
1076
 
        assert (backing_server is None or
1077
 
                isinstance(backing_server, local.LocalURLServer)), (
1078
 
            "backing_server should not be %r, because this can only serve the "
1079
 
            "local current working directory." % (backing_server,))
 
1083
        if not (backing_server is None or
 
1084
                isinstance(backing_server, local.LocalURLServer)):
 
1085
            raise AssertionError(
 
1086
                "backing_server should not be %r, because this can only serve the "
 
1087
                "local current working directory." % (backing_server,))
1080
1088
        self._original_vendor = ssh._ssh_vendor_manager._cached_ssh_vendor
1081
1089
        ssh._ssh_vendor_manager._cached_ssh_vendor = self._vendor
1082
1090
        if sys.platform == 'win32':
1144
1152
            def close(self):
1145
1153
                pass
1146
1154
 
1147
 
        server = paramiko.SFTPServer(FakeChannel(), 'sftp', StubServer(self), StubSFTPServer,
1148
 
                                     root=self._root, home=self._server_homedir)
 
1155
        server = paramiko.SFTPServer(
 
1156
            FakeChannel(), 'sftp', StubServer(self), StubSFTPServer,
 
1157
            root=self._root, home=self._server_homedir)
1149
1158
        try:
1150
 
            server.start_subsystem('sftp', None, sock)
 
1159
            server.start_subsystem(
 
1160
                'sftp', None, ssh.SocketAsChannelAdapter(sock))
1151
1161
        except socket.error, e:
1152
1162
            if (len(e.args) > 0) and (e.args[0] == errno.EPIPE):
1153
1163
                # it's okay for the client to disconnect abruptly
1197
1207
        super(SFTPSiblingAbsoluteServer, self).setUp(backing_server)
1198
1208
 
1199
1209
 
1200
 
def _sftp_connect(host, port, username, password):
1201
 
    """Connect to the remote sftp server.
1202
 
 
1203
 
    :raises: a TransportError 'could not connect'.
1204
 
 
1205
 
    :returns: an paramiko.sftp_client.SFTPClient
1206
 
 
1207
 
    TODO: Raise a more reasonable ConnectionFailed exception
1208
 
    """
1209
 
    idx = (host, port, username)
1210
 
    try:
1211
 
        return _connected_hosts[idx]
1212
 
    except KeyError:
1213
 
        pass
1214
 
    
1215
 
    sftp = _sftp_connect_uncached(host, port, username, password)
1216
 
    _connected_hosts[idx] = sftp
1217
 
    return sftp
1218
 
 
1219
 
def _sftp_connect_uncached(host, port, username, password):
1220
 
    vendor = ssh._get_ssh_vendor()
1221
 
    sftp = vendor.connect_sftp(username, password, host, port)
1222
 
    return sftp
1223
 
 
1224
 
 
1225
1210
def get_test_permutations():
1226
1211
    """Return the permutations to be used in testing."""
1227
1212
    return [(SFTPTransport, SFTPAbsoluteServer),