~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/sftp.py

  • Committer: Martin Pool
  • Date: 2009-08-20 04:53:23 UTC
  • mto: This revision was merged to the branch mainline in revision 4632.
  • Revision ID: mbp@sourcefrog.net-20090820045323-4hsicfa87pdq3l29
Correction to xdg_cache_dir and add a simple test

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 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
 
 
19
# TODO: Remove the transport-based lock_read and lock_write methods.  They'll
 
20
# then raise TransportNotPossible, which will break remote access to any
 
21
# formats which rely on OS-level locks.  That should be fine as those formats
 
22
# are pretty old, but these combinations may have to be removed from the test
 
23
# suite.  Those formats all date back to 0.7; so we should be able to remove
 
24
# these methods when we officially drop support for those formats.
 
25
 
 
26
import bisect
20
27
import errno
21
 
import getpass
22
28
import itertools
23
29
import os
24
30
import random
25
 
import re
26
31
import select
27
32
import socket
28
33
import stat
29
 
import subprocess
30
34
import sys
31
35
import time
32
36
import urllib
33
37
import urlparse
34
 
import weakref
 
38
import warnings
35
39
 
36
 
from bzrlib.config import config_dir, ensure_config_dir_exists
37
 
from bzrlib.errors import (ConnectionError,
38
 
                           FileExists, 
39
 
                           TransportNotPossible, NoSuchFile, PathNotChild,
 
40
from bzrlib import (
 
41
    config,
 
42
    debug,
 
43
    errors,
 
44
    urlutils,
 
45
    )
 
46
from bzrlib.errors import (FileExists,
 
47
                           NoSuchFile, PathNotChild,
40
48
                           TransportError,
41
 
                           LockError, 
 
49
                           LockError,
42
50
                           PathError,
43
51
                           ParamikoNotPresent,
44
52
                           )
45
53
from bzrlib.osutils import pathjoin, fancy_rename, getcwd
46
 
from bzrlib.trace import mutter, warning, error
 
54
from bzrlib.symbol_versioning import (
 
55
        deprecated_function,
 
56
        )
 
57
from bzrlib.trace import mutter, warning
47
58
from bzrlib.transport import (
48
 
    register_urlparse_netloc_protocol,
 
59
    FileFileStream,
 
60
    _file_streams,
 
61
    local,
49
62
    Server,
50
 
    split_url,
51
 
    Transport,
 
63
    ssh,
 
64
    ConnectedTransport,
52
65
    )
53
 
import bzrlib.ui
54
 
import bzrlib.urlutils as urlutils
 
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')
55
79
 
56
80
try:
57
81
    import paramiko
63
87
                               CMD_HANDLE, CMD_OPEN)
64
88
    from paramiko.sftp_attr import SFTPAttributes
65
89
    from paramiko.sftp_file import SFTPFile
66
 
    from paramiko.sftp_client import SFTPClient
67
 
 
68
 
 
69
 
register_urlparse_netloc_protocol('sftp')
70
 
 
71
 
 
72
 
def _ignore_sigint():
73
 
    # TODO: This should possibly ignore SIGHUP as well, but bzr currently
74
 
    # doesn't handle it itself.
75
 
    # <https://launchpad.net/products/bzr/+bug/41433/+index>
76
 
    import signal
77
 
    signal.signal(signal.SIGINT, signal.SIG_IGN)
78
 
    
79
 
 
80
 
def os_specific_subprocess_params():
81
 
    """Get O/S specific subprocess parameters."""
82
 
    if sys.platform == 'win32':
83
 
        # setting the process group and closing fds is not supported on 
84
 
        # win32
85
 
        return {}
86
 
    else:
87
 
        # We close fds other than the pipes as the child process does not need 
88
 
        # them to be open.
89
 
        #
90
 
        # We also set the child process to ignore SIGINT.  Normally the signal
91
 
        # would be sent to every process in the foreground process group, but
92
 
        # this causes it to be seen only by bzr and not by ssh.  Python will
93
 
        # generate a KeyboardInterrupt in bzr, and we will then have a chance
94
 
        # to release locks or do other cleanup over ssh before the connection
95
 
        # goes away.  
96
 
        # <https://launchpad.net/products/bzr/+bug/5987>
97
 
        #
98
 
        # Running it in a separate process group is not good because then it
99
 
        # can't get non-echoed input of a password or passphrase.
100
 
        # <https://launchpad.net/products/bzr/+bug/40508>
101
 
        return {'preexec_fn': _ignore_sigint,
102
 
                'close_fds': True,
103
 
                }
104
90
 
105
91
 
106
92
_paramiko_version = getattr(paramiko, '__version_info__', (0, 0, 0))
107
93
# don't use prefetch unless paramiko version >= 1.5.5 (there were bugs earlier)
108
94
_default_do_prefetch = (_paramiko_version >= (1, 5, 5))
109
95
 
110
 
# Paramiko 1.5 tries to open a socket.AF_UNIX in order to connect
111
 
# to ssh-agent. That attribute doesn't exist on win32 (it does in cygwin)
112
 
# so we get an AttributeError exception. So we will not try to
113
 
# connect to an agent if we are on win32 and using Paramiko older than 1.6
114
 
_use_ssh_agent = (sys.platform != 'win32' or _paramiko_version >= (1, 6, 0))
115
 
 
116
 
 
117
 
_ssh_vendor = None
118
 
def _get_ssh_vendor():
119
 
    """Find out what version of SSH is on the system."""
120
 
    global _ssh_vendor
121
 
    if _ssh_vendor is not None:
122
 
        return _ssh_vendor
123
 
 
124
 
    _ssh_vendor = 'none'
125
 
 
126
 
    if 'BZR_SSH' in os.environ:
127
 
        _ssh_vendor = os.environ['BZR_SSH']
128
 
        if _ssh_vendor == 'paramiko':
129
 
            _ssh_vendor = 'none'
130
 
        return _ssh_vendor
131
 
 
132
 
    try:
133
 
        p = subprocess.Popen(['ssh', '-V'],
134
 
                             stdin=subprocess.PIPE,
135
 
                             stdout=subprocess.PIPE,
136
 
                             stderr=subprocess.PIPE,
137
 
                             **os_specific_subprocess_params())
138
 
        returncode = p.returncode
139
 
        stdout, stderr = p.communicate()
140
 
    except OSError:
141
 
        returncode = -1
142
 
        stdout = stderr = ''
143
 
    if 'OpenSSH' in stderr:
144
 
        mutter('ssh implementation is OpenSSH')
145
 
        _ssh_vendor = 'openssh'
146
 
    elif 'SSH Secure Shell' in stderr:
147
 
        mutter('ssh implementation is SSH Corp.')
148
 
        _ssh_vendor = 'ssh'
149
 
 
150
 
    if _ssh_vendor != 'none':
151
 
        return _ssh_vendor
152
 
 
153
 
    # XXX: 20051123 jamesh
154
 
    # A check for putty's plink or lsh would go here.
155
 
 
156
 
    mutter('falling back to paramiko implementation')
157
 
    return _ssh_vendor
158
 
 
159
 
 
160
 
class SFTPSubprocess:
161
 
    """A socket-like object that talks to an ssh subprocess via pipes."""
162
 
    def __init__(self, hostname, vendor, port=None, user=None):
163
 
        assert vendor in ['openssh', 'ssh']
164
 
        if vendor == 'openssh':
165
 
            args = ['ssh',
166
 
                    '-oForwardX11=no', '-oForwardAgent=no',
167
 
                    '-oClearAllForwardings=yes', '-oProtocol=2',
168
 
                    '-oNoHostAuthenticationForLocalhost=yes']
169
 
            if port is not None:
170
 
                args.extend(['-p', str(port)])
171
 
            if user is not None:
172
 
                args.extend(['-l', user])
173
 
            args.extend(['-s', hostname, 'sftp'])
174
 
        elif vendor == 'ssh':
175
 
            args = ['ssh', '-x']
176
 
            if port is not None:
177
 
                args.extend(['-p', str(port)])
178
 
            if user is not None:
179
 
                args.extend(['-l', user])
180
 
            args.extend(['-s', 'sftp', hostname])
181
 
 
182
 
        self.proc = subprocess.Popen(args,
183
 
                                     stdin=subprocess.PIPE,
184
 
                                     stdout=subprocess.PIPE,
185
 
                                     **os_specific_subprocess_params())
186
 
 
187
 
    def send(self, data):
188
 
        return os.write(self.proc.stdin.fileno(), data)
189
 
 
190
 
    def recv_ready(self):
191
 
        # TODO: jam 20051215 this function is necessary to support the
192
 
        # pipelined() function. In reality, it probably should use
193
 
        # poll() or select() to actually return if there is data
194
 
        # available, otherwise we probably don't get any benefit
195
 
        return True
196
 
 
197
 
    def recv(self, count):
198
 
        return os.read(self.proc.stdout.fileno(), count)
199
 
 
200
 
    def close(self):
201
 
        self.proc.stdin.close()
202
 
        self.proc.stdout.close()
203
 
        self.proc.wait()
204
 
 
205
 
 
206
 
class LoopbackSFTP(object):
207
 
    """Simple wrapper for a socket that pretends to be a paramiko Channel."""
208
 
 
209
 
    def __init__(self, sock):
210
 
        self.__socket = sock
211
 
 
212
 
    def send(self, data):
213
 
        return self.__socket.send(data)
214
 
 
215
 
    def recv(self, n):
216
 
        return self.__socket.recv(n)
217
 
 
218
 
    def recv_ready(self):
219
 
        return True
220
 
 
221
 
    def close(self):
222
 
        self.__socket.close()
223
 
 
224
 
 
225
 
SYSTEM_HOSTKEYS = {}
226
 
BZR_HOSTKEYS = {}
227
 
 
228
 
# This is a weakref dictionary, so that we can reuse connections
229
 
# that are still active. Long term, it might be nice to have some
230
 
# sort of expiration policy, such as disconnect if inactive for
231
 
# X seconds. But that requires a lot more fanciness.
232
 
_connected_hosts = weakref.WeakValueDictionary()
233
 
 
234
 
def clear_connection_cache():
235
 
    """Remove all hosts from the SFTP connection cache.
236
 
 
237
 
    Primarily useful for test cases wanting to force garbage collection.
238
 
    """
239
 
    _connected_hosts.clear()
240
 
 
241
 
 
242
 
def load_host_keys():
243
 
    """
244
 
    Load system host keys (probably doesn't work on windows) and any
245
 
    "discovered" keys from previous sessions.
246
 
    """
247
 
    global SYSTEM_HOSTKEYS, BZR_HOSTKEYS
248
 
    try:
249
 
        SYSTEM_HOSTKEYS = paramiko.util.load_host_keys(os.path.expanduser('~/.ssh/known_hosts'))
250
 
    except Exception, e:
251
 
        mutter('failed to load system host keys: ' + str(e))
252
 
    bzr_hostkey_path = pathjoin(config_dir(), 'ssh_host_keys')
253
 
    try:
254
 
        BZR_HOSTKEYS = paramiko.util.load_host_keys(bzr_hostkey_path)
255
 
    except Exception, e:
256
 
        mutter('failed to load bzr host keys: ' + str(e))
257
 
        save_host_keys()
258
 
 
259
 
 
260
 
def save_host_keys():
261
 
    """
262
 
    Save "discovered" host keys in $(config)/ssh_host_keys/.
263
 
    """
264
 
    global SYSTEM_HOSTKEYS, BZR_HOSTKEYS
265
 
    bzr_hostkey_path = pathjoin(config_dir(), 'ssh_host_keys')
266
 
    ensure_config_dir_exists()
267
 
 
268
 
    try:
269
 
        f = open(bzr_hostkey_path, 'w')
270
 
        f.write('# SSH host keys collected by bzr\n')
271
 
        for hostname, keys in BZR_HOSTKEYS.iteritems():
272
 
            for keytype, key in keys.iteritems():
273
 
                f.write('%s %s %s\n' % (hostname, keytype, key.get_base64()))
274
 
        f.close()
275
 
    except IOError, e:
276
 
        mutter('failed to save bzr host keys: ' + str(e))
277
 
 
278
96
 
279
97
class SFTPLock(object):
280
 
    """This fakes a lock in a remote location."""
 
98
    """This fakes a lock in a remote location.
 
99
 
 
100
    A present lock is indicated just by the existence of a file.  This
 
101
    doesn't work well on all transports and they are only used in
 
102
    deprecated storage formats.
 
103
    """
 
104
 
281
105
    __slots__ = ['path', 'lock_path', 'lock_file', 'transport']
 
106
 
282
107
    def __init__(self, path, transport):
283
 
        assert isinstance(transport, SFTPTransport)
284
 
 
285
108
        self.lock_file = None
286
109
        self.path = path
287
110
        self.lock_path = path + '.write-lock'
311
134
            pass
312
135
 
313
136
 
314
 
class SFTPTransport (Transport):
315
 
    """Transport implementation for SFTP access"""
 
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):
 
326
    """Transport implementation for SFTP access."""
316
327
 
317
328
    _do_prefetch = _default_do_prefetch
318
329
    # TODO: jam 20060717 Conceivably these could be configurable, either
332
343
    # up the request itself, rather than us having to worry about it
333
344
    _max_request_size = 32768
334
345
 
335
 
    def __init__(self, base, clone_from=None):
336
 
        assert base.startswith('sftp://')
337
 
        self._parse_url(base)
338
 
        base = self._unparse_url()
339
 
        if base[-1] != '/':
340
 
            base += '/'
341
 
        super(SFTPTransport, self).__init__(base)
342
 
        if clone_from is None:
343
 
            self._sftp_connect()
344
 
        else:
345
 
            # use the same ssh connection, etc
346
 
            self._sftp = clone_from._sftp
347
 
        # super saves 'self.base'
348
 
    
349
 
    def should_cache(self):
350
 
        """
351
 
        Return True if the data pulled across should be cached locally.
352
 
        """
353
 
        return True
354
 
 
355
 
    def clone(self, offset=None):
356
 
        """
357
 
        Return a new SFTPTransport with root at self.base + offset.
358
 
        We share the same SFTP session between such transports, because it's
359
 
        fairly expensive to set them up.
360
 
        """
361
 
        if offset is None:
362
 
            return SFTPTransport(self.base, self)
363
 
        else:
364
 
            return SFTPTransport(self.abspath(offset), self)
365
 
 
366
 
    def abspath(self, relpath):
367
 
        """
368
 
        Return the full url to the given relative path.
369
 
        
370
 
        @param relpath: the relative path or path components
371
 
        @type relpath: str or list
372
 
        """
373
 
        return self._unparse_url(self._remote_path(relpath))
374
 
    
 
346
    def __init__(self, base, _from_transport=None):
 
347
        super(SFTPTransport, self).__init__(base,
 
348
                                            _from_transport=_from_transport)
 
349
 
375
350
    def _remote_path(self, relpath):
376
351
        """Return the path to be passed along the sftp protocol for relpath.
377
 
        
378
 
        relpath is a urlencoded string.
379
 
        """
380
 
        # FIXME: share the common code across transports
381
 
        assert isinstance(relpath, basestring)
382
 
        relpath = urlutils.unescape(relpath).split('/')
383
 
        basepath = self._path.split('/')
384
 
        if len(basepath) > 0 and basepath[-1] == '':
385
 
            basepath = basepath[:-1]
386
 
 
387
 
        for p in relpath:
388
 
            if p == '..':
389
 
                if len(basepath) == 0:
390
 
                    # In most filesystems, a request for the parent
391
 
                    # of root, just returns root.
392
 
                    continue
393
 
                basepath.pop()
394
 
            elif p == '.':
395
 
                continue # No-op
396
 
            else:
397
 
                basepath.append(p)
398
 
 
399
 
        path = '/'.join(basepath)
400
 
        # mutter('relpath => remotepath %s => %s', relpath, path)
401
 
        return path
402
 
 
403
 
    def relpath(self, abspath):
404
 
        username, password, host, port, path = self._split_url(abspath)
405
 
        error = []
406
 
        if (username != self._username):
407
 
            error.append('username mismatch')
408
 
        if (host != self._host):
409
 
            error.append('host mismatch')
410
 
        if (port != self._port):
411
 
            error.append('port mismatch')
412
 
        if (not path.startswith(self._path)):
413
 
            error.append('path mismatch')
414
 
        if error:
415
 
            extra = ': ' + ', '.join(error)
416
 
            raise PathNotChild(abspath, self.base, extra=extra)
417
 
        pl = len(self._path)
418
 
        return path[pl:].strip('/')
 
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
 
382
        else:
 
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
419
402
 
420
403
    def has(self, relpath):
421
404
        """
422
405
        Does the target location exist?
423
406
        """
424
407
        try:
425
 
            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')
426
411
            return True
427
412
        except IOError:
428
413
            return False
429
414
 
430
415
    def get(self, relpath):
431
 
        """
432
 
        Get the file at the given relative path.
 
416
        """Get the file at the given relative path.
433
417
 
434
418
        :param relpath: The relative path to the file
435
419
        """
436
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
437
426
            path = self._remote_path(relpath)
438
 
            f = self._sftp.file(path, mode='rb')
 
427
            f = self._get_sftp().file(path, mode='rb')
439
428
            if self._do_prefetch and (getattr(f, 'prefetch', None) is not None):
440
429
                f.prefetch()
441
430
            return f
442
431
        except (IOError, paramiko.SSHException), e:
443
 
            self._translate_io_exception(e, path, ': error retrieving')
444
 
 
445
 
    def readv(self, relpath, offsets):
 
432
            self._translate_io_exception(e, path, ': error retrieving',
 
433
                failure_exc=errors.ReadError)
 
434
 
 
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):
446
446
        """See Transport.readv()"""
447
447
        # We overload the default readv() because we want to use a file
448
448
        # that does not have prefetch enabled.
452
452
 
453
453
        try:
454
454
            path = self._remote_path(relpath)
455
 
            fp = self._sftp.file(path, mode='rb')
 
455
            fp = self._get_sftp().file(path, mode='rb')
456
456
            readv = getattr(fp, 'readv', None)
457
457
            if readv:
458
 
                return self._sftp_readv(fp, offsets)
459
 
            mutter('seek and read %s offsets', len(offsets))
460
 
            return self._seek_and_read(fp, offsets)
 
458
                return self._sftp_readv(fp, offsets, relpath)
 
459
            if 'sftp' in debug.debug_flags:
 
460
                mutter('seek and read %s offsets', len(offsets))
 
461
            return self._seek_and_read(fp, offsets, relpath)
461
462
        except (IOError, paramiko.SSHException), e:
462
463
            self._translate_io_exception(e, path, ': error retrieving')
463
464
 
464
 
    def _sftp_readv(self, fp, offsets):
 
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):
465
474
        """Use the readv() member of fp to do async readv.
466
475
 
467
 
        And then read them using paramiko.readv(). paramiko.readv()
 
476
        Then read them using paramiko.readv(). paramiko.readv()
468
477
        does not support ranges > 64K, so it caps the request size, and
469
 
        just reads until it gets all the stuff it wants
470
 
        """
471
 
        offsets = list(offsets)
472
 
        sorted_offsets = sorted(offsets)
473
 
 
474
 
        # The algorithm works as follows:
475
 
        # 1) Coalesce nearby reads into a single chunk
476
 
        #    This generates a list of combined regions, the total size
477
 
        #    and the size of the sub regions. This coalescing step is limited
478
 
        #    in the number of nearby chunks to combine, and is allowed to
479
 
        #    skip small breaks in the requests. Limiting it makes sure that
480
 
        #    we can start yielding some data earlier, and skipping means we
481
 
        #    make fewer requests. (Beneficial even when using async)
482
 
        # 2) Break up this combined regions into chunks that are smaller
483
 
        #    than 64KiB. Technically the limit is 65536, but we are a
484
 
        #    little bit conservative. This is because sftp has a maximum
485
 
        #    return chunk size of 64KiB (max size of an unsigned short)
486
 
        # 3) Issue a readv() to paramiko to create an async request for
487
 
        #    all of this data
488
 
        # 4) Read in the data as it comes back, until we've read one
489
 
        #    continuous section as determined in step 1
490
 
        # 5) Break up the full sections into hunks for the original requested
491
 
        #    offsets. And put them in a cache
492
 
        # 6) Check if the next request is in the cache, and if it is, remove
493
 
        #    it from the cache, and yield its data. Continue until no more
494
 
        #    entries are in the cache.
495
 
        # 7) loop back to step 4 until all data has been read
496
 
        #
497
 
        # TODO: jam 20060725 This could be optimized one step further, by
498
 
        #       attempting to yield whatever data we have read, even before
499
 
        #       the first coallesced section has been fully processed.
500
 
 
501
 
        # When coalescing for use with readv(), we don't really need to
502
 
        # use any fudge factor, because the requests are made asynchronously
503
 
        coalesced = list(self._coalesce_offsets(sorted_offsets,
504
 
                               limit=self._max_readv_combine,
505
 
                               fudge_factor=0,
506
 
                               ))
507
 
        requests = []
508
 
        for c_offset in coalesced:
509
 
            start = c_offset.start
510
 
            size = c_offset.length
511
 
 
512
 
            # We need to break this up into multiple requests
513
 
            while size > 0:
514
 
                next_size = min(size, self._max_request_size)
515
 
                requests.append((start, next_size))
516
 
                size -= next_size
517
 
                start += next_size
518
 
 
519
 
        mutter('SFTP.readv() %s offsets => %s coalesced => %s requests',
520
 
                len(offsets), len(coalesced), len(requests))
521
 
 
522
 
        # Queue the current read until we have read the full coalesced section
523
 
        cur_data = []
524
 
        cur_data_len = 0
525
 
        cur_coalesced_stack = iter(coalesced)
526
 
        cur_coalesced = cur_coalesced_stack.next()
527
 
 
528
 
        # Cache the results, but only until they have been fulfilled
529
 
        data_map = {}
530
 
        # turn the list of offsets into a stack
531
 
        offset_stack = iter(offsets)
532
 
        cur_offset_and_size = offset_stack.next()
533
 
 
534
 
        for data in fp.readv(requests):
535
 
            cur_data += data
536
 
            cur_data_len += len(data)
537
 
 
538
 
            if cur_data_len < cur_coalesced.length:
539
 
                continue
540
 
            assert cur_data_len == cur_coalesced.length, \
541
 
                "Somehow we read too much: %s != %s" % (cur_data_len,
542
 
                                                        cur_coalesced.length)
543
 
            all_data = ''.join(cur_data)
544
 
            cur_data = []
545
 
            cur_data_len = 0
546
 
 
547
 
            for suboffset, subsize in cur_coalesced.ranges:
548
 
                key = (cur_coalesced.start+suboffset, subsize)
549
 
                data_map[key] = all_data[suboffset:suboffset+subsize]
550
 
 
551
 
            # Now that we've read some data, see if we can yield anything back
552
 
            while cur_offset_and_size in data_map:
553
 
                this_data = data_map.pop(cur_offset_and_size)
554
 
                yield cur_offset_and_size[0], this_data
555
 
                cur_offset_and_size = offset_stack.next()
556
 
 
557
 
            # Now that we've read all of the data for this coalesced section
558
 
            # on to the next
559
 
            cur_coalesced = cur_coalesced_stack.next()
560
 
 
561
 
    def put(self, relpath, f, mode=None):
562
 
        """
563
 
        Copy the file-like or string object into the location.
 
478
        just reads until it gets all the stuff it wants.
 
479
        """
 
480
        helper = _SFTPReadvHelper(offsets, relpath, self._report_activity)
 
481
        return helper.request_and_yield_offsets(fp)
 
482
 
 
483
    def put_file(self, relpath, f, mode=None):
 
484
        """
 
485
        Copy the file-like object into the location.
564
486
 
565
487
        :param relpath: Location to put the contents, relative to base.
566
 
        :param f:       File-like or string object.
 
488
        :param f:       File-like object.
567
489
        :param mode: The final mode for the file
568
490
        """
569
491
        final_path = self._remote_path(relpath)
570
 
        self._put(final_path, f, mode=mode)
 
492
        return self._put(final_path, f, mode=mode)
571
493
 
572
494
    def _put(self, abspath, f, mode=None):
573
495
        """Helper function so both put() and copy_abspaths can reuse the code"""
578
500
        try:
579
501
            try:
580
502
                fout.set_pipelined(True)
581
 
                self._pump(f, fout)
 
503
                length = self._pump(f, fout)
582
504
            except (IOError, paramiko.SSHException), e:
583
505
                self._translate_io_exception(e, tmp_abspath)
 
506
            # XXX: This doesn't truly help like we would like it to.
 
507
            #      The problem is that openssh strips sticky bits. So while we
 
508
            #      can properly set group write permission, we lose the group
 
509
            #      sticky bit. So it is probably best to stop chmodding, and
 
510
            #      just tell users that they need to set the umask correctly.
 
511
            #      The attr.st_mode = mode, in _sftp_open_exclusive
 
512
            #      will handle when the user wants the final mode to be more
 
513
            #      restrictive. And then we avoid a round trip. Unless
 
514
            #      paramiko decides to expose an async chmod()
 
515
 
 
516
            # This is designed to chmod() right before we close.
 
517
            # Because we set_pipelined() earlier, theoretically we might
 
518
            # avoid the round trip for fout.close()
584
519
            if mode is not None:
585
 
                self._sftp.chmod(tmp_abspath, mode)
 
520
                self._get_sftp().chmod(tmp_abspath, mode)
586
521
            fout.close()
587
522
            closed = True
588
523
            self._rename_and_overwrite(tmp_abspath, abspath)
 
524
            return length
589
525
        except Exception, e:
590
526
            # If we fail, try to clean up the temporary file
591
527
            # before we throw the exception
597
533
            try:
598
534
                if not closed:
599
535
                    fout.close()
600
 
                self._sftp.remove(tmp_abspath)
 
536
                self._get_sftp().remove(tmp_abspath)
601
537
            except:
602
538
                # raise the saved except
603
539
                raise e
604
540
            # raise the original with its traceback if we can.
605
541
            raise
606
542
 
 
543
    def _put_non_atomic_helper(self, relpath, writer, mode=None,
 
544
                               create_parent_dir=False,
 
545
                               dir_mode=None):
 
546
        abspath = self._remote_path(relpath)
 
547
 
 
548
        # TODO: jam 20060816 paramiko doesn't publicly expose a way to
 
549
        #       set the file mode at create time. If it does, use it.
 
550
        #       But for now, we just chmod later anyway.
 
551
 
 
552
        def _open_and_write_file():
 
553
            """Try to open the target file, raise error on failure"""
 
554
            fout = None
 
555
            try:
 
556
                try:
 
557
                    fout = self._get_sftp().file(abspath, mode='wb')
 
558
                    fout.set_pipelined(True)
 
559
                    writer(fout)
 
560
                except (paramiko.SSHException, IOError), e:
 
561
                    self._translate_io_exception(e, abspath,
 
562
                                                 ': unable to open')
 
563
 
 
564
                # This is designed to chmod() right before we close.
 
565
                # Because we set_pipelined() earlier, theoretically we might
 
566
                # avoid the round trip for fout.close()
 
567
                if mode is not None:
 
568
                    self._get_sftp().chmod(abspath, mode)
 
569
            finally:
 
570
                if fout is not None:
 
571
                    fout.close()
 
572
 
 
573
        if not create_parent_dir:
 
574
            _open_and_write_file()
 
575
            return
 
576
 
 
577
        # Try error handling to create the parent directory if we need to
 
578
        try:
 
579
            _open_and_write_file()
 
580
        except NoSuchFile:
 
581
            # Try to create the parent directory, and then go back to
 
582
            # writing the file
 
583
            parent_dir = os.path.dirname(abspath)
 
584
            self._mkdir(parent_dir, dir_mode)
 
585
            _open_and_write_file()
 
586
 
 
587
    def put_file_non_atomic(self, relpath, f, mode=None,
 
588
                            create_parent_dir=False,
 
589
                            dir_mode=None):
 
590
        """Copy the file-like object into the target location.
 
591
 
 
592
        This function is not strictly safe to use. It is only meant to
 
593
        be used when you already know that the target does not exist.
 
594
        It is not safe, because it will open and truncate the remote
 
595
        file. So there may be a time when the file has invalid contents.
 
596
 
 
597
        :param relpath: The remote location to put the contents.
 
598
        :param f:       File-like object.
 
599
        :param mode:    Possible access permissions for new file.
 
600
                        None means do not set remote permissions.
 
601
        :param create_parent_dir: If we cannot create the target file because
 
602
                        the parent directory does not exist, go ahead and
 
603
                        create it, and then try again.
 
604
        """
 
605
        def writer(fout):
 
606
            self._pump(f, fout)
 
607
        self._put_non_atomic_helper(relpath, writer, mode=mode,
 
608
                                    create_parent_dir=create_parent_dir,
 
609
                                    dir_mode=dir_mode)
 
610
 
 
611
    def put_bytes_non_atomic(self, relpath, bytes, mode=None,
 
612
                             create_parent_dir=False,
 
613
                             dir_mode=None):
 
614
        def writer(fout):
 
615
            fout.write(bytes)
 
616
        self._put_non_atomic_helper(relpath, writer, mode=mode,
 
617
                                    create_parent_dir=create_parent_dir,
 
618
                                    dir_mode=dir_mode)
 
619
 
607
620
    def iter_files_recursive(self):
608
621
        """Walk the relative paths of all files in this transport."""
 
622
        # progress is handled by list_dir
609
623
        queue = list(self.list_dir('.'))
610
624
        while queue:
611
 
            relpath = urllib.quote(queue.pop(0))
 
625
            relpath = queue.pop(0)
612
626
            st = self.stat(relpath)
613
627
            if stat.S_ISDIR(st.st_mode):
614
628
                for i, basename in enumerate(self.list_dir(relpath)):
616
630
            else:
617
631
                yield relpath
618
632
 
 
633
    def _mkdir(self, abspath, mode=None):
 
634
        if mode is None:
 
635
            local_mode = 0777
 
636
        else:
 
637
            local_mode = mode
 
638
        try:
 
639
            self._report_activity(len(abspath), 'write')
 
640
            self._get_sftp().mkdir(abspath, local_mode)
 
641
            self._report_activity(1, 'read')
 
642
            if mode is not None:
 
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)
 
658
        except (paramiko.SSHException, IOError), e:
 
659
            self._translate_io_exception(e, abspath, ': unable to mkdir',
 
660
                failure_exc=FileExists)
 
661
 
619
662
    def mkdir(self, relpath, mode=None):
620
663
        """Create a directory at the given path."""
621
 
        path = self._remote_path(relpath)
 
664
        self._mkdir(self._remote_path(relpath), mode=mode)
 
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
622
679
        try:
623
 
            # In the paramiko documentation, it says that passing a mode flag 
624
 
            # will filtered against the server umask.
625
 
            # StubSFTPServer does not do this, which would be nice, because it is
626
 
            # what we really want :)
627
 
            # However, real servers do use umask, so we really should do it that way
628
 
            self._sftp.mkdir(path)
629
 
            if mode is not None:
630
 
                self._sftp.chmod(path, mode=mode)
 
680
            handle = self._get_sftp().file(abspath, mode='wb')
 
681
            handle.set_pipelined(True)
631
682
        except (paramiko.SSHException, IOError), e:
632
 
            self._translate_io_exception(e, path, ': unable to mkdir',
633
 
                failure_exc=FileExists)
 
683
            self._translate_io_exception(e, abspath,
 
684
                                         ': unable to open')
 
685
        _file_streams[self.abspath(relpath)] = handle
 
686
        return FileFileStream(self, relpath, handle)
634
687
 
635
 
    def _translate_io_exception(self, e, path, more_info='', 
 
688
    def _translate_io_exception(self, e, path, more_info='',
636
689
                                failure_exc=PathError):
637
690
        """Translate a paramiko or IOError into a friendlier exception.
638
691
 
643
696
        :param failure_exc: Paramiko has the super fun ability to raise completely
644
697
                           opaque errors that just set "e.args = ('Failure',)" with
645
698
                           no more information.
646
 
                           If this parameter is set, it defines the exception 
 
699
                           If this parameter is set, it defines the exception
647
700
                           to raise in these cases.
648
701
        """
649
702
        # paramiko seems to generate detailless errors.
650
703
        self._translate_error(e, path, raise_generic=False)
651
 
        if hasattr(e, 'args'):
 
704
        if getattr(e, 'args', None) is not None:
652
705
            if (e.args == ('No such file or directory',) or
653
706
                e.args == ('No such file',)):
654
707
                raise NoSuchFile(path, str(e) + more_info)
655
 
            if (e.args == ('mkdir failed',)):
 
708
            if (e.args == ('mkdir failed',) or
 
709
                e.args[0].startswith('syserr: File exists')):
656
710
                raise FileExists(path, str(e) + more_info)
657
711
            # strange but true, for the paramiko server.
658
712
            if (e.args == ('Failure',)):
659
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))
660
720
            mutter('Raising exception with args %s', e.args)
661
 
        if hasattr(e, 'errno'):
 
721
        if getattr(e, 'errno', None) is not None:
662
722
            mutter('Raising exception with errno %s', e.errno)
663
723
        raise e
664
724
 
665
 
    def append(self, relpath, f, mode=None):
 
725
    def append_file(self, relpath, f, mode=None):
666
726
        """
667
727
        Append the text in the file-like object into the final
668
728
        location.
669
729
        """
670
730
        try:
671
731
            path = self._remote_path(relpath)
672
 
            fout = self._sftp.file(path, 'ab')
 
732
            fout = self._get_sftp().file(path, 'ab')
673
733
            if mode is not None:
674
 
                self._sftp.chmod(path, mode)
 
734
                self._get_sftp().chmod(path, mode)
675
735
            result = fout.tell()
676
736
            self._pump(f, fout)
677
737
            return result
681
741
    def rename(self, rel_from, rel_to):
682
742
        """Rename without special overwriting"""
683
743
        try:
684
 
            self._sftp.rename(self._remote_path(rel_from),
 
744
            self._get_sftp().rename(self._remote_path(rel_from),
685
745
                              self._remote_path(rel_to))
686
746
        except (IOError, paramiko.SSHException), e:
687
747
            self._translate_io_exception(e, rel_from,
689
749
 
690
750
    def _rename_and_overwrite(self, abs_from, abs_to):
691
751
        """Do a fancy rename on the remote server.
692
 
        
 
752
 
693
753
        Using the implementation provided by osutils.
694
754
        """
695
755
        try:
 
756
            sftp = self._get_sftp()
696
757
            fancy_rename(abs_from, abs_to,
697
 
                    rename_func=self._sftp.rename,
698
 
                    unlink_func=self._sftp.remove)
 
758
                         rename_func=sftp.rename,
 
759
                         unlink_func=sftp.remove)
699
760
        except (IOError, paramiko.SSHException), e:
700
 
            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))
701
763
 
702
764
    def move(self, rel_from, rel_to):
703
765
        """Move the item at rel_from to the location at rel_to"""
709
771
        """Delete the item at relpath"""
710
772
        path = self._remote_path(relpath)
711
773
        try:
712
 
            self._sftp.remove(path)
 
774
            self._get_sftp().remove(path)
713
775
        except (IOError, paramiko.SSHException), e:
714
776
            self._translate_io_exception(e, path, ': unable to delete')
715
 
            
 
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
 
716
783
    def listable(self):
717
784
        """Return True if this store supports listing."""
718
785
        return True
722
789
        Return a list of all files at the given location.
723
790
        """
724
791
        # does anything actually use this?
 
792
        # -- Unknown
 
793
        # This is at least used by copy_tree for remote upgrades.
 
794
        # -- David Allouche 2006-08-11
725
795
        path = self._remote_path(relpath)
726
796
        try:
727
 
            return self._sftp.listdir(path)
 
797
            entries = self._get_sftp().listdir(path)
 
798
            self._report_activity(sum(map(len, entries)), 'read')
728
799
        except (IOError, paramiko.SSHException), e:
729
800
            self._translate_io_exception(e, path, ': failed to list_dir')
 
801
        return [urlutils.escape(entry) for entry in entries]
730
802
 
731
803
    def rmdir(self, relpath):
732
804
        """See Transport.rmdir."""
733
805
        path = self._remote_path(relpath)
734
806
        try:
735
 
            return self._sftp.rmdir(path)
 
807
            return self._get_sftp().rmdir(path)
736
808
        except (IOError, paramiko.SSHException), e:
737
809
            self._translate_io_exception(e, path, ': failed to rmdir')
738
810
 
740
812
        """Return the stat information for a file."""
741
813
        path = self._remote_path(relpath)
742
814
        try:
743
 
            return self._sftp.stat(path)
 
815
            return self._get_sftp().stat(path)
744
816
        except (IOError, paramiko.SSHException), e:
745
817
            self._translate_io_exception(e, path, ': unable to stat')
746
818
 
770
842
        # that we have taken the lock.
771
843
        return SFTPLock(relpath, self)
772
844
 
773
 
    def _unparse_url(self, path=None):
774
 
        if path is None:
775
 
            path = self._path
776
 
        path = urllib.quote(path)
777
 
        # handle homedir paths
778
 
        if not path.startswith('/'):
779
 
            path = "/~/" + path
780
 
        netloc = urllib.quote(self._host)
781
 
        if self._username is not None:
782
 
            netloc = '%s@%s' % (urllib.quote(self._username), netloc)
783
 
        if self._port is not None:
784
 
            netloc = '%s:%d' % (netloc, self._port)
785
 
        return urlparse.urlunparse(('sftp', netloc, path, '', '', ''))
786
 
 
787
 
    def _split_url(self, url):
788
 
        (scheme, username, password, host, port, path) = split_url(url)
789
 
        assert scheme == 'sftp'
790
 
 
791
 
        # the initial slash should be removed from the path, and treated
792
 
        # as a homedir relative path (the path begins with a double slash
793
 
        # if it is absolute).
794
 
        # see draft-ietf-secsh-scp-sftp-ssh-uri-03.txt
795
 
        # RBC 20060118 we are not using this as its too user hostile. instead
796
 
        # we are following lftp and using /~/foo to mean '~/foo'.
797
 
        # handle homedir paths
798
 
        if path.startswith('/~/'):
799
 
            path = path[3:]
800
 
        elif path == '/~':
801
 
            path = ''
802
 
        return (username, password, host, port, path)
803
 
 
804
 
    def _parse_url(self, url):
805
 
        (self._username, self._password,
806
 
         self._host, self._port, self._path) = self._split_url(url)
807
 
 
808
 
    def _sftp_connect(self):
809
 
        """Connect to the remote sftp server.
810
 
        After this, self._sftp should have a valid connection (or
811
 
        we raise an TransportError 'could not connect').
812
 
 
813
 
        TODO: Raise a more reasonable ConnectionFailed exception
814
 
        """
815
 
        global _connected_hosts
816
 
 
817
 
        idx = (self._host, self._port, self._username)
818
 
        try:
819
 
            self._sftp = _connected_hosts[idx]
820
 
            return
821
 
        except KeyError:
822
 
            pass
823
 
        
824
 
        vendor = _get_ssh_vendor()
825
 
        if vendor == 'loopback':
826
 
            sock = socket.socket()
827
 
            try:
828
 
                sock.connect((self._host, self._port))
829
 
            except socket.error, e:
830
 
                raise ConnectionError('Unable to connect to SSH host %s:%s: %s'
831
 
                                      % (self._host, self._port, e))
832
 
            self._sftp = SFTPClient(LoopbackSFTP(sock))
833
 
        elif vendor != 'none':
834
 
            try:
835
 
                sock = SFTPSubprocess(self._host, vendor, self._port,
836
 
                                      self._username)
837
 
                self._sftp = SFTPClient(sock)
838
 
            except (EOFError, paramiko.SSHException), e:
839
 
                raise ConnectionError('Unable to connect to SSH host %s:%s: %s'
840
 
                                      % (self._host, self._port, e))
841
 
            except (OSError, IOError), e:
842
 
                # If the machine is fast enough, ssh can actually exit
843
 
                # before we try and send it the sftp request, which
844
 
                # raises a Broken Pipe
845
 
                if e.errno not in (errno.EPIPE,):
846
 
                    raise
847
 
                raise ConnectionError('Unable to connect to SSH host %s:%s: %s'
848
 
                                      % (self._host, self._port, e))
849
 
        else:
850
 
            self._paramiko_connect()
851
 
 
852
 
        _connected_hosts[idx] = self._sftp
853
 
 
854
 
    def _paramiko_connect(self):
855
 
        global SYSTEM_HOSTKEYS, BZR_HOSTKEYS
856
 
        
857
 
        load_host_keys()
858
 
 
859
 
        try:
860
 
            t = paramiko.Transport((self._host, self._port or 22))
861
 
            t.set_log_channel('bzr.paramiko')
862
 
            t.start_client()
863
 
        except (paramiko.SSHException, socket.error), e:
864
 
            raise ConnectionError('Unable to reach SSH host %s:%s: %s' 
865
 
                                  % (self._host, self._port, e))
866
 
            
867
 
        server_key = t.get_remote_server_key()
868
 
        server_key_hex = paramiko.util.hexify(server_key.get_fingerprint())
869
 
        keytype = server_key.get_name()
870
 
        if SYSTEM_HOSTKEYS.has_key(self._host) and SYSTEM_HOSTKEYS[self._host].has_key(keytype):
871
 
            our_server_key = SYSTEM_HOSTKEYS[self._host][keytype]
872
 
            our_server_key_hex = paramiko.util.hexify(our_server_key.get_fingerprint())
873
 
        elif BZR_HOSTKEYS.has_key(self._host) and BZR_HOSTKEYS[self._host].has_key(keytype):
874
 
            our_server_key = BZR_HOSTKEYS[self._host][keytype]
875
 
            our_server_key_hex = paramiko.util.hexify(our_server_key.get_fingerprint())
876
 
        else:
877
 
            warning('Adding %s host key for %s: %s' % (keytype, self._host, server_key_hex))
878
 
            if not BZR_HOSTKEYS.has_key(self._host):
879
 
                BZR_HOSTKEYS[self._host] = {}
880
 
            BZR_HOSTKEYS[self._host][keytype] = server_key
881
 
            our_server_key = server_key
882
 
            our_server_key_hex = paramiko.util.hexify(our_server_key.get_fingerprint())
883
 
            save_host_keys()
884
 
        if server_key != our_server_key:
885
 
            filename1 = os.path.expanduser('~/.ssh/known_hosts')
886
 
            filename2 = pathjoin(config_dir(), 'ssh_host_keys')
887
 
            raise TransportError('Host keys for %s do not match!  %s != %s' % \
888
 
                (self._host, our_server_key_hex, server_key_hex),
889
 
                ['Try editing %s or %s' % (filename1, filename2)])
890
 
 
891
 
        self._sftp_auth(t)
892
 
        
893
 
        try:
894
 
            self._sftp = t.open_sftp_client()
895
 
        except paramiko.SSHException, e:
896
 
            raise ConnectionError('Unable to start sftp client %s:%d' %
897
 
                                  (self._host, self._port), e)
898
 
 
899
 
    def _sftp_auth(self, transport):
900
 
        # paramiko requires a username, but it might be none if nothing was supplied
901
 
        # use the local username, just in case.
902
 
        # We don't override self._username, because if we aren't using paramiko,
903
 
        # the username might be specified in ~/.ssh/config and we don't want to
904
 
        # force it to something else
905
 
        # Also, it would mess up the self.relpath() functionality
906
 
        username = self._username or getpass.getuser()
907
 
 
908
 
        if _use_ssh_agent:
909
 
            agent = paramiko.Agent()
910
 
            for key in agent.get_keys():
911
 
                mutter('Trying SSH agent key %s' % paramiko.util.hexify(key.get_fingerprint()))
912
 
                try:
913
 
                    transport.auth_publickey(username, key)
914
 
                    return
915
 
                except paramiko.SSHException, e:
916
 
                    pass
917
 
        
918
 
        # okay, try finding id_rsa or id_dss?  (posix only)
919
 
        if self._try_pkey_auth(transport, paramiko.RSAKey, username, 'id_rsa'):
920
 
            return
921
 
        if self._try_pkey_auth(transport, paramiko.DSSKey, username, 'id_dsa'):
922
 
            return
923
 
 
924
 
        if self._password:
925
 
            try:
926
 
                transport.auth_password(username, self._password)
927
 
                return
928
 
            except paramiko.SSHException, e:
929
 
                pass
930
 
 
931
 
            # FIXME: Don't keep a password held in memory if you can help it
932
 
            #self._password = None
933
 
 
934
 
        # give up and ask for a password
935
 
        password = bzrlib.ui.ui_factory.get_password(
936
 
                prompt='SSH %(user)s@%(host)s password',
937
 
                user=username, host=self._host)
938
 
        try:
939
 
            transport.auth_password(username, password)
940
 
        except paramiko.SSHException, e:
941
 
            raise ConnectionError('Unable to authenticate to SSH host as %s@%s' %
942
 
                                  (username, self._host), e)
943
 
 
944
 
    def _try_pkey_auth(self, transport, pkey_class, username, filename):
945
 
        filename = os.path.expanduser('~/.ssh/' + filename)
946
 
        try:
947
 
            key = pkey_class.from_private_key_file(filename)
948
 
            transport.auth_publickey(username, key)
949
 
            return True
950
 
        except paramiko.PasswordRequiredException:
951
 
            password = bzrlib.ui.ui_factory.get_password(
952
 
                    prompt='SSH %(filename)s password',
953
 
                    filename=filename)
954
 
            try:
955
 
                key = pkey_class.from_private_key_file(filename, password)
956
 
                transport.auth_publickey(username, key)
957
 
                return True
958
 
            except paramiko.SSHException:
959
 
                mutter('SSH authentication via %s key failed.' % (os.path.basename(filename),))
960
 
        except paramiko.SSHException:
961
 
            mutter('SSH authentication via %s key failed.' % (os.path.basename(filename),))
962
 
        except IOError:
963
 
            pass
964
 
        return False
965
 
 
966
845
    def _sftp_open_exclusive(self, abspath, mode=None):
967
846
        """Open a remote path exclusively.
968
847
 
977
856
        :param abspath: The remote absolute path where the file should be opened
978
857
        :param mode: The mode permissions bits for the new file
979
858
        """
980
 
        path = self._sftp._adjust_cwd(abspath)
 
859
        # TODO: jam 20060816 Paramiko >= 1.6.2 (probably earlier) supports
 
860
        #       using the 'x' flag to indicate SFTP_FLAG_EXCL.
 
861
        #       However, there is no way to set the permission mode at open
 
862
        #       time using the sftp_client.file() functionality.
 
863
        path = self._get_sftp()._adjust_cwd(abspath)
981
864
        # mutter('sftp abspath %s => %s', abspath, path)
982
865
        attr = SFTPAttributes()
983
866
        if mode is not None:
984
867
            attr.st_mode = mode
985
 
        omode = (SFTP_FLAG_WRITE | SFTP_FLAG_CREATE 
 
868
        omode = (SFTP_FLAG_WRITE | SFTP_FLAG_CREATE
986
869
                | SFTP_FLAG_TRUNC | SFTP_FLAG_EXCL)
987
870
        try:
988
 
            t, msg = self._sftp._request(CMD_OPEN, path, omode, attr)
 
871
            t, msg = self._get_sftp()._request(CMD_OPEN, path, omode, attr)
989
872
            if t != CMD_HANDLE:
990
873
                raise TransportError('Expected an SFTP handle')
991
874
            handle = msg.get_string()
992
 
            return SFTPFile(self._sftp, handle, 'wb', -1)
 
875
            return SFTPFile(self._get_sftp(), handle, 'wb', -1)
993
876
        except (paramiko.SSHException, IOError), e:
994
877
            self._translate_io_exception(e, abspath, ': unable to open',
995
878
                failure_exc=FileExists)
996
879
 
 
880
    def _can_roundtrip_unix_modebits(self):
 
881
        if sys.platform == 'win32':
 
882
            # anyone else?
 
883
            return False
 
884
        else:
 
885
            return True
997
886
 
998
887
# ------------- server test implementation --------------
999
 
import socket
1000
888
import threading
1001
889
 
1002
890
from bzrlib.tests.stub_sftp import StubServer, StubSFTPServer
1061
949
                # probably a failed test; unit test thread will log the
1062
950
                # failure/error
1063
951
                sys.excepthook(*sys.exc_info())
1064
 
                warning('Exception from within unit test server thread: %r' % 
 
952
                warning('Exception from within unit test server thread: %r' %
1065
953
                        x)
1066
954
 
1067
955
 
 
956
class SocketDelay(object):
 
957
    """A socket decorator to make TCP appear slower.
 
958
 
 
959
    This changes recv, send, and sendall to add a fixed latency to each python
 
960
    call if a new roundtrip is detected. That is, when a recv is called and the
 
961
    flag new_roundtrip is set, latency is charged. Every send and send_all
 
962
    sets this flag.
 
963
 
 
964
    In addition every send, sendall and recv sleeps a bit per character send to
 
965
    simulate bandwidth.
 
966
 
 
967
    Not all methods are implemented, this is deliberate as this class is not a
 
968
    replacement for the builtin sockets layer. fileno is not implemented to
 
969
    prevent the proxy being bypassed.
 
970
    """
 
971
 
 
972
    simulated_time = 0
 
973
    _proxied_arguments = dict.fromkeys([
 
974
        "close", "getpeername", "getsockname", "getsockopt", "gettimeout",
 
975
        "setblocking", "setsockopt", "settimeout", "shutdown"])
 
976
 
 
977
    def __init__(self, sock, latency, bandwidth=1.0,
 
978
                 really_sleep=True):
 
979
        """
 
980
        :param bandwith: simulated bandwith (MegaBit)
 
981
        :param really_sleep: If set to false, the SocketDelay will just
 
982
        increase a counter, instead of calling time.sleep. This is useful for
 
983
        unittesting the SocketDelay.
 
984
        """
 
985
        self.sock = sock
 
986
        self.latency = latency
 
987
        self.really_sleep = really_sleep
 
988
        self.time_per_byte = 1 / (bandwidth / 8.0 * 1024 * 1024)
 
989
        self.new_roundtrip = False
 
990
 
 
991
    def sleep(self, s):
 
992
        if self.really_sleep:
 
993
            time.sleep(s)
 
994
        else:
 
995
            SocketDelay.simulated_time += s
 
996
 
 
997
    def __getattr__(self, attr):
 
998
        if attr in SocketDelay._proxied_arguments:
 
999
            return getattr(self.sock, attr)
 
1000
        raise AttributeError("'SocketDelay' object has no attribute %r" %
 
1001
                             attr)
 
1002
 
 
1003
    def dup(self):
 
1004
        return SocketDelay(self.sock.dup(), self.latency, self.time_per_byte,
 
1005
                           self._sleep)
 
1006
 
 
1007
    def recv(self, *args):
 
1008
        data = self.sock.recv(*args)
 
1009
        if data and self.new_roundtrip:
 
1010
            self.new_roundtrip = False
 
1011
            self.sleep(self.latency)
 
1012
        self.sleep(len(data) * self.time_per_byte)
 
1013
        return data
 
1014
 
 
1015
    def sendall(self, data, flags=0):
 
1016
        if not self.new_roundtrip:
 
1017
            self.new_roundtrip = True
 
1018
            self.sleep(self.latency)
 
1019
        self.sleep(len(data) * self.time_per_byte)
 
1020
        return self.sock.sendall(data, flags)
 
1021
 
 
1022
    def send(self, data, flags=0):
 
1023
        if not self.new_roundtrip:
 
1024
            self.new_roundtrip = True
 
1025
            self.sleep(self.latency)
 
1026
        bytes_sent = self.sock.send(data, flags)
 
1027
        self.sleep(bytes_sent * self.time_per_byte)
 
1028
        return bytes_sent
 
1029
 
 
1030
 
1068
1031
class SFTPServer(Server):
1069
1032
    """Common code for SFTP server facilities."""
1070
1033
 
1071
 
    def __init__(self):
 
1034
    def __init__(self, server_interface=StubServer):
1072
1035
        self._original_vendor = None
1073
1036
        self._homedir = None
1074
1037
        self._server_homedir = None
1075
1038
        self._listener = None
1076
1039
        self._root = None
1077
 
        self._vendor = 'none'
 
1040
        self._vendor = ssh.ParamikoVendor()
 
1041
        self._server_interface = server_interface
1078
1042
        # sftp server logs
1079
1043
        self.logs = []
 
1044
        self.add_latency = 0
1080
1045
 
1081
1046
    def _get_sftp_url(self, path):
1082
1047
        """Calculate an sftp url to this server for path."""
1086
1051
        """StubServer uses this to log when a new server is created."""
1087
1052
        self.logs.append(message)
1088
1053
 
 
1054
    def _run_server_entry(self, sock):
 
1055
        """Entry point for all implementations of _run_server.
 
1056
 
 
1057
        If self.add_latency is > 0.000001 then sock is given a latency adding
 
1058
        decorator.
 
1059
        """
 
1060
        if self.add_latency > 0.000001:
 
1061
            sock = SocketDelay(sock, self.add_latency)
 
1062
        return self._run_server(sock)
 
1063
 
1089
1064
    def _run_server(self, s):
1090
1065
        ssh_server = paramiko.Transport(s)
1091
1066
        key_file = pathjoin(self._homedir, 'test_rsa.key')
1094
1069
        f.close()
1095
1070
        host_key = paramiko.RSAKey.from_private_key_file(key_file)
1096
1071
        ssh_server.add_server_key(host_key)
1097
 
        server = StubServer(self)
 
1072
        server = self._server_interface(self)
1098
1073
        ssh_server.set_subsystem_handler('sftp', paramiko.SFTPServer,
1099
1074
                                         StubSFTPServer, root=self._root,
1100
1075
                                         home=self._server_homedir)
1101
1076
        event = threading.Event()
1102
1077
        ssh_server.start_server(event, server)
1103
1078
        event.wait(5.0)
1104
 
    
1105
 
    def setUp(self):
1106
 
        global _ssh_vendor
1107
 
        self._original_vendor = _ssh_vendor
1108
 
        _ssh_vendor = self._vendor
 
1079
 
 
1080
    def setUp(self, backing_server=None):
 
1081
        # XXX: TODO: make sftpserver back onto backing_server rather than local
 
1082
        # disk.
 
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,))
 
1088
        self._original_vendor = ssh._ssh_vendor_manager._cached_ssh_vendor
 
1089
        ssh._ssh_vendor_manager._cached_ssh_vendor = self._vendor
1109
1090
        if sys.platform == 'win32':
1110
1091
            # Win32 needs to use the UNICODE api
1111
1092
            self._homedir = getcwd()
1117
1098
        self._root = '/'
1118
1099
        if sys.platform == 'win32':
1119
1100
            self._root = ''
1120
 
        self._listener = SocketListener(self._run_server)
 
1101
        self._listener = SocketListener(self._run_server_entry)
1121
1102
        self._listener.setDaemon(True)
1122
1103
        self._listener.start()
1123
1104
 
1124
1105
    def tearDown(self):
1125
1106
        """See bzrlib.transport.Server.tearDown."""
1126
 
        global _ssh_vendor
1127
1107
        self._listener.stop()
1128
 
        _ssh_vendor = self._original_vendor
 
1108
        ssh._ssh_vendor_manager._cached_ssh_vendor = self._original_vendor
1129
1109
 
1130
1110
    def get_bogus_url(self):
1131
1111
        """See bzrlib.transport.Server.get_bogus_url."""
1142
1122
 
1143
1123
    def get_url(self):
1144
1124
        """See bzrlib.transport.Server.get_url."""
1145
 
        return self._get_sftp_url(urlutils.escape(self._homedir[1:]))
 
1125
        homedir = self._homedir
 
1126
        if sys.platform != 'win32':
 
1127
            # Remove the initial '/' on all platforms but win32
 
1128
            homedir = homedir[1:]
 
1129
        return self._get_sftp_url(urlutils.escape(homedir))
1146
1130
 
1147
1131
 
1148
1132
class SFTPServerWithoutSSH(SFTPServer):
1150
1134
 
1151
1135
    def __init__(self):
1152
1136
        super(SFTPServerWithoutSSH, self).__init__()
1153
 
        self._vendor = 'loopback'
 
1137
        self._vendor = ssh.LoopbackVendor()
1154
1138
 
1155
1139
    def _run_server(self, sock):
 
1140
        # Re-import these as locals, so that they're still accessible during
 
1141
        # interpreter shutdown (when all module globals get set to None, leading
 
1142
        # to confusing errors like "'NoneType' object has no attribute 'error'".
1156
1143
        class FakeChannel(object):
1157
1144
            def get_transport(self):
1158
1145
                return self
1165
1152
            def close(self):
1166
1153
                pass
1167
1154
 
1168
 
        server = paramiko.SFTPServer(FakeChannel(), 'sftp', StubServer(self), StubSFTPServer,
1169
 
                                     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)
1170
1158
        try:
1171
 
            server.start_subsystem('sftp', None, sock)
 
1159
            server.start_subsystem(
 
1160
                'sftp', None, ssh.SocketAsChannelAdapter(sock))
1172
1161
        except socket.error, e:
1173
1162
            if (len(e.args) > 0) and (e.args[0] == errno.EPIPE):
1174
1163
                # it's okay for the client to disconnect abruptly
1177
1166
            else:
1178
1167
                raise
1179
1168
        except Exception, e:
1180
 
            import sys; sys.stderr.write('\nEXCEPTION %r\n\n' % e.__class__)
 
1169
            # This typically seems to happen during interpreter shutdown, so
 
1170
            # most of the useful ways to report this error are won't work.
 
1171
            # Writing the exception type, and then the text of the exception,
 
1172
            # seems to be the best we can do.
 
1173
            import sys
 
1174
            sys.stderr.write('\nEXCEPTION %r: ' % (e.__class__,))
 
1175
            sys.stderr.write('%s\n\n' % (e,))
1181
1176
        server.finish_subsystem()
1182
1177
 
1183
1178
 
1186
1181
 
1187
1182
    def get_url(self):
1188
1183
        """See bzrlib.transport.Server.get_url."""
1189
 
        if sys.platform == 'win32':
1190
 
            return self._get_sftp_url(urlutils.escape(self._homedir))
1191
 
        else:
1192
 
            return self._get_sftp_url(urlutils.escape(self._homedir[1:]))
 
1184
        homedir = self._homedir
 
1185
        if sys.platform != 'win32':
 
1186
            # Remove the initial '/' on all platforms but win32
 
1187
            homedir = homedir[1:]
 
1188
        return self._get_sftp_url(urlutils.escape(homedir))
1193
1189
 
1194
1190
 
1195
1191
class SFTPHomeDirServer(SFTPServerWithoutSSH):
1201
1197
 
1202
1198
 
1203
1199
class SFTPSiblingAbsoluteServer(SFTPAbsoluteServer):
1204
 
    """A test servere for sftp transports, using absolute urls to non-home."""
1205
 
 
1206
 
    def setUp(self):
 
1200
    """A test server for sftp transports where only absolute paths will work.
 
1201
 
 
1202
    It does this by serving from a deeply-nested directory that doesn't exist.
 
1203
    """
 
1204
 
 
1205
    def setUp(self, backing_server=None):
1207
1206
        self._server_homedir = '/dev/noone/runs/tests/here'
1208
 
        super(SFTPSiblingAbsoluteServer, self).setUp()
 
1207
        super(SFTPSiblingAbsoluteServer, self).setUp(backing_server)
1209
1208
 
1210
1209
 
1211
1210
def get_test_permutations():