~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.py

  • Committer: Vincent Ladeuil
  • Date: 2010-03-10 09:33:04 UTC
  • mto: (5082.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 5083.
  • Revision ID: v.ladeuil+lp@free.fr-20100310093304-4245t4tazd4sxoav
Cleanup test from overly cautious checks.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2006-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
24
24
bzrlib/transport/smart/__init__.py.
25
25
"""
26
26
 
27
 
from __future__ import absolute_import
28
 
 
29
27
import errno
30
28
import os
 
29
import socket
31
30
import sys
32
 
import time
 
31
import urllib
33
32
 
34
 
import bzrlib
35
33
from bzrlib.lazy_import import lazy_import
36
34
lazy_import(globals(), """
37
 
import select
38
 
import socket
 
35
import atexit
39
36
import thread
40
37
import weakref
41
38
 
42
39
from bzrlib import (
43
40
    debug,
44
41
    errors,
 
42
    symbol_versioning,
45
43
    trace,
46
 
    transport,
47
44
    ui,
48
45
    urlutils,
49
46
    )
50
 
from bzrlib.i18n import gettext
51
 
from bzrlib.smart import client, protocol, request, signals, vfs
 
47
from bzrlib.smart import client, protocol, request, vfs
52
48
from bzrlib.transport import ssh
53
49
""")
 
50
#usually already imported, and getting IllegalScoperReplacer on it here.
54
51
from bzrlib import osutils
55
52
 
56
 
# Throughout this module buffer size parameters are either limited to be at
57
 
# most _MAX_READ_SIZE, or are ignored and _MAX_READ_SIZE is used instead.
58
 
# For this module's purposes, MAX_SOCKET_CHUNK is a reasonable size for reads
59
 
# from non-sockets as well.
60
 
_MAX_READ_SIZE = osutils.MAX_SOCKET_CHUNK
 
53
# We must not read any more than 64k at a time so we don't risk "no buffer
 
54
# space available" errors on some platforms.  Windows in particular is likely
 
55
# to give error 10053 or 10055 if we read more than 64k from a socket.
 
56
_MAX_READ_SIZE = 64 * 1024
 
57
 
61
58
 
62
59
def _get_protocol_factory_for_bytes(bytes):
63
60
    """Determine the right protocol factory for 'bytes'.
181
178
        ui.ui_factory.report_transport_activity(self, bytes, direction)
182
179
 
183
180
 
184
 
_bad_file_descriptor = (errno.EBADF,)
185
 
if sys.platform == 'win32':
186
 
    # Given on Windows if you pass a closed socket to select.select. Probably
187
 
    # also given if you pass a file handle to select.
188
 
    WSAENOTSOCK = 10038
189
 
    _bad_file_descriptor += (WSAENOTSOCK,)
190
 
 
191
 
 
192
181
class SmartServerStreamMedium(SmartMedium):
193
182
    """Handles smart commands coming over a stream.
194
183
 
207
196
        the stream.  See also the _push_back method.
208
197
    """
209
198
 
210
 
    _timer = time.time
211
 
 
212
 
    def __init__(self, backing_transport, root_client_path='/', timeout=None):
 
199
    def __init__(self, backing_transport, root_client_path='/'):
213
200
        """Construct new server.
214
201
 
215
202
        :param backing_transport: Transport for the directory served.
218
205
        self.backing_transport = backing_transport
219
206
        self.root_client_path = root_client_path
220
207
        self.finished = False
221
 
        if timeout is None:
222
 
            raise AssertionError('You must supply a timeout.')
223
 
        self._client_timeout = timeout
224
 
        self._client_poll_timeout = min(timeout / 10.0, 1.0)
225
208
        SmartMedium.__init__(self)
226
209
 
227
210
    def serve(self):
233
216
            while not self.finished:
234
217
                server_protocol = self._build_protocol()
235
218
                self._serve_one_request(server_protocol)
236
 
        except errors.ConnectionTimeout, e:
237
 
            trace.note('%s' % (e,))
238
 
            trace.log_exception_quietly()
239
 
            self._disconnect_client()
240
 
            # We reported it, no reason to make a big fuss.
241
 
            return
242
219
        except Exception, e:
243
220
            stderr.write("%s terminating on exception %s\n" % (self, e))
244
221
            raise
245
 
        self._disconnect_client()
246
 
 
247
 
    def _stop_gracefully(self):
248
 
        """When we finish this message, stop looking for more."""
249
 
        trace.mutter('Stopping %s' % (self,))
250
 
        self.finished = True
251
 
 
252
 
    def _disconnect_client(self):
253
 
        """Close the current connection. We stopped due to a timeout/etc."""
254
 
        # The default implementation is a no-op, because that is all we used to
255
 
        # do when disconnecting from a client. I suppose we never had the
256
 
        # *server* initiate a disconnect, before
257
 
 
258
 
    def _wait_for_bytes_with_timeout(self, timeout_seconds):
259
 
        """Wait for more bytes to be read, but timeout if none available.
260
 
 
261
 
        This allows us to detect idle connections, and stop trying to read from
262
 
        them, without setting the socket itself to non-blocking. This also
263
 
        allows us to specify when we watch for idle timeouts.
264
 
 
265
 
        :return: Did we timeout? (True if we timed out, False if there is data
266
 
            to be read)
267
 
        """
268
 
        raise NotImplementedError(self._wait_for_bytes_with_timeout)
269
222
 
270
223
    def _build_protocol(self):
271
224
        """Identifies the version of the incoming request, and returns an
276
229
 
277
230
        :returns: a SmartServerRequestProtocol.
278
231
        """
279
 
        self._wait_for_bytes_with_timeout(self._client_timeout)
280
 
        if self.finished:
281
 
            # We're stopping, so don't try to do any more work
282
 
            return None
283
232
        bytes = self._get_line()
284
233
        protocol_factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
285
234
        protocol = protocol_factory(
287
236
        protocol.accept_bytes(unused_bytes)
288
237
        return protocol
289
238
 
290
 
    def _wait_on_descriptor(self, fd, timeout_seconds):
291
 
        """select() on a file descriptor, waiting for nonblocking read()
292
 
 
293
 
        This will raise a ConnectionTimeout exception if we do not get a
294
 
        readable handle before timeout_seconds.
295
 
        :return: None
296
 
        """
297
 
        t_end = self._timer() + timeout_seconds
298
 
        poll_timeout = min(timeout_seconds, self._client_poll_timeout)
299
 
        rs = xs = None
300
 
        while not rs and not xs and self._timer() < t_end:
301
 
            if self.finished:
302
 
                return
303
 
            try:
304
 
                rs, _, xs = select.select([fd], [], [fd], poll_timeout)
305
 
            except (select.error, socket.error) as e:
306
 
                err = getattr(e, 'errno', None)
307
 
                if err is None and getattr(e, 'args', None) is not None:
308
 
                    # select.error doesn't have 'errno', it just has args[0]
309
 
                    err = e.args[0]
310
 
                if err in _bad_file_descriptor:
311
 
                    return # Not a socket indicates read() will fail
312
 
                elif err == errno.EINTR:
313
 
                    # Interrupted, keep looping.
314
 
                    continue
315
 
                raise
316
 
        if rs or xs:
317
 
            return
318
 
        raise errors.ConnectionTimeout('disconnecting client after %.1f seconds'
319
 
                                       % (timeout_seconds,))
320
 
 
321
239
    def _serve_one_request(self, protocol):
322
240
        """Read one request from input, process, send back a response.
323
241
 
324
242
        :param protocol: a SmartServerRequestProtocol.
325
243
        """
326
 
        if protocol is None:
327
 
            return
328
244
        try:
329
245
            self._serve_one_request_unguarded(protocol)
330
246
        except KeyboardInterrupt:
346
262
 
347
263
class SmartServerSocketStreamMedium(SmartServerStreamMedium):
348
264
 
349
 
    def __init__(self, sock, backing_transport, root_client_path='/',
350
 
                 timeout=None):
 
265
    def __init__(self, sock, backing_transport, root_client_path='/'):
351
266
        """Constructor.
352
267
 
353
268
        :param sock: the socket the server will read from.  It will be put
354
269
            into blocking mode.
355
270
        """
356
271
        SmartServerStreamMedium.__init__(
357
 
            self, backing_transport, root_client_path=root_client_path,
358
 
            timeout=timeout)
 
272
            self, backing_transport, root_client_path=root_client_path)
359
273
        sock.setblocking(True)
360
274
        self.socket = sock
361
 
        # Get the getpeername now, as we might be closed later when we care.
362
 
        try:
363
 
            self._client_info = sock.getpeername()
364
 
        except socket.error:
365
 
            self._client_info = '<unknown>'
366
 
 
367
 
    def __str__(self):
368
 
        return '%s(client=%s)' % (self.__class__.__name__, self._client_info)
369
 
 
370
 
    def __repr__(self):
371
 
        return '%s.%s(client=%s)' % (self.__module__, self.__class__.__name__,
372
 
            self._client_info)
373
275
 
374
276
    def _serve_one_request_unguarded(self, protocol):
375
277
        while protocol.next_read_size():
376
278
            # We can safely try to read large chunks.  If there is less data
377
 
            # than MAX_SOCKET_CHUNK ready, the socket will just return a
378
 
            # short read immediately rather than block.
379
 
            bytes = self.read_bytes(osutils.MAX_SOCKET_CHUNK)
 
279
            # than _MAX_READ_SIZE ready, the socket wil just return a short
 
280
            # read immediately rather than block.
 
281
            bytes = self.read_bytes(_MAX_READ_SIZE)
380
282
            if bytes == '':
381
283
                self.finished = True
382
284
                return
384
286
 
385
287
        self._push_back(protocol.unused_data)
386
288
 
387
 
    def _disconnect_client(self):
388
 
        """Close the current connection. We stopped due to a timeout/etc."""
389
 
        self.socket.close()
390
 
 
391
 
    def _wait_for_bytes_with_timeout(self, timeout_seconds):
392
 
        """Wait for more bytes to be read, but timeout if none available.
393
 
 
394
 
        This allows us to detect idle connections, and stop trying to read from
395
 
        them, without setting the socket itself to non-blocking. This also
396
 
        allows us to specify when we watch for idle timeouts.
397
 
 
398
 
        :return: None, this will raise ConnectionTimeout if we time out before
399
 
            data is available.
400
 
        """
401
 
        return self._wait_on_descriptor(self.socket, timeout_seconds)
402
 
 
403
289
    def _read_bytes(self, desired_count):
404
 
        return osutils.read_bytes_from_socket(
405
 
            self.socket, self._report_activity)
 
290
        return _read_bytes_from_socket(
 
291
            self.socket.recv, desired_count, self._report_activity)
406
292
 
407
293
    def terminate_due_to_error(self):
408
294
        # TODO: This should log to a server log file, but no such thing
409
295
        # exists yet.  Andrew Bennetts 2006-09-29.
410
 
        self.socket.close()
 
296
        osutils.until_no_eintr(self.socket.close)
411
297
        self.finished = True
412
298
 
413
299
    def _write_out(self, bytes):
422
308
 
423
309
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
424
310
 
425
 
    def __init__(self, in_file, out_file, backing_transport, timeout=None):
 
311
    def __init__(self, in_file, out_file, backing_transport):
426
312
        """Construct new server.
427
313
 
428
314
        :param in_file: Python file from which requests can be read.
429
315
        :param out_file: Python file to write responses.
430
316
        :param backing_transport: Transport for the directory served.
431
317
        """
432
 
        SmartServerStreamMedium.__init__(self, backing_transport,
433
 
            timeout=timeout)
 
318
        SmartServerStreamMedium.__init__(self, backing_transport)
434
319
        if sys.platform == 'win32':
435
320
            # force binary mode for files
436
321
            import msvcrt
441
326
        self._in = in_file
442
327
        self._out = out_file
443
328
 
444
 
    def serve(self):
445
 
        """See SmartServerStreamMedium.serve"""
446
 
        # This is the regular serve, except it adds signal trapping for soft
447
 
        # shutdown.
448
 
        stop_gracefully = self._stop_gracefully
449
 
        signals.register_on_hangup(id(self), stop_gracefully)
450
 
        try:
451
 
            return super(SmartServerPipeStreamMedium, self).serve()
452
 
        finally:
453
 
            signals.unregister_on_hangup(id(self))
454
 
 
455
329
    def _serve_one_request_unguarded(self, protocol):
456
330
        while True:
457
331
            # We need to be careful not to read past the end of the current
460
334
            bytes_to_read = protocol.next_read_size()
461
335
            if bytes_to_read == 0:
462
336
                # Finished serving this request.
463
 
                self._out.flush()
 
337
                osutils.until_no_eintr(self._out.flush)
464
338
                return
465
339
            bytes = self.read_bytes(bytes_to_read)
466
340
            if bytes == '':
467
341
                # Connection has been closed.
468
342
                self.finished = True
469
 
                self._out.flush()
 
343
                osutils.until_no_eintr(self._out.flush)
470
344
                return
471
345
            protocol.accept_bytes(bytes)
472
346
 
473
 
    def _disconnect_client(self):
474
 
        self._in.close()
475
 
        self._out.flush()
476
 
        self._out.close()
477
 
 
478
 
    def _wait_for_bytes_with_timeout(self, timeout_seconds):
479
 
        """Wait for more bytes to be read, but timeout if none available.
480
 
 
481
 
        This allows us to detect idle connections, and stop trying to read from
482
 
        them, without setting the socket itself to non-blocking. This also
483
 
        allows us to specify when we watch for idle timeouts.
484
 
 
485
 
        :return: None, this will raise ConnectionTimeout if we time out before
486
 
            data is available.
487
 
        """
488
 
        if (getattr(self._in, 'fileno', None) is None
489
 
            or sys.platform == 'win32'):
490
 
            # You can't select() file descriptors on Windows.
491
 
            return
492
 
        return self._wait_on_descriptor(self._in, timeout_seconds)
493
 
 
494
347
    def _read_bytes(self, desired_count):
495
 
        return self._in.read(desired_count)
 
348
        return osutils.until_no_eintr(self._in.read, desired_count)
496
349
 
497
350
    def terminate_due_to_error(self):
498
351
        # TODO: This should log to a server log file, but no such thing
499
352
        # exists yet.  Andrew Bennetts 2006-09-29.
500
 
        self._out.close()
 
353
        osutils.until_no_eintr(self._out.close)
501
354
        self.finished = True
502
355
 
503
356
    def _write_out(self, bytes):
504
 
        self._out.write(bytes)
 
357
        osutils.until_no_eintr(self._out.write, bytes)
505
358
 
506
359
 
507
360
class SmartClientMediumRequest(object):
640
493
        return self._medium._get_line()
641
494
 
642
495
 
643
 
class _VfsRefuser(object):
644
 
    """An object that refuses all VFS requests.
645
 
 
646
 
    """
647
 
 
648
 
    def __init__(self):
649
 
        client._SmartClient.hooks.install_named_hook(
650
 
            'call', self.check_vfs, 'vfs refuser')
651
 
 
652
 
    def check_vfs(self, params):
653
 
        try:
654
 
            request_method = request.request_handlers.get(params.method)
655
 
        except KeyError:
656
 
            # A method we don't know about doesn't count as a VFS method.
657
 
            return
658
 
        if issubclass(request_method, vfs.VfsRequest):
659
 
            raise errors.HpssVfsRequestNotAllowed(params.method, params.args)
660
 
 
661
 
 
662
496
class _DebugCounter(object):
663
497
    """An object that counts the HPSS calls made to each client medium.
664
498
 
665
 
    When a medium is garbage-collected, or failing that when
666
 
    bzrlib.global_state exits, the total number of calls made on that medium
667
 
    are reported via trace.note.
 
499
    When a medium is garbage-collected, or failing that when atexit functions
 
500
    are run, the total number of calls made on that medium are reported via
 
501
    trace.note.
668
502
    """
669
503
 
670
504
    def __init__(self):
671
505
        self.counts = weakref.WeakKeyDictionary()
672
506
        client._SmartClient.hooks.install_named_hook(
673
507
            'call', self.increment_call_count, 'hpss call counter')
674
 
        bzrlib.global_state.cleanups.add_cleanup(self.flush_all)
 
508
        atexit.register(self.flush_all)
675
509
 
676
510
    def track(self, medium):
677
511
        """Start tracking calls made to a medium.
711
545
        value['count'] = 0
712
546
        value['vfs_count'] = 0
713
547
        if count != 0:
714
 
            trace.note(gettext('HPSS calls: {0} ({1} vfs) {2}').format(
715
 
                       count, vfs_count, medium_repr))
 
548
            trace.note('HPSS calls: %d (%d vfs) %s',
 
549
                       count, vfs_count, medium_repr)
716
550
 
717
551
    def flush_all(self):
718
552
        for ref in list(self.counts.keys()):
719
553
            self.done(ref)
720
554
 
721
555
_debug_counter = None
722
 
_vfs_refuser = None
723
556
 
724
557
 
725
558
class SmartClientMedium(SmartMedium):
742
575
            if _debug_counter is None:
743
576
                _debug_counter = _DebugCounter()
744
577
            _debug_counter.track(self)
745
 
        if 'hpss_client_no_vfs' in debug.debug_flags:
746
 
            global _vfs_refuser
747
 
            if _vfs_refuser is None:
748
 
                _vfs_refuser = _VfsRefuser()
749
578
 
750
579
    def _is_remote_before(self, version_tuple):
751
580
        """Is it possible the remote side supports RPCs for a given version?
780
609
            # which is newer than a previously supplied older-than version.
781
610
            # This indicates that some smart verb call is not guarded
782
611
            # appropriately (it should simply not have been tried).
783
 
            trace.mutter(
 
612
            raise AssertionError(
784
613
                "_remember_remote_is_before(%r) called, but "
785
614
                "_remember_remote_is_before(%r) was called previously."
786
 
                , version_tuple, self._remote_version_is_before)
787
 
            if 'hpss' in debug.debug_flags:
788
 
                ui.ui_factory.show_warning(
789
 
                    "_remember_remote_is_before(%r) called, but "
790
 
                    "_remember_remote_is_before(%r) was called previously."
791
 
                    % (version_tuple, self._remote_version_is_before))
792
 
            return
 
615
                % (version_tuple, self._remote_version_is_before))
793
616
        self._remote_version_is_before = version_tuple
794
617
 
795
618
    def protocol_version(self):
842
665
        """
843
666
        medium_base = urlutils.join(self.base, '/')
844
667
        rel_url = urlutils.relative_url(medium_base, transport.base)
845
 
        return urlutils.unquote(rel_url)
 
668
        return urllib.unquote(rel_url)
846
669
 
847
670
 
848
671
class SmartClientStreamMedium(SmartClientMedium):
883
706
        """
884
707
        return SmartClientStreamMediumRequest(self)
885
708
 
886
 
    def reset(self):
887
 
        """We have been disconnected, reset current state.
888
 
 
889
 
        This resets things like _current_request and connected state.
890
 
        """
891
 
        self.disconnect()
892
 
        self._current_request = None
893
 
 
894
709
 
895
710
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
896
711
    """A client medium using simple pipes.
905
720
 
906
721
    def _accept_bytes(self, bytes):
907
722
        """See SmartClientStreamMedium.accept_bytes."""
908
 
        try:
909
 
            self._writeable_pipe.write(bytes)
910
 
        except IOError, e:
911
 
            if e.errno in (errno.EINVAL, errno.EPIPE):
912
 
                raise errors.ConnectionReset(
913
 
                    "Error trying to write to subprocess", e)
914
 
            raise
 
723
        osutils.until_no_eintr(self._writeable_pipe.write, bytes)
915
724
        self._report_activity(len(bytes), 'write')
916
725
 
917
726
    def _flush(self):
918
727
        """See SmartClientStreamMedium._flush()."""
919
 
        # Note: If flush were to fail, we'd like to raise ConnectionReset, etc.
920
 
        #       However, testing shows that even when the child process is
921
 
        #       gone, this doesn't error.
922
 
        self._writeable_pipe.flush()
 
728
        osutils.until_no_eintr(self._writeable_pipe.flush)
923
729
 
924
730
    def _read_bytes(self, count):
925
731
        """See SmartClientStreamMedium._read_bytes."""
926
 
        bytes_to_read = min(count, _MAX_READ_SIZE)
927
 
        bytes = self._readable_pipe.read(bytes_to_read)
 
732
        bytes = osutils.until_no_eintr(self._readable_pipe.read, count)
928
733
        self._report_activity(len(bytes), 'read')
929
734
        return bytes
930
735
 
931
736
 
932
 
class SSHParams(object):
933
 
    """A set of parameters for starting a remote bzr via SSH."""
 
737
class SmartSSHClientMedium(SmartClientStreamMedium):
 
738
    """A client medium using SSH."""
934
739
 
935
740
    def __init__(self, host, port=None, username=None, password=None,
936
 
            bzr_remote_path='bzr'):
937
 
        self.host = host
938
 
        self.port = port
939
 
        self.username = username
940
 
        self.password = password
941
 
        self.bzr_remote_path = bzr_remote_path
942
 
 
943
 
 
944
 
class SmartSSHClientMedium(SmartClientStreamMedium):
945
 
    """A client medium using SSH.
946
 
 
947
 
    It delegates IO to a SmartSimplePipesClientMedium or
948
 
    SmartClientAlreadyConnectedSocketMedium (depending on platform).
949
 
    """
950
 
 
951
 
    def __init__(self, base, ssh_params, vendor=None):
 
741
            base=None, vendor=None, bzr_remote_path=None):
952
742
        """Creates a client that will connect on the first use.
953
743
 
954
 
        :param ssh_params: A SSHParams instance.
955
744
        :param vendor: An optional override for the ssh vendor to use. See
956
745
            bzrlib.transport.ssh for details on ssh vendors.
957
746
        """
958
 
        self._real_medium = None
959
 
        self._ssh_params = ssh_params
 
747
        self._connected = False
 
748
        self._host = host
 
749
        self._password = password
 
750
        self._port = port
 
751
        self._username = username
960
752
        # for the benefit of progress making a short description of this
961
753
        # transport
962
754
        self._scheme = 'bzr+ssh'
964
756
        # _DebugCounter so we have to store all the values used in our repr
965
757
        # method before calling the super init.
966
758
        SmartClientStreamMedium.__init__(self, base)
 
759
        self._read_from = None
 
760
        self._ssh_connection = None
967
761
        self._vendor = vendor
968
 
        self._ssh_connection = None
 
762
        self._write_to = None
 
763
        self._bzr_remote_path = bzr_remote_path
969
764
 
970
765
    def __repr__(self):
971
 
        if self._ssh_params.port is None:
 
766
        if self._port is None:
972
767
            maybe_port = ''
973
768
        else:
974
 
            maybe_port = ':%s' % self._ssh_params.port
975
 
        if self._ssh_params.username is None:
976
 
            maybe_user = ''
977
 
        else:
978
 
            maybe_user = '%s@' % self._ssh_params.username
979
 
        return "%s(%s://%s%s%s/)" % (
 
769
            maybe_port = ':%s' % self._port
 
770
        return "%s(%s://%s@%s%s/)" % (
980
771
            self.__class__.__name__,
981
772
            self._scheme,
982
 
            maybe_user,
983
 
            self._ssh_params.host,
 
773
            self._username,
 
774
            self._host,
984
775
            maybe_port)
985
776
 
986
777
    def _accept_bytes(self, bytes):
987
778
        """See SmartClientStreamMedium.accept_bytes."""
988
779
        self._ensure_connection()
989
 
        self._real_medium.accept_bytes(bytes)
 
780
        osutils.until_no_eintr(self._write_to.write, bytes)
 
781
        self._report_activity(len(bytes), 'write')
990
782
 
991
783
    def disconnect(self):
992
784
        """See SmartClientMedium.disconnect()."""
993
 
        if self._real_medium is not None:
994
 
            self._real_medium.disconnect()
995
 
            self._real_medium = None
996
 
        if self._ssh_connection is not None:
997
 
            self._ssh_connection.close()
998
 
            self._ssh_connection = None
 
785
        if not self._connected:
 
786
            return
 
787
        osutils.until_no_eintr(self._read_from.close)
 
788
        osutils.until_no_eintr(self._write_to.close)
 
789
        self._ssh_connection.close()
 
790
        self._connected = False
999
791
 
1000
792
    def _ensure_connection(self):
1001
793
        """Connect this medium if not already connected."""
1002
 
        if self._real_medium is not None:
 
794
        if self._connected:
1003
795
            return
1004
796
        if self._vendor is None:
1005
797
            vendor = ssh._get_ssh_vendor()
1006
798
        else:
1007
799
            vendor = self._vendor
1008
 
        self._ssh_connection = vendor.connect_ssh(self._ssh_params.username,
1009
 
                self._ssh_params.password, self._ssh_params.host,
1010
 
                self._ssh_params.port,
1011
 
                command=[self._ssh_params.bzr_remote_path, 'serve', '--inet',
 
800
        self._ssh_connection = vendor.connect_ssh(self._username,
 
801
                self._password, self._host, self._port,
 
802
                command=[self._bzr_remote_path, 'serve', '--inet',
1012
803
                         '--directory=/', '--allow-writes'])
1013
 
        io_kind, io_object = self._ssh_connection.get_sock_or_pipes()
1014
 
        if io_kind == 'socket':
1015
 
            self._real_medium = SmartClientAlreadyConnectedSocketMedium(
1016
 
                self.base, io_object)
1017
 
        elif io_kind == 'pipes':
1018
 
            read_from, write_to = io_object
1019
 
            self._real_medium = SmartSimplePipesClientMedium(
1020
 
                read_from, write_to, self.base)
1021
 
        else:
1022
 
            raise AssertionError(
1023
 
                "Unexpected io_kind %r from %r"
1024
 
                % (io_kind, self._ssh_connection))
1025
 
        for hook in transport.Transport.hooks["post_connect"]:
1026
 
            hook(self)
 
804
        self._read_from, self._write_to = \
 
805
            self._ssh_connection.get_filelike_channels()
 
806
        self._connected = True
1027
807
 
1028
808
    def _flush(self):
1029
809
        """See SmartClientStreamMedium._flush()."""
1030
 
        self._real_medium._flush()
 
810
        self._write_to.flush()
1031
811
 
1032
812
    def _read_bytes(self, count):
1033
813
        """See SmartClientStreamMedium.read_bytes."""
1034
 
        if self._real_medium is None:
 
814
        if not self._connected:
1035
815
            raise errors.MediumNotConnected(self)
1036
 
        return self._real_medium.read_bytes(count)
 
816
        bytes_to_read = min(count, _MAX_READ_SIZE)
 
817
        bytes = osutils.until_no_eintr(self._read_from.read, bytes_to_read)
 
818
        self._report_activity(len(bytes), 'read')
 
819
        return bytes
1037
820
 
1038
821
 
1039
822
# Port 4155 is the default port for bzr://, registered with IANA.
1041
824
BZR_DEFAULT_PORT = 4155
1042
825
 
1043
826
 
1044
 
class SmartClientSocketMedium(SmartClientStreamMedium):
1045
 
    """A client medium using a socket.
1046
 
 
1047
 
    This class isn't usable directly.  Use one of its subclasses instead.
1048
 
    """
1049
 
 
1050
 
    def __init__(self, base):
 
827
class SmartTCPClientMedium(SmartClientStreamMedium):
 
828
    """A client medium using TCP."""
 
829
 
 
830
    def __init__(self, host, port, base):
 
831
        """Creates a client that will connect on the first use."""
1051
832
        SmartClientStreamMedium.__init__(self, base)
 
833
        self._connected = False
 
834
        self._host = host
 
835
        self._port = port
1052
836
        self._socket = None
1053
 
        self._connected = False
1054
837
 
1055
838
    def _accept_bytes(self, bytes):
1056
839
        """See SmartClientMedium.accept_bytes."""
1057
840
        self._ensure_connection()
1058
841
        osutils.send_all(self._socket, bytes, self._report_activity)
1059
842
 
1060
 
    def _ensure_connection(self):
1061
 
        """Connect this medium if not already connected."""
1062
 
        raise NotImplementedError(self._ensure_connection)
1063
 
 
1064
 
    def _flush(self):
1065
 
        """See SmartClientStreamMedium._flush().
1066
 
 
1067
 
        For sockets we do no flushing. For TCP sockets we may want to turn off
1068
 
        TCP_NODELAY and add a means to do a flush, but that can be done in the
1069
 
        future.
1070
 
        """
1071
 
 
1072
 
    def _read_bytes(self, count):
1073
 
        """See SmartClientMedium.read_bytes."""
1074
 
        if not self._connected:
1075
 
            raise errors.MediumNotConnected(self)
1076
 
        return osutils.read_bytes_from_socket(
1077
 
            self._socket, self._report_activity)
1078
 
 
1079
843
    def disconnect(self):
1080
844
        """See SmartClientMedium.disconnect()."""
1081
845
        if not self._connected:
1082
846
            return
1083
 
        self._socket.close()
 
847
        osutils.until_no_eintr(self._socket.close)
1084
848
        self._socket = None
1085
849
        self._connected = False
1086
850
 
1087
 
 
1088
 
class SmartTCPClientMedium(SmartClientSocketMedium):
1089
 
    """A client medium that creates a TCP connection."""
1090
 
 
1091
 
    def __init__(self, host, port, base):
1092
 
        """Creates a client that will connect on the first use."""
1093
 
        SmartClientSocketMedium.__init__(self, base)
1094
 
        self._host = host
1095
 
        self._port = port
1096
 
 
1097
851
    def _ensure_connection(self):
1098
852
        """Connect this medium if not already connected."""
1099
853
        if self._connected:
1132
886
            raise errors.ConnectionError("failed to connect to %s:%d: %s" %
1133
887
                    (self._host, port, err_msg))
1134
888
        self._connected = True
1135
 
        for hook in transport.Transport.hooks["post_connect"]:
1136
 
            hook(self)
1137
 
 
1138
 
 
1139
 
class SmartClientAlreadyConnectedSocketMedium(SmartClientSocketMedium):
1140
 
    """A client medium for an already connected socket.
1141
 
    
1142
 
    Note that this class will assume it "owns" the socket, so it will close it
1143
 
    when its disconnect method is called.
1144
 
    """
1145
 
 
1146
 
    def __init__(self, base, sock):
1147
 
        SmartClientSocketMedium.__init__(self, base)
1148
 
        self._socket = sock
1149
 
        self._connected = True
1150
 
 
1151
 
    def _ensure_connection(self):
1152
 
        # Already connected, by definition!  So nothing to do.
1153
 
        pass
 
889
 
 
890
    def _flush(self):
 
891
        """See SmartClientStreamMedium._flush().
 
892
 
 
893
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and
 
894
        add a means to do a flush, but that can be done in the future.
 
895
        """
 
896
 
 
897
    def _read_bytes(self, count):
 
898
        """See SmartClientMedium.read_bytes."""
 
899
        if not self._connected:
 
900
            raise errors.MediumNotConnected(self)
 
901
        return _read_bytes_from_socket(
 
902
            self._socket.recv, count, self._report_activity)
1154
903
 
1155
904
 
1156
905
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
1191
940
        This invokes self._medium._flush to ensure all bytes are transmitted.
1192
941
        """
1193
942
        self._medium._flush()
 
943
 
 
944
 
 
945
def _read_bytes_from_socket(sock, desired_count, report_activity):
 
946
    # We ignore the desired_count because on sockets it's more efficient to
 
947
    # read large chunks (of _MAX_READ_SIZE bytes) at a time.
 
948
    try:
 
949
        bytes = osutils.until_no_eintr(sock, _MAX_READ_SIZE)
 
950
    except socket.error, e:
 
951
        if len(e.args) and e.args[0] in (errno.ECONNRESET, 10054):
 
952
            # The connection was closed by the other side.  Callers expect an
 
953
            # empty string to signal end-of-stream.
 
954
            bytes = ''
 
955
        else:
 
956
            raise
 
957
    else:
 
958
        report_activity(len(bytes), 'read')
 
959
    return bytes
 
960