~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/remote.py

  • Committer: Martin Pool
  • Date: 2005-09-13 23:18:12 UTC
  • Revision ID: mbp@sourcefrog.net-20050913231812-dc1df8fb11f78a31
- rename last_patch to last_revision

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006 Canonical Ltd
2
 
#
3
 
# This program is free software; you can redistribute it and/or modify
4
 
# it under the terms of the GNU General Public License as published by
5
 
# the Free Software Foundation; either version 2 of the License, or
6
 
# (at your option) any later version.
7
 
#
8
 
# This program is distributed in the hope that it will be useful,
9
 
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
 
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
 
# GNU General Public License for more details.
12
 
#
13
 
# You should have received a copy of the GNU General Public License
14
 
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
"""RemoteTransport client for the smart-server.
18
 
 
19
 
This module shouldn't be accessed directly.  The classes defined here should be
20
 
imported from bzrlib.smart.
21
 
"""
22
 
 
23
 
__all__ = ['RemoteTransport', 'RemoteTCPTransport', 'RemoteSSHTransport']
24
 
 
25
 
from cStringIO import StringIO
26
 
 
27
 
from bzrlib import (
28
 
    config,
29
 
    debug,
30
 
    errors,
31
 
    remote,
32
 
    trace,
33
 
    transport,
34
 
    urlutils,
35
 
    )
36
 
from bzrlib.smart import client, medium
37
 
from bzrlib.symbol_versioning import (deprecated_method, one_four)
38
 
 
39
 
 
40
 
class _SmartStat(object):
41
 
 
42
 
    def __init__(self, size, mode):
43
 
        self.st_size = size
44
 
        self.st_mode = mode
45
 
 
46
 
 
47
 
class RemoteTransport(transport.ConnectedTransport):
48
 
    """Connection to a smart server.
49
 
 
50
 
    The connection holds references to the medium that can be used to send
51
 
    requests to the server.
52
 
 
53
 
    The connection has a notion of the current directory to which it's
54
 
    connected; this is incorporated in filenames passed to the server.
55
 
    
56
 
    This supports some higher-level RPC operations and can also be treated 
57
 
    like a Transport to do file-like operations.
58
 
 
59
 
    The connection can be made over a tcp socket, an ssh pipe or a series of
60
 
    http requests.  There are concrete subclasses for each type:
61
 
    RemoteTCPTransport, etc.
62
 
    """
63
 
 
64
 
    # When making a readv request, cap it at requesting 5MB of data
65
 
    _max_readv_bytes = 5*1024*1024
66
 
 
67
 
    # IMPORTANT FOR IMPLEMENTORS: RemoteTransport MUST NOT be given encoding
68
 
    # responsibilities: Put those on SmartClient or similar. This is vital for
69
 
    # the ability to support multiple versions of the smart protocol over time:
70
 
    # RemoteTransport is an adapter from the Transport object model to the 
71
 
    # SmartClient model, not an encoder.
72
 
 
73
 
    # FIXME: the medium parameter should be private, only the tests requires
74
 
    # it. It may be even clearer to define a TestRemoteTransport that handles
75
 
    # the specific cases of providing a _client and/or a _medium, and leave
76
 
    # RemoteTransport as an abstract class.
77
 
    def __init__(self, url, _from_transport=None, medium=None, _client=None):
78
 
        """Constructor.
79
 
 
80
 
        :param _from_transport: Another RemoteTransport instance that this
81
 
            one is being cloned from.  Attributes such as the medium will
82
 
            be reused.
83
 
 
84
 
        :param medium: The medium to use for this RemoteTransport.  If None,
85
 
            the medium from the _from_transport is shared.  If both this
86
 
            and _from_transport are None, a new medium will be built.
87
 
            _from_transport and medium cannot both be specified.
88
 
 
89
 
        :param _client: Override the _SmartClient used by this transport.  This
90
 
            should only be used for testing purposes; normally this is
91
 
            determined from the medium.
92
 
        """
93
 
        super(RemoteTransport, self).__init__(url,
94
 
                                              _from_transport=_from_transport)
95
 
 
96
 
        # The medium is the connection, except when we need to share it with
97
 
        # other objects (RemoteBzrDir, RemoteRepository etc). In these cases
98
 
        # what we want to share is really the shared connection.
99
 
 
100
 
        if _from_transport is None:
101
 
            # If no _from_transport is specified, we need to intialize the
102
 
            # shared medium.
103
 
            credentials = None
104
 
            if medium is None:
105
 
                medium, credentials = self._build_medium()
106
 
                if 'hpss' in debug.debug_flags:
107
 
                    trace.mutter('hpss: Built a new medium: %s',
108
 
                                 medium.__class__.__name__)
109
 
            self._shared_connection = transport._SharedConnection(medium,
110
 
                                                                  credentials,
111
 
                                                                  self.base)
112
 
        elif medium is None:
113
 
            # No medium was specified, so share the medium from the
114
 
            # _from_transport.
115
 
            medium = self._shared_connection.connection
116
 
        else:
117
 
            raise AssertionError(
118
 
                "Both _from_transport (%r) and medium (%r) passed to "
119
 
                "RemoteTransport.__init__, but these parameters are mutally "
120
 
                "exclusive." % (_from_transport, medium))
121
 
 
122
 
        if _client is None:
123
 
            self._client = client._SmartClient(medium)
124
 
        else:
125
 
            self._client = _client
126
 
 
127
 
    def _build_medium(self):
128
 
        """Create the medium if _from_transport does not provide one.
129
 
 
130
 
        The medium is analogous to the connection for ConnectedTransport: it
131
 
        allows connection sharing.
132
 
        """
133
 
        # No credentials
134
 
        return None, None
135
 
 
136
 
    def is_readonly(self):
137
 
        """Smart server transport can do read/write file operations."""
138
 
        try:
139
 
            resp = self._call2('Transport.is_readonly')
140
 
        except errors.UnknownSmartMethod:
141
 
            # XXX: nasty hack: servers before 0.16 don't have a
142
 
            # 'Transport.is_readonly' verb, so we do what clients before 0.16
143
 
            # did: assume False.
144
 
            return False
145
 
        if resp == ('yes', ):
146
 
            return True
147
 
        elif resp == ('no', ):
148
 
            return False
149
 
        else:
150
 
            raise errors.UnexpectedSmartServerResponse(resp)
151
 
 
152
 
    def get_smart_client(self):
153
 
        return self._get_connection()
154
 
 
155
 
    def get_smart_medium(self):
156
 
        return self._get_connection()
157
 
 
158
 
    @deprecated_method(one_four)
159
 
    def get_shared_medium(self):
160
 
        return self._get_shared_connection()
161
 
 
162
 
    def _remote_path(self, relpath):
163
 
        """Returns the Unicode version of the absolute path for relpath."""
164
 
        return self._combine_paths(self._path, relpath)
165
 
 
166
 
    def _call(self, method, *args):
167
 
        resp = self._call2(method, *args)
168
 
        self._ensure_ok(resp)
169
 
 
170
 
    def _call2(self, method, *args):
171
 
        """Call a method on the remote server."""
172
 
        try:
173
 
            return self._client.call(method, *args)
174
 
        except errors.ErrorFromSmartServer, err:
175
 
            # The first argument, if present, is always a path.
176
 
            if args:
177
 
                context = {'relpath': args[0]}
178
 
            else:
179
 
                context = {}
180
 
            self._translate_error(err, **context)
181
 
 
182
 
    def _call_with_body_bytes(self, method, args, body):
183
 
        """Call a method on the remote server with body bytes."""
184
 
        try:
185
 
            return self._client.call_with_body_bytes(method, args, body)
186
 
        except errors.ErrorFromSmartServer, err:
187
 
            # The first argument, if present, is always a path.
188
 
            if args:
189
 
                context = {'relpath': args[0]}
190
 
            else:
191
 
                context = {}
192
 
            self._translate_error(err, **context)
193
 
 
194
 
    def has(self, relpath):
195
 
        """Indicate whether a remote file of the given name exists or not.
196
 
 
197
 
        :see: Transport.has()
198
 
        """
199
 
        resp = self._call2('has', self._remote_path(relpath))
200
 
        if resp == ('yes', ):
201
 
            return True
202
 
        elif resp == ('no', ):
203
 
            return False
204
 
        else:
205
 
            raise errors.UnexpectedSmartServerResponse(resp)
206
 
 
207
 
    def get(self, relpath):
208
 
        """Return file-like object reading the contents of a remote file.
209
 
        
210
 
        :see: Transport.get_bytes()/get_file()
211
 
        """
212
 
        return StringIO(self.get_bytes(relpath))
213
 
 
214
 
    def get_bytes(self, relpath):
215
 
        remote = self._remote_path(relpath)
216
 
        try:
217
 
            resp, response_handler = self._client.call_expecting_body('get', remote)
218
 
        except errors.ErrorFromSmartServer, err:
219
 
            self._translate_error(err, relpath)
220
 
        if resp != ('ok', ):
221
 
            response_handler.cancel_read_body()
222
 
            raise errors.UnexpectedSmartServerResponse(resp)
223
 
        return response_handler.read_body_bytes()
224
 
 
225
 
    def _serialise_optional_mode(self, mode):
226
 
        if mode is None:
227
 
            return ''
228
 
        else:
229
 
            return '%d' % mode
230
 
 
231
 
    def mkdir(self, relpath, mode=None):
232
 
        resp = self._call2('mkdir', self._remote_path(relpath),
233
 
            self._serialise_optional_mode(mode))
234
 
 
235
 
    def open_write_stream(self, relpath, mode=None):
236
 
        """See Transport.open_write_stream."""
237
 
        self.put_bytes(relpath, "", mode)
238
 
        result = transport.AppendBasedFileStream(self, relpath)
239
 
        transport._file_streams[self.abspath(relpath)] = result
240
 
        return result
241
 
 
242
 
    def put_bytes(self, relpath, upload_contents, mode=None):
243
 
        # FIXME: upload_file is probably not safe for non-ascii characters -
244
 
        # should probably just pass all parameters as length-delimited
245
 
        # strings?
246
 
        if type(upload_contents) is unicode:
247
 
            # Although not strictly correct, we raise UnicodeEncodeError to be
248
 
            # compatible with other transports.
249
 
            raise UnicodeEncodeError(
250
 
                'undefined', upload_contents, 0, 1,
251
 
                'put_bytes must be given bytes, not unicode.')
252
 
        resp = self._call_with_body_bytes('put',
253
 
            (self._remote_path(relpath), self._serialise_optional_mode(mode)),
254
 
            upload_contents)
255
 
        self._ensure_ok(resp)
256
 
        return len(upload_contents)
257
 
 
258
 
    def put_bytes_non_atomic(self, relpath, bytes, mode=None,
259
 
                             create_parent_dir=False,
260
 
                             dir_mode=None):
261
 
        """See Transport.put_bytes_non_atomic."""
262
 
        # FIXME: no encoding in the transport!
263
 
        create_parent_str = 'F'
264
 
        if create_parent_dir:
265
 
            create_parent_str = 'T'
266
 
 
267
 
        resp = self._call_with_body_bytes(
268
 
            'put_non_atomic',
269
 
            (self._remote_path(relpath), self._serialise_optional_mode(mode),
270
 
             create_parent_str, self._serialise_optional_mode(dir_mode)),
271
 
            bytes)
272
 
        self._ensure_ok(resp)
273
 
 
274
 
    def put_file(self, relpath, upload_file, mode=None):
275
 
        # its not ideal to seek back, but currently put_non_atomic_file depends
276
 
        # on transports not reading before failing - which is a faulty
277
 
        # assumption I think - RBC 20060915
278
 
        pos = upload_file.tell()
279
 
        try:
280
 
            return self.put_bytes(relpath, upload_file.read(), mode)
281
 
        except:
282
 
            upload_file.seek(pos)
283
 
            raise
284
 
 
285
 
    def put_file_non_atomic(self, relpath, f, mode=None,
286
 
                            create_parent_dir=False,
287
 
                            dir_mode=None):
288
 
        return self.put_bytes_non_atomic(relpath, f.read(), mode=mode,
289
 
                                         create_parent_dir=create_parent_dir,
290
 
                                         dir_mode=dir_mode)
291
 
 
292
 
    def append_file(self, relpath, from_file, mode=None):
293
 
        return self.append_bytes(relpath, from_file.read(), mode)
294
 
        
295
 
    def append_bytes(self, relpath, bytes, mode=None):
296
 
        resp = self._call_with_body_bytes(
297
 
            'append',
298
 
            (self._remote_path(relpath), self._serialise_optional_mode(mode)),
299
 
            bytes)
300
 
        if resp[0] == 'appended':
301
 
            return int(resp[1])
302
 
        raise errors.UnexpectedSmartServerResponse(resp)
303
 
 
304
 
    def delete(self, relpath):
305
 
        resp = self._call2('delete', self._remote_path(relpath))
306
 
        self._ensure_ok(resp)
307
 
 
308
 
    def external_url(self):
309
 
        """See bzrlib.transport.Transport.external_url."""
310
 
        # the external path for RemoteTransports is the base
311
 
        return self.base
312
 
 
313
 
    def recommended_page_size(self):
314
 
        """Return the recommended page size for this transport."""
315
 
        return 64 * 1024
316
 
        
317
 
    def _readv(self, relpath, offsets):
318
 
        if not offsets:
319
 
            return
320
 
 
321
 
        offsets = list(offsets)
322
 
 
323
 
        sorted_offsets = sorted(offsets)
324
 
        coalesced = list(self._coalesce_offsets(sorted_offsets,
325
 
                               limit=self._max_readv_combine,
326
 
                               fudge_factor=self._bytes_to_read_before_seek,
327
 
                               max_size=self._max_readv_bytes))
328
 
 
329
 
        # now that we've coallesced things, avoid making enormous requests
330
 
        requests = []
331
 
        cur_request = []
332
 
        cur_len = 0
333
 
        for c in coalesced:
334
 
            if c.length + cur_len > self._max_readv_bytes:
335
 
                requests.append(cur_request)
336
 
                cur_request = [c]
337
 
                cur_len = c.length
338
 
                continue
339
 
            cur_request.append(c)
340
 
            cur_len += c.length
341
 
        if cur_request:
342
 
            requests.append(cur_request)
343
 
        if 'hpss' in debug.debug_flags:
344
 
            trace.mutter('%s.readv %s offsets => %s coalesced'
345
 
                         ' => %s requests (%s)',
346
 
                         self.__class__.__name__, len(offsets), len(coalesced),
347
 
                         len(requests), sum(map(len, requests)))
348
 
        # Cache the results, but only until they have been fulfilled
349
 
        data_map = {}
350
 
        # turn the list of offsets into a single stack to iterate
351
 
        offset_stack = iter(offsets)
352
 
        # using a list so it can be modified when passing down and coming back
353
 
        next_offset = [offset_stack.next()]
354
 
        for cur_request in requests:
355
 
            try:
356
 
                result = self._client.call_with_body_readv_array(
357
 
                    ('readv', self._remote_path(relpath),),
358
 
                    [(c.start, c.length) for c in cur_request])
359
 
                resp, response_handler = result
360
 
            except errors.ErrorFromSmartServer, err:
361
 
                self._translate_error(err, relpath)
362
 
 
363
 
            if resp[0] != 'readv':
364
 
                # This should raise an exception
365
 
                response_handler.cancel_read_body()
366
 
                raise errors.UnexpectedSmartServerResponse(resp)
367
 
 
368
 
            for res in self._handle_response(offset_stack, cur_request,
369
 
                                             response_handler,
370
 
                                             data_map,
371
 
                                             next_offset):
372
 
                yield res
373
 
 
374
 
    def _handle_response(self, offset_stack, coalesced, response_handler,
375
 
                         data_map, next_offset):
376
 
        cur_offset_and_size = next_offset[0]
377
 
        # FIXME: this should know how many bytes are needed, for clarity.
378
 
        data = response_handler.read_body_bytes()
379
 
        data_offset = 0
380
 
        for c_offset in coalesced:
381
 
            if len(data) < c_offset.length:
382
 
                raise errors.ShortReadvError(relpath, c_offset.start,
383
 
                            c_offset.length, actual=len(data))
384
 
            for suboffset, subsize in c_offset.ranges:
385
 
                key = (c_offset.start+suboffset, subsize)
386
 
                this_data = data[data_offset+suboffset:
387
 
                                 data_offset+suboffset+subsize]
388
 
                # Special case when the data is in-order, rather than packing
389
 
                # into a map and then back out again. Benchmarking shows that
390
 
                # this has 100% hit rate, but leave in the data_map work just
391
 
                # in case.
392
 
                # TODO: Could we get away with using buffer() to avoid the
393
 
                #       memory copy?  Callers would need to realize they may
394
 
                #       not have a real string.
395
 
                if key == cur_offset_and_size:
396
 
                    yield cur_offset_and_size[0], this_data
397
 
                    cur_offset_and_size = next_offset[0] = offset_stack.next()
398
 
                else:
399
 
                    data_map[key] = this_data
400
 
            data_offset += c_offset.length
401
 
 
402
 
            # Now that we've read some data, see if we can yield anything back
403
 
            while cur_offset_and_size in data_map:
404
 
                this_data = data_map.pop(cur_offset_and_size)
405
 
                yield cur_offset_and_size[0], this_data
406
 
                cur_offset_and_size = next_offset[0] = offset_stack.next()
407
 
 
408
 
    def rename(self, rel_from, rel_to):
409
 
        self._call('rename',
410
 
                   self._remote_path(rel_from),
411
 
                   self._remote_path(rel_to))
412
 
 
413
 
    def move(self, rel_from, rel_to):
414
 
        self._call('move',
415
 
                   self._remote_path(rel_from),
416
 
                   self._remote_path(rel_to))
417
 
 
418
 
    def rmdir(self, relpath):
419
 
        resp = self._call('rmdir', self._remote_path(relpath))
420
 
 
421
 
    def _ensure_ok(self, resp):
422
 
        if resp[0] != 'ok':
423
 
            raise errors.UnexpectedSmartServerResponse(resp)
424
 
        
425
 
    def _translate_error(self, err, relpath=None):
426
 
        remote._translate_error(err, path=relpath)
427
 
 
428
 
    def disconnect(self):
429
 
        self.get_smart_medium().disconnect()
430
 
 
431
 
    def stat(self, relpath):
432
 
        resp = self._call2('stat', self._remote_path(relpath))
433
 
        if resp[0] == 'stat':
434
 
            return _SmartStat(int(resp[1]), int(resp[2], 8))
435
 
        raise errors.UnexpectedSmartServerResponse(resp)
436
 
 
437
 
    ## def lock_read(self, relpath):
438
 
    ##     """Lock the given file for shared (read) access.
439
 
    ##     :return: A lock object, which should be passed to Transport.unlock()
440
 
    ##     """
441
 
    ##     # The old RemoteBranch ignore lock for reading, so we will
442
 
    ##     # continue that tradition and return a bogus lock object.
443
 
    ##     class BogusLock(object):
444
 
    ##         def __init__(self, path):
445
 
    ##             self.path = path
446
 
    ##         def unlock(self):
447
 
    ##             pass
448
 
    ##     return BogusLock(relpath)
449
 
 
450
 
    def listable(self):
451
 
        return True
452
 
 
453
 
    def list_dir(self, relpath):
454
 
        resp = self._call2('list_dir', self._remote_path(relpath))
455
 
        if resp[0] == 'names':
456
 
            return [name.encode('ascii') for name in resp[1:]]
457
 
        raise errors.UnexpectedSmartServerResponse(resp)
458
 
 
459
 
    def iter_files_recursive(self):
460
 
        resp = self._call2('iter_files_recursive', self._remote_path(''))
461
 
        if resp[0] == 'names':
462
 
            return resp[1:]
463
 
        raise errors.UnexpectedSmartServerResponse(resp)
464
 
 
465
 
 
466
 
class RemoteTCPTransport(RemoteTransport):
467
 
    """Connection to smart server over plain tcp.
468
 
    
469
 
    This is essentially just a factory to get 'RemoteTransport(url,
470
 
        SmartTCPClientMedium).
471
 
    """
472
 
 
473
 
    def _build_medium(self):
474
 
        client_medium = medium.SmartTCPClientMedium(
475
 
            self._host, self._port, self.base)
476
 
        return client_medium, None
477
 
 
478
 
 
479
 
class RemoteTCPTransportV2Only(RemoteTransport):
480
 
    """Connection to smart server over plain tcp with the client hard-coded to
481
 
    assume protocol v2 and remote server version <= 1.6.
482
 
 
483
 
    This should only be used for testing.
484
 
    """
485
 
 
486
 
    def _build_medium(self):
487
 
        client_medium = medium.SmartTCPClientMedium(
488
 
            self._host, self._port, self.base)
489
 
        client_medium._protocol_version = 2
490
 
        client_medium._remember_remote_is_before((1, 6))
491
 
        return client_medium, None
492
 
 
493
 
 
494
 
class RemoteSSHTransport(RemoteTransport):
495
 
    """Connection to smart server over SSH.
496
 
 
497
 
    This is essentially just a factory to get 'RemoteTransport(url,
498
 
        SmartSSHClientMedium).
499
 
    """
500
 
 
501
 
    def _build_medium(self):
502
 
        location_config = config.LocationConfig(self.base)
503
 
        bzr_remote_path = location_config.get_bzr_remote_path()
504
 
        user = self._user
505
 
        if user is None:
506
 
            auth = config.AuthenticationConfig()
507
 
            user = auth.get_user('ssh', self._host, self._port)
508
 
        client_medium = medium.SmartSSHClientMedium(self._host, self._port,
509
 
            user, self._password, self.base,
510
 
            bzr_remote_path=bzr_remote_path)
511
 
        return client_medium, (user, self._password)
512
 
 
513
 
 
514
 
class RemoteHTTPTransport(RemoteTransport):
515
 
    """Just a way to connect between a bzr+http:// url and http://.
516
 
    
517
 
    This connection operates slightly differently than the RemoteSSHTransport.
518
 
    It uses a plain http:// transport underneath, which defines what remote
519
 
    .bzr/smart URL we are connected to. From there, all paths that are sent are
520
 
    sent as relative paths, this way, the remote side can properly
521
 
    de-reference them, since it is likely doing rewrite rules to translate an
522
 
    HTTP path into a local path.
523
 
    """
524
 
 
525
 
    def __init__(self, base, _from_transport=None, http_transport=None):
526
 
        if http_transport is None:
527
 
            # FIXME: the password may be lost here because it appears in the
528
 
            # url only for an intial construction (when the url came from the
529
 
            # command-line).
530
 
            http_url = base[len('bzr+'):]
531
 
            self._http_transport = transport.get_transport(http_url)
532
 
        else:
533
 
            self._http_transport = http_transport
534
 
        super(RemoteHTTPTransport, self).__init__(
535
 
            base, _from_transport=_from_transport)
536
 
 
537
 
    def _build_medium(self):
538
 
        # We let http_transport take care of the credentials
539
 
        return self._http_transport.get_smart_medium(), None
540
 
 
541
 
    def _remote_path(self, relpath):
542
 
        """After connecting, HTTP Transport only deals in relative URLs."""
543
 
        # Adjust the relpath based on which URL this smart transport is
544
 
        # connected to.
545
 
        http_base = urlutils.normalize_url(self.get_smart_medium().base)
546
 
        url = urlutils.join(self.base[len('bzr+'):], relpath)
547
 
        url = urlutils.normalize_url(url)
548
 
        return urlutils.relative_url(http_base, url)
549
 
 
550
 
    def clone(self, relative_url):
551
 
        """Make a new RemoteHTTPTransport related to me.
552
 
 
553
 
        This is re-implemented rather than using the default
554
 
        RemoteTransport.clone() because we must be careful about the underlying
555
 
        http transport.
556
 
 
557
 
        Also, the cloned smart transport will POST to the same .bzr/smart
558
 
        location as this transport (although obviously the relative paths in the
559
 
        smart requests may be different).  This is so that the server doesn't
560
 
        have to handle .bzr/smart requests at arbitrary places inside .bzr
561
 
        directories, just at the initial URL the user uses.
562
 
        """
563
 
        if relative_url:
564
 
            abs_url = self.abspath(relative_url)
565
 
        else:
566
 
            abs_url = self.base
567
 
        return RemoteHTTPTransport(abs_url,
568
 
                                   _from_transport=self,
569
 
                                   http_transport=self._http_transport)
570
 
 
571
 
    def _redirected_to(self, source, target):
572
 
        """See transport._redirected_to"""
573
 
        redirected = self._http_transport._redirected_to(source, target)
574
 
        if (redirected is not None
575
 
            and isinstance(redirected, type(self._http_transport))):
576
 
            return RemoteHTTPTransport('bzr+' + redirected.external_url(),
577
 
                                       http_transport=redirected)
578
 
        else:
579
 
            # Either None or a transport for a different protocol
580
 
            return redirected
581
 
 
582
 
 
583
 
class HintingSSHTransport(transport.Transport):
584
 
    """Simple transport that handles ssh:// and points out bzr+ssh://."""
585
 
 
586
 
    def __init__(self, url):
587
 
        raise errors.UnsupportedProtocol(url, 
588
 
            'bzr supports bzr+ssh to operate over ssh, use "bzr+%s".' % url)
589
 
 
590
 
 
591
 
def get_test_permutations():
592
 
    """Return (transport, server) permutations for testing."""
593
 
    ### We may need a little more test framework support to construct an
594
 
    ### appropriate RemoteTransport in the future.
595
 
    from bzrlib.smart import server
596
 
    return [(RemoteTCPTransport, server.SmartTCPServer_for_testing)]