~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/sftp.py

  • Committer: Alexander Belchenko
  • Date: 2006-07-30 16:43:12 UTC
  • mto: (1711.2.111 jam-integration)
  • mto: This revision was merged to the branch mainline in revision 1906.
  • Revision ID: bialix@ukr.net-20060730164312-b025fd3ff0cee59e
rename  gpl.txt => COPYING.txt

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006, 2007, 2008, 2009 Canonical Ltd
 
1
# Copyright (C) 2005 Robey Pointer <robey@lag.net>
 
2
# Copyright (C) 2005, 2006 Canonical Ltd
2
3
#
3
4
# This program is free software; you can redistribute it and/or modify
4
5
# it under the terms of the GNU General Public License as published by
12
13
#
13
14
# You should have received a copy of the GNU General Public License
14
15
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
17
 
17
18
"""Implementation of Transport over SFTP, using paramiko."""
18
19
 
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
27
20
import errno
 
21
import getpass
28
22
import itertools
29
23
import os
30
24
import random
 
25
import re
31
26
import select
32
27
import socket
33
28
import stat
 
29
import subprocess
34
30
import sys
35
31
import time
36
32
import urllib
37
33
import urlparse
38
 
import warnings
 
34
import weakref
39
35
 
40
 
from bzrlib import (
41
 
    config,
42
 
    debug,
43
 
    errors,
44
 
    urlutils,
45
 
    )
46
 
from bzrlib.errors import (FileExists,
47
 
                           NoSuchFile, PathNotChild,
 
36
from bzrlib.config import config_dir, ensure_config_dir_exists
 
37
from bzrlib.errors import (ConnectionError,
 
38
                           FileExists, 
 
39
                           TransportNotPossible, NoSuchFile, PathNotChild,
48
40
                           TransportError,
49
 
                           LockError,
 
41
                           LockError, 
50
42
                           PathError,
51
43
                           ParamikoNotPresent,
52
44
                           )
53
45
from bzrlib.osutils import pathjoin, fancy_rename, getcwd
54
 
from bzrlib.symbol_versioning import (
55
 
        deprecated_function,
56
 
        )
57
 
from bzrlib.trace import mutter, warning
 
46
from bzrlib.trace import mutter, warning, error
58
47
from bzrlib.transport import (
59
 
    FileFileStream,
60
 
    _file_streams,
61
 
    local,
 
48
    register_urlparse_netloc_protocol,
62
49
    Server,
63
 
    ssh,
64
 
    ConnectedTransport,
 
50
    split_url,
 
51
    Transport,
65
52
    )
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')
 
53
import bzrlib.ui
 
54
import bzrlib.urlutils as urlutils
79
55
 
80
56
try:
81
57
    import paramiko
87
63
                               CMD_HANDLE, CMD_OPEN)
88
64
    from paramiko.sftp_attr import SFTPAttributes
89
65
    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
                }
90
104
 
91
105
 
92
106
_paramiko_version = getattr(paramiko, '__version_info__', (0, 0, 0))
93
107
# don't use prefetch unless paramiko version >= 1.5.5 (there were bugs earlier)
94
108
_default_do_prefetch = (_paramiko_version >= (1, 5, 5))
95
109
 
 
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
 
96
278
 
97
279
class SFTPLock(object):
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
 
 
 
280
    """This fakes a lock in a remote location."""
105
281
    __slots__ = ['path', 'lock_path', 'lock_file', 'transport']
106
 
 
107
282
    def __init__(self, path, transport):
 
283
        assert isinstance(transport, SFTPTransport)
 
284
 
108
285
        self.lock_file = None
109
286
        self.path = path
110
287
        self.lock_path = path + '.write-lock'
134
311
            pass
135
312
 
136
313
 
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."""
 
314
class SFTPTransport (Transport):
 
315
    """Transport implementation for SFTP access"""
327
316
 
328
317
    _do_prefetch = _default_do_prefetch
329
318
    # TODO: jam 20060717 Conceivably these could be configurable, either
343
332
    # up the request itself, rather than us having to worry about it
344
333
    _max_request_size = 32768
345
334
 
346
 
    def __init__(self, base, _from_transport=None):
347
 
        super(SFTPTransport, self).__init__(base,
348
 
                                            _from_transport=_from_transport)
349
 
 
 
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
    
350
375
    def _remote_path(self, relpath):
351
376
        """Return the path to be passed along the sftp protocol for relpath.
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
 
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('/')
402
419
 
403
420
    def has(self, relpath):
404
421
        """
405
422
        Does the target location exist?
406
423
        """
407
424
        try:
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')
 
425
            self._sftp.stat(self._remote_path(relpath))
411
426
            return True
412
427
        except IOError:
413
428
            return False
414
429
 
415
430
    def get(self, relpath):
416
 
        """Get the file at the given relative path.
 
431
        """
 
432
        Get the file at the given relative path.
417
433
 
418
434
        :param relpath: The relative path to the file
419
435
        """
420
436
        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
426
437
            path = self._remote_path(relpath)
427
 
            f = self._get_sftp().file(path, mode='rb')
 
438
            f = self._sftp.file(path, mode='rb')
428
439
            if self._do_prefetch and (getattr(f, 'prefetch', None) is not None):
429
440
                f.prefetch()
430
441
            return f
431
442
        except (IOError, paramiko.SSHException), e:
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):
 
443
            self._translate_io_exception(e, path, ': error retrieving')
 
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._get_sftp().file(path, mode='rb')
 
455
            fp = self._sftp.file(path, mode='rb')
456
456
            readv = getattr(fp, 'readv', None)
457
457
            if readv:
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)
 
458
                return self._sftp_readv(fp, offsets)
 
459
            mutter('seek and read %s offsets', len(offsets))
 
460
            return self._seek_and_read(fp, offsets)
462
461
        except (IOError, paramiko.SSHException), e:
463
462
            self._translate_io_exception(e, path, ': error retrieving')
464
463
 
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):
 
464
    def _sftp_readv(self, fp, offsets):
474
465
        """Use the readv() member of fp to do async readv.
475
466
 
476
 
        Then read them using paramiko.readv(). paramiko.readv()
 
467
        And then read them using paramiko.readv(). paramiko.readv()
477
468
        does not support ranges > 64K, so it caps the request size, and
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.
 
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.
486
564
 
487
565
        :param relpath: Location to put the contents, relative to base.
488
 
        :param f:       File-like object.
 
566
        :param f:       File-like or string object.
489
567
        :param mode: The final mode for the file
490
568
        """
491
569
        final_path = self._remote_path(relpath)
492
 
        return self._put(final_path, f, mode=mode)
 
570
        self._put(final_path, f, mode=mode)
493
571
 
494
572
    def _put(self, abspath, f, mode=None):
495
573
        """Helper function so both put() and copy_abspaths can reuse the code"""
500
578
        try:
501
579
            try:
502
580
                fout.set_pipelined(True)
503
 
                length = self._pump(f, fout)
 
581
                self._pump(f, fout)
504
582
            except (IOError, paramiko.SSHException), e:
505
583
                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()
519
584
            if mode is not None:
520
 
                self._get_sftp().chmod(tmp_abspath, mode)
 
585
                self._sftp.chmod(tmp_abspath, mode)
521
586
            fout.close()
522
587
            closed = True
523
588
            self._rename_and_overwrite(tmp_abspath, abspath)
524
 
            return length
525
589
        except Exception, e:
526
590
            # If we fail, try to clean up the temporary file
527
591
            # before we throw the exception
533
597
            try:
534
598
                if not closed:
535
599
                    fout.close()
536
 
                self._get_sftp().remove(tmp_abspath)
 
600
                self._sftp.remove(tmp_abspath)
537
601
            except:
538
602
                # raise the saved except
539
603
                raise e
540
604
            # raise the original with its traceback if we can.
541
605
            raise
542
606
 
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
 
 
620
607
    def iter_files_recursive(self):
621
608
        """Walk the relative paths of all files in this transport."""
622
 
        # progress is handled by list_dir
623
609
        queue = list(self.list_dir('.'))
624
610
        while queue:
625
 
            relpath = queue.pop(0)
 
611
            relpath = urllib.quote(queue.pop(0))
626
612
            st = self.stat(relpath)
627
613
            if stat.S_ISDIR(st.st_mode):
628
614
                for i, basename in enumerate(self.list_dir(relpath)):
630
616
            else:
631
617
                yield relpath
632
618
 
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
 
 
662
619
    def mkdir(self, relpath, mode=None):
663
620
        """Create a directory at the given path."""
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
 
621
        path = self._remote_path(relpath)
679
622
        try:
680
 
            handle = self._get_sftp().file(abspath, mode='wb')
681
 
            handle.set_pipelined(True)
 
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)
682
631
        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)
 
632
            self._translate_io_exception(e, path, ': unable to mkdir',
 
633
                failure_exc=FileExists)
687
634
 
688
 
    def _translate_io_exception(self, e, path, more_info='',
 
635
    def _translate_io_exception(self, e, path, more_info='', 
689
636
                                failure_exc=PathError):
690
637
        """Translate a paramiko or IOError into a friendlier exception.
691
638
 
696
643
        :param failure_exc: Paramiko has the super fun ability to raise completely
697
644
                           opaque errors that just set "e.args = ('Failure',)" with
698
645
                           no more information.
699
 
                           If this parameter is set, it defines the exception
 
646
                           If this parameter is set, it defines the exception 
700
647
                           to raise in these cases.
701
648
        """
702
649
        # paramiko seems to generate detailless errors.
703
650
        self._translate_error(e, path, raise_generic=False)
704
 
        if getattr(e, 'args', None) is not None:
 
651
        if hasattr(e, 'args'):
705
652
            if (e.args == ('No such file or directory',) or
706
653
                e.args == ('No such file',)):
707
654
                raise NoSuchFile(path, str(e) + more_info)
708
 
            if (e.args == ('mkdir failed',) or
709
 
                e.args[0].startswith('syserr: File exists')):
 
655
            if (e.args == ('mkdir failed',)):
710
656
                raise FileExists(path, str(e) + more_info)
711
657
            # strange but true, for the paramiko server.
712
658
            if (e.args == ('Failure',)):
713
659
                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))
720
660
            mutter('Raising exception with args %s', e.args)
721
 
        if getattr(e, 'errno', None) is not None:
 
661
        if hasattr(e, 'errno'):
722
662
            mutter('Raising exception with errno %s', e.errno)
723
663
        raise e
724
664
 
725
 
    def append_file(self, relpath, f, mode=None):
 
665
    def append(self, relpath, f, mode=None):
726
666
        """
727
667
        Append the text in the file-like object into the final
728
668
        location.
729
669
        """
730
670
        try:
731
671
            path = self._remote_path(relpath)
732
 
            fout = self._get_sftp().file(path, 'ab')
 
672
            fout = self._sftp.file(path, 'ab')
733
673
            if mode is not None:
734
 
                self._get_sftp().chmod(path, mode)
 
674
                self._sftp.chmod(path, mode)
735
675
            result = fout.tell()
736
676
            self._pump(f, fout)
737
677
            return result
741
681
    def rename(self, rel_from, rel_to):
742
682
        """Rename without special overwriting"""
743
683
        try:
744
 
            self._get_sftp().rename(self._remote_path(rel_from),
 
684
            self._sftp.rename(self._remote_path(rel_from),
745
685
                              self._remote_path(rel_to))
746
686
        except (IOError, paramiko.SSHException), e:
747
687
            self._translate_io_exception(e, rel_from,
749
689
 
750
690
    def _rename_and_overwrite(self, abs_from, abs_to):
751
691
        """Do a fancy rename on the remote server.
752
 
 
 
692
        
753
693
        Using the implementation provided by osutils.
754
694
        """
755
695
        try:
756
 
            sftp = self._get_sftp()
757
696
            fancy_rename(abs_from, abs_to,
758
 
                         rename_func=sftp.rename,
759
 
                         unlink_func=sftp.remove)
 
697
                    rename_func=self._sftp.rename,
 
698
                    unlink_func=self._sftp.remove)
760
699
        except (IOError, paramiko.SSHException), e:
761
 
            self._translate_io_exception(e, abs_from,
762
 
                                         ': unable to rename to %r' % (abs_to))
 
700
            self._translate_io_exception(e, abs_from, ': unable to rename to %r' % (abs_to))
763
701
 
764
702
    def move(self, rel_from, rel_to):
765
703
        """Move the item at rel_from to the location at rel_to"""
771
709
        """Delete the item at relpath"""
772
710
        path = self._remote_path(relpath)
773
711
        try:
774
 
            self._get_sftp().remove(path)
 
712
            self._sftp.remove(path)
775
713
        except (IOError, paramiko.SSHException), e:
776
714
            self._translate_io_exception(e, path, ': unable to delete')
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
 
 
 
715
            
783
716
    def listable(self):
784
717
        """Return True if this store supports listing."""
785
718
        return True
789
722
        Return a list of all files at the given location.
790
723
        """
791
724
        # 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
795
725
        path = self._remote_path(relpath)
796
726
        try:
797
 
            entries = self._get_sftp().listdir(path)
798
 
            self._report_activity(sum(map(len, entries)), 'read')
 
727
            return self._sftp.listdir(path)
799
728
        except (IOError, paramiko.SSHException), e:
800
729
            self._translate_io_exception(e, path, ': failed to list_dir')
801
 
        return [urlutils.escape(entry) for entry in entries]
802
730
 
803
731
    def rmdir(self, relpath):
804
732
        """See Transport.rmdir."""
805
733
        path = self._remote_path(relpath)
806
734
        try:
807
 
            return self._get_sftp().rmdir(path)
 
735
            return self._sftp.rmdir(path)
808
736
        except (IOError, paramiko.SSHException), e:
809
737
            self._translate_io_exception(e, path, ': failed to rmdir')
810
738
 
812
740
        """Return the stat information for a file."""
813
741
        path = self._remote_path(relpath)
814
742
        try:
815
 
            return self._get_sftp().stat(path)
 
743
            return self._sftp.stat(path)
816
744
        except (IOError, paramiko.SSHException), e:
817
745
            self._translate_io_exception(e, path, ': unable to stat')
818
746
 
842
770
        # that we have taken the lock.
843
771
        return SFTPLock(relpath, self)
844
772
 
 
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
            sock = SFTPSubprocess(self._host, vendor, self._port,
 
835
                                  self._username)
 
836
            self._sftp = SFTPClient(sock)
 
837
        else:
 
838
            self._paramiko_connect()
 
839
 
 
840
        _connected_hosts[idx] = self._sftp
 
841
 
 
842
    def _paramiko_connect(self):
 
843
        global SYSTEM_HOSTKEYS, BZR_HOSTKEYS
 
844
        
 
845
        load_host_keys()
 
846
 
 
847
        try:
 
848
            t = paramiko.Transport((self._host, self._port or 22))
 
849
            t.set_log_channel('bzr.paramiko')
 
850
            t.start_client()
 
851
        except paramiko.SSHException, e:
 
852
            raise ConnectionError('Unable to reach SSH host %s:%s: %s' 
 
853
                                  % (self._host, self._port, e))
 
854
            
 
855
        server_key = t.get_remote_server_key()
 
856
        server_key_hex = paramiko.util.hexify(server_key.get_fingerprint())
 
857
        keytype = server_key.get_name()
 
858
        if SYSTEM_HOSTKEYS.has_key(self._host) and SYSTEM_HOSTKEYS[self._host].has_key(keytype):
 
859
            our_server_key = SYSTEM_HOSTKEYS[self._host][keytype]
 
860
            our_server_key_hex = paramiko.util.hexify(our_server_key.get_fingerprint())
 
861
        elif BZR_HOSTKEYS.has_key(self._host) and BZR_HOSTKEYS[self._host].has_key(keytype):
 
862
            our_server_key = BZR_HOSTKEYS[self._host][keytype]
 
863
            our_server_key_hex = paramiko.util.hexify(our_server_key.get_fingerprint())
 
864
        else:
 
865
            warning('Adding %s host key for %s: %s' % (keytype, self._host, server_key_hex))
 
866
            if not BZR_HOSTKEYS.has_key(self._host):
 
867
                BZR_HOSTKEYS[self._host] = {}
 
868
            BZR_HOSTKEYS[self._host][keytype] = server_key
 
869
            our_server_key = server_key
 
870
            our_server_key_hex = paramiko.util.hexify(our_server_key.get_fingerprint())
 
871
            save_host_keys()
 
872
        if server_key != our_server_key:
 
873
            filename1 = os.path.expanduser('~/.ssh/known_hosts')
 
874
            filename2 = pathjoin(config_dir(), 'ssh_host_keys')
 
875
            raise TransportError('Host keys for %s do not match!  %s != %s' % \
 
876
                (self._host, our_server_key_hex, server_key_hex),
 
877
                ['Try editing %s or %s' % (filename1, filename2)])
 
878
 
 
879
        self._sftp_auth(t)
 
880
        
 
881
        try:
 
882
            self._sftp = t.open_sftp_client()
 
883
        except paramiko.SSHException, e:
 
884
            raise ConnectionError('Unable to start sftp client %s:%d' %
 
885
                                  (self._host, self._port), e)
 
886
 
 
887
    def _sftp_auth(self, transport):
 
888
        # paramiko requires a username, but it might be none if nothing was supplied
 
889
        # use the local username, just in case.
 
890
        # We don't override self._username, because if we aren't using paramiko,
 
891
        # the username might be specified in ~/.ssh/config and we don't want to
 
892
        # force it to something else
 
893
        # Also, it would mess up the self.relpath() functionality
 
894
        username = self._username or getpass.getuser()
 
895
 
 
896
        if _use_ssh_agent:
 
897
            agent = paramiko.Agent()
 
898
            for key in agent.get_keys():
 
899
                mutter('Trying SSH agent key %s' % paramiko.util.hexify(key.get_fingerprint()))
 
900
                try:
 
901
                    transport.auth_publickey(username, key)
 
902
                    return
 
903
                except paramiko.SSHException, e:
 
904
                    pass
 
905
        
 
906
        # okay, try finding id_rsa or id_dss?  (posix only)
 
907
        if self._try_pkey_auth(transport, paramiko.RSAKey, username, 'id_rsa'):
 
908
            return
 
909
        if self._try_pkey_auth(transport, paramiko.DSSKey, username, 'id_dsa'):
 
910
            return
 
911
 
 
912
        if self._password:
 
913
            try:
 
914
                transport.auth_password(username, self._password)
 
915
                return
 
916
            except paramiko.SSHException, e:
 
917
                pass
 
918
 
 
919
            # FIXME: Don't keep a password held in memory if you can help it
 
920
            #self._password = None
 
921
 
 
922
        # give up and ask for a password
 
923
        password = bzrlib.ui.ui_factory.get_password(
 
924
                prompt='SSH %(user)s@%(host)s password',
 
925
                user=username, host=self._host)
 
926
        try:
 
927
            transport.auth_password(username, password)
 
928
        except paramiko.SSHException, e:
 
929
            raise ConnectionError('Unable to authenticate to SSH host as %s@%s' %
 
930
                                  (username, self._host), e)
 
931
 
 
932
    def _try_pkey_auth(self, transport, pkey_class, username, filename):
 
933
        filename = os.path.expanduser('~/.ssh/' + filename)
 
934
        try:
 
935
            key = pkey_class.from_private_key_file(filename)
 
936
            transport.auth_publickey(username, key)
 
937
            return True
 
938
        except paramiko.PasswordRequiredException:
 
939
            password = bzrlib.ui.ui_factory.get_password(
 
940
                    prompt='SSH %(filename)s password',
 
941
                    filename=filename)
 
942
            try:
 
943
                key = pkey_class.from_private_key_file(filename, password)
 
944
                transport.auth_publickey(username, key)
 
945
                return True
 
946
            except paramiko.SSHException:
 
947
                mutter('SSH authentication via %s key failed.' % (os.path.basename(filename),))
 
948
        except paramiko.SSHException:
 
949
            mutter('SSH authentication via %s key failed.' % (os.path.basename(filename),))
 
950
        except IOError:
 
951
            pass
 
952
        return False
 
953
 
845
954
    def _sftp_open_exclusive(self, abspath, mode=None):
846
955
        """Open a remote path exclusively.
847
956
 
856
965
        :param abspath: The remote absolute path where the file should be opened
857
966
        :param mode: The mode permissions bits for the new file
858
967
        """
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)
 
968
        path = self._sftp._adjust_cwd(abspath)
864
969
        # mutter('sftp abspath %s => %s', abspath, path)
865
970
        attr = SFTPAttributes()
866
971
        if mode is not None:
867
972
            attr.st_mode = mode
868
 
        omode = (SFTP_FLAG_WRITE | SFTP_FLAG_CREATE
 
973
        omode = (SFTP_FLAG_WRITE | SFTP_FLAG_CREATE 
869
974
                | SFTP_FLAG_TRUNC | SFTP_FLAG_EXCL)
870
975
        try:
871
 
            t, msg = self._get_sftp()._request(CMD_OPEN, path, omode, attr)
 
976
            t, msg = self._sftp._request(CMD_OPEN, path, omode, attr)
872
977
            if t != CMD_HANDLE:
873
978
                raise TransportError('Expected an SFTP handle')
874
979
            handle = msg.get_string()
875
 
            return SFTPFile(self._get_sftp(), handle, 'wb', -1)
 
980
            return SFTPFile(self._sftp, handle, 'wb', -1)
876
981
        except (paramiko.SSHException, IOError), e:
877
982
            self._translate_io_exception(e, abspath, ': unable to open',
878
983
                failure_exc=FileExists)
879
984
 
880
 
    def _can_roundtrip_unix_modebits(self):
881
 
        if sys.platform == 'win32':
882
 
            # anyone else?
883
 
            return False
884
 
        else:
885
 
            return True
886
985
 
887
986
# ------------- server test implementation --------------
 
987
import socket
888
988
import threading
889
989
 
890
990
from bzrlib.tests.stub_sftp import StubServer, StubSFTPServer
917
1017
        self._socket.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
918
1018
        self._socket.bind(('localhost', 0))
919
1019
        self._socket.listen(1)
920
 
        self.host, self.port = self._socket.getsockname()[:2]
 
1020
        self.port = self._socket.getsockname()[1]
921
1021
        self._stop_event = threading.Event()
922
1022
 
923
1023
    def stop(self):
949
1049
                # probably a failed test; unit test thread will log the
950
1050
                # failure/error
951
1051
                sys.excepthook(*sys.exc_info())
952
 
                warning('Exception from within unit test server thread: %r' %
 
1052
                warning('Exception from within unit test server thread: %r' % 
953
1053
                        x)
954
1054
 
955
1055
 
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
 
 
1031
1056
class SFTPServer(Server):
1032
1057
    """Common code for SFTP server facilities."""
1033
1058
 
1034
 
    def __init__(self, server_interface=StubServer):
 
1059
    def __init__(self):
1035
1060
        self._original_vendor = None
1036
1061
        self._homedir = None
1037
1062
        self._server_homedir = None
1038
1063
        self._listener = None
1039
1064
        self._root = None
1040
 
        self._vendor = ssh.ParamikoVendor()
1041
 
        self._server_interface = server_interface
 
1065
        self._vendor = 'none'
1042
1066
        # sftp server logs
1043
1067
        self.logs = []
1044
 
        self.add_latency = 0
1045
1068
 
1046
1069
    def _get_sftp_url(self, path):
1047
1070
        """Calculate an sftp url to this server for path."""
1048
 
        return 'sftp://foo:bar@%s:%d/%s' % (self._listener.host,
1049
 
                                            self._listener.port, path)
 
1071
        return 'sftp://foo:bar@localhost:%d/%s' % (self._listener.port, path)
1050
1072
 
1051
1073
    def log(self, message):
1052
1074
        """StubServer uses this to log when a new server is created."""
1053
1075
        self.logs.append(message)
1054
1076
 
1055
 
    def _run_server_entry(self, sock):
1056
 
        """Entry point for all implementations of _run_server.
1057
 
 
1058
 
        If self.add_latency is > 0.000001 then sock is given a latency adding
1059
 
        decorator.
1060
 
        """
1061
 
        if self.add_latency > 0.000001:
1062
 
            sock = SocketDelay(sock, self.add_latency)
1063
 
        return self._run_server(sock)
1064
 
 
1065
1077
    def _run_server(self, s):
1066
1078
        ssh_server = paramiko.Transport(s)
1067
1079
        key_file = pathjoin(self._homedir, 'test_rsa.key')
1070
1082
        f.close()
1071
1083
        host_key = paramiko.RSAKey.from_private_key_file(key_file)
1072
1084
        ssh_server.add_server_key(host_key)
1073
 
        server = self._server_interface(self)
 
1085
        server = StubServer(self)
1074
1086
        ssh_server.set_subsystem_handler('sftp', paramiko.SFTPServer,
1075
1087
                                         StubSFTPServer, root=self._root,
1076
1088
                                         home=self._server_homedir)
1077
1089
        event = threading.Event()
1078
1090
        ssh_server.start_server(event, server)
1079
1091
        event.wait(5.0)
1080
 
 
1081
 
    def start_server(self, backing_server=None):
1082
 
        # XXX: TODO: make sftpserver back onto backing_server rather than local
1083
 
        # disk.
1084
 
        if not (backing_server is None or
1085
 
                isinstance(backing_server, local.LocalURLServer)):
1086
 
            raise AssertionError(
1087
 
                "backing_server should not be %r, because this can only serve the "
1088
 
                "local current working directory." % (backing_server,))
1089
 
        self._original_vendor = ssh._ssh_vendor_manager._cached_ssh_vendor
1090
 
        ssh._ssh_vendor_manager._cached_ssh_vendor = self._vendor
 
1092
    
 
1093
    def setUp(self):
 
1094
        global _ssh_vendor
 
1095
        self._original_vendor = _ssh_vendor
 
1096
        _ssh_vendor = self._vendor
1091
1097
        if sys.platform == 'win32':
1092
1098
            # Win32 needs to use the UNICODE api
1093
1099
            self._homedir = getcwd()
1099
1105
        self._root = '/'
1100
1106
        if sys.platform == 'win32':
1101
1107
            self._root = ''
1102
 
        self._listener = SocketListener(self._run_server_entry)
 
1108
        self._listener = SocketListener(self._run_server)
1103
1109
        self._listener.setDaemon(True)
1104
1110
        self._listener.start()
1105
1111
 
1106
 
    def stop_server(self):
 
1112
    def tearDown(self):
 
1113
        """See bzrlib.transport.Server.tearDown."""
 
1114
        global _ssh_vendor
1107
1115
        self._listener.stop()
1108
 
        ssh._ssh_vendor_manager._cached_ssh_vendor = self._original_vendor
 
1116
        _ssh_vendor = self._original_vendor
1109
1117
 
1110
1118
    def get_bogus_url(self):
1111
1119
        """See bzrlib.transport.Server.get_bogus_url."""
1112
 
        # this is chosen to try to prevent trouble with proxies, wierd dns, etc
1113
 
        # we bind a random socket, so that we get a guaranteed unused port
1114
 
        # we just never listen on that port
1115
 
        s = socket.socket()
1116
 
        s.bind(('localhost', 0))
1117
 
        return 'sftp://%s:%s/' % s.getsockname()
 
1120
        # this is chosen to try to prevent trouble with proxies, wierd dns,
 
1121
        # etc
 
1122
        return 'sftp://127.0.0.1:1/'
 
1123
 
1118
1124
 
1119
1125
 
1120
1126
class SFTPFullAbsoluteServer(SFTPServer):
1122
1128
 
1123
1129
    def get_url(self):
1124
1130
        """See bzrlib.transport.Server.get_url."""
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))
 
1131
        return self._get_sftp_url(urlutils.escape(self._homedir[1:]))
1130
1132
 
1131
1133
 
1132
1134
class SFTPServerWithoutSSH(SFTPServer):
1134
1136
 
1135
1137
    def __init__(self):
1136
1138
        super(SFTPServerWithoutSSH, self).__init__()
1137
 
        self._vendor = ssh.LoopbackVendor()
 
1139
        self._vendor = 'loopback'
1138
1140
 
1139
1141
    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'".
1143
1142
        class FakeChannel(object):
1144
1143
            def get_transport(self):
1145
1144
                return self
1152
1151
            def close(self):
1153
1152
                pass
1154
1153
 
1155
 
        server = paramiko.SFTPServer(
1156
 
            FakeChannel(), 'sftp', StubServer(self), StubSFTPServer,
1157
 
            root=self._root, home=self._server_homedir)
 
1154
        server = paramiko.SFTPServer(FakeChannel(), 'sftp', StubServer(self), StubSFTPServer,
 
1155
                                     root=self._root, home=self._server_homedir)
1158
1156
        try:
1159
 
            server.start_subsystem(
1160
 
                'sftp', None, ssh.SocketAsChannelAdapter(sock))
 
1157
            server.start_subsystem('sftp', None, sock)
1161
1158
        except socket.error, e:
1162
1159
            if (len(e.args) > 0) and (e.args[0] == errno.EPIPE):
1163
1160
                # it's okay for the client to disconnect abruptly
1166
1163
            else:
1167
1164
                raise
1168
1165
        except Exception, e:
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,))
 
1166
            import sys; sys.stderr.write('\nEXCEPTION %r\n\n' % e.__class__)
1176
1167
        server.finish_subsystem()
1177
1168
 
1178
1169
 
1181
1172
 
1182
1173
    def get_url(self):
1183
1174
        """See bzrlib.transport.Server.get_url."""
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))
 
1175
        if sys.platform == 'win32':
 
1176
            return self._get_sftp_url(urlutils.escape(self._homedir))
 
1177
        else:
 
1178
            return self._get_sftp_url(urlutils.escape(self._homedir[1:]))
1189
1179
 
1190
1180
 
1191
1181
class SFTPHomeDirServer(SFTPServerWithoutSSH):
1197
1187
 
1198
1188
 
1199
1189
class SFTPSiblingAbsoluteServer(SFTPAbsoluteServer):
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 start_server(self, backing_server=None):
 
1190
    """A test servere for sftp transports, using absolute urls to non-home."""
 
1191
 
 
1192
    def setUp(self):
1206
1193
        self._server_homedir = '/dev/noone/runs/tests/here'
1207
 
        super(SFTPSiblingAbsoluteServer, self).start_server(backing_server)
 
1194
        super(SFTPSiblingAbsoluteServer, self).setUp()
1208
1195
 
1209
1196
 
1210
1197
def get_test_permutations():