/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

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
 
import errno
30
27
import os
31
28
import sys
32
 
import time
 
29
import urllib
33
30
 
34
 
import bzrlib
35
31
from bzrlib.lazy_import import lazy_import
36
32
lazy_import(globals(), """
37
 
import select
 
33
import atexit
38
34
import socket
39
35
import thread
40
36
import weakref
42
38
from bzrlib import (
43
39
    debug,
44
40
    errors,
 
41
    symbol_versioning,
45
42
    trace,
46
 
    transport,
47
43
    ui,
48
44
    urlutils,
49
45
    )
50
 
from bzrlib.i18n import gettext
51
 
from bzrlib.smart import client, protocol, request, signals, vfs
 
46
from bzrlib.smart import client, protocol, request, vfs
52
47
from bzrlib.transport import ssh
53
48
""")
54
49
from bzrlib import osutils
181
176
        ui.ui_factory.report_transport_activity(self, bytes, direction)
182
177
 
183
178
 
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
179
class SmartServerStreamMedium(SmartMedium):
193
180
    """Handles smart commands coming over a stream.
194
181
 
207
194
        the stream.  See also the _push_back method.
208
195
    """
209
196
 
210
 
    _timer = time.time
211
 
 
212
 
    def __init__(self, backing_transport, root_client_path='/', timeout=None):
 
197
    def __init__(self, backing_transport, root_client_path='/'):
213
198
        """Construct new server.
214
199
 
215
200
        :param backing_transport: Transport for the directory served.
218
203
        self.backing_transport = backing_transport
219
204
        self.root_client_path = root_client_path
220
205
        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
206
        SmartMedium.__init__(self)
226
207
 
227
208
    def serve(self):
233
214
            while not self.finished:
234
215
                server_protocol = self._build_protocol()
235
216
                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
217
        except Exception, e:
243
218
            stderr.write("%s terminating on exception %s\n" % (self, e))
244
219
            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
220
 
270
221
    def _build_protocol(self):
271
222
        """Identifies the version of the incoming request, and returns an
276
227
 
277
228
        :returns: a SmartServerRequestProtocol.
278
229
        """
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
230
        bytes = self._get_line()
284
231
        protocol_factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
285
232
        protocol = protocol_factory(
287
234
        protocol.accept_bytes(unused_bytes)
288
235
        return protocol
289
236
 
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
237
    def _serve_one_request(self, protocol):
322
238
        """Read one request from input, process, send back a response.
323
239
 
324
240
        :param protocol: a SmartServerRequestProtocol.
325
241
        """
326
 
        if protocol is None:
327
 
            return
328
242
        try:
329
243
            self._serve_one_request_unguarded(protocol)
330
244
        except KeyboardInterrupt:
346
260
 
347
261
class SmartServerSocketStreamMedium(SmartServerStreamMedium):
348
262
 
349
 
    def __init__(self, sock, backing_transport, root_client_path='/',
350
 
                 timeout=None):
 
263
    def __init__(self, sock, backing_transport, root_client_path='/'):
351
264
        """Constructor.
352
265
 
353
266
        :param sock: the socket the server will read from.  It will be put
354
267
            into blocking mode.
355
268
        """
356
269
        SmartServerStreamMedium.__init__(
357
 
            self, backing_transport, root_client_path=root_client_path,
358
 
            timeout=timeout)
 
270
            self, backing_transport, root_client_path=root_client_path)
359
271
        sock.setblocking(True)
360
272
        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
273
 
374
274
    def _serve_one_request_unguarded(self, protocol):
375
275
        while protocol.next_read_size():
384
284
 
385
285
        self._push_back(protocol.unused_data)
386
286
 
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
287
    def _read_bytes(self, desired_count):
404
288
        return osutils.read_bytes_from_socket(
405
289
            self.socket, self._report_activity)
422
306
 
423
307
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
424
308
 
425
 
    def __init__(self, in_file, out_file, backing_transport, timeout=None):
 
309
    def __init__(self, in_file, out_file, backing_transport):
426
310
        """Construct new server.
427
311
 
428
312
        :param in_file: Python file from which requests can be read.
429
313
        :param out_file: Python file to write responses.
430
314
        :param backing_transport: Transport for the directory served.
431
315
        """
432
 
        SmartServerStreamMedium.__init__(self, backing_transport,
433
 
            timeout=timeout)
 
316
        SmartServerStreamMedium.__init__(self, backing_transport)
434
317
        if sys.platform == 'win32':
435
318
            # force binary mode for files
436
319
            import msvcrt
441
324
        self._in = in_file
442
325
        self._out = out_file
443
326
 
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
327
    def _serve_one_request_unguarded(self, protocol):
456
328
        while True:
457
329
            # We need to be careful not to read past the end of the current
470
342
                return
471
343
            protocol.accept_bytes(bytes)
472
344
 
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
345
    def _read_bytes(self, desired_count):
495
346
        return self._in.read(desired_count)
496
347
 
640
491
        return self._medium._get_line()
641
492
 
642
493
 
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
494
class _DebugCounter(object):
663
495
    """An object that counts the HPSS calls made to each client medium.
664
496
 
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.
 
497
    When a medium is garbage-collected, or failing that when atexit functions
 
498
    are run, the total number of calls made on that medium are reported via
 
499
    trace.note.
668
500
    """
669
501
 
670
502
    def __init__(self):
671
503
        self.counts = weakref.WeakKeyDictionary()
672
504
        client._SmartClient.hooks.install_named_hook(
673
505
            'call', self.increment_call_count, 'hpss call counter')
674
 
        bzrlib.global_state.cleanups.add_cleanup(self.flush_all)
 
506
        atexit.register(self.flush_all)
675
507
 
676
508
    def track(self, medium):
677
509
        """Start tracking calls made to a medium.
711
543
        value['count'] = 0
712
544
        value['vfs_count'] = 0
713
545
        if count != 0:
714
 
            trace.note(gettext('HPSS calls: {0} ({1} vfs) {2}').format(
715
 
                       count, vfs_count, medium_repr))
 
546
            trace.note('HPSS calls: %d (%d vfs) %s',
 
547
                       count, vfs_count, medium_repr)
716
548
 
717
549
    def flush_all(self):
718
550
        for ref in list(self.counts.keys()):
719
551
            self.done(ref)
720
552
 
721
553
_debug_counter = None
722
 
_vfs_refuser = None
723
554
 
724
555
 
725
556
class SmartClientMedium(SmartMedium):
742
573
            if _debug_counter is None:
743
574
                _debug_counter = _DebugCounter()
744
575
            _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
576
 
750
577
    def _is_remote_before(self, version_tuple):
751
578
        """Is it possible the remote side supports RPCs for a given version?
842
669
        """
843
670
        medium_base = urlutils.join(self.base, '/')
844
671
        rel_url = urlutils.relative_url(medium_base, transport.base)
845
 
        return urlutils.unquote(rel_url)
 
672
        return urllib.unquote(rel_url)
846
673
 
847
674
 
848
675
class SmartClientStreamMedium(SmartClientMedium):
883
710
        """
884
711
        return SmartClientStreamMediumRequest(self)
885
712
 
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
713
 
895
714
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
896
715
    """A client medium using simple pipes.
897
716
 
898
717
    This client does not manage the pipes: it assumes they will always be open.
 
718
 
 
719
    Note that if readable_pipe.read might raise IOError or OSError with errno
 
720
    of EINTR, it must be safe to retry the read.  Plain CPython fileobjects
 
721
    (such as used for sys.stdin) are safe.
899
722
    """
900
723
 
901
724
    def __init__(self, readable_pipe, writeable_pipe, base):
905
728
 
906
729
    def _accept_bytes(self, bytes):
907
730
        """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:\n%s" % (e,))
914
 
            raise
 
731
        self._writeable_pipe.write(bytes)
915
732
        self._report_activity(len(bytes), 'write')
916
733
 
917
734
    def _flush(self):
918
735
        """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
736
        self._writeable_pipe.flush()
923
737
 
924
738
    def _read_bytes(self, count):
925
739
        """See SmartClientStreamMedium._read_bytes."""
926
 
        bytes_to_read = min(count, _MAX_READ_SIZE)
927
 
        bytes = self._readable_pipe.read(bytes_to_read)
 
740
        bytes = osutils.until_no_eintr(self._readable_pipe.read, count)
928
741
        self._report_activity(len(bytes), 'read')
929
742
        return bytes
930
743
 
931
744
 
932
 
class SSHParams(object):
933
 
    """A set of parameters for starting a remote bzr via SSH."""
 
745
class SmartSSHClientMedium(SmartClientStreamMedium):
 
746
    """A client medium using SSH."""
934
747
 
935
748
    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):
 
749
            base=None, vendor=None, bzr_remote_path=None):
952
750
        """Creates a client that will connect on the first use.
953
751
 
954
 
        :param ssh_params: A SSHParams instance.
955
752
        :param vendor: An optional override for the ssh vendor to use. See
956
753
            bzrlib.transport.ssh for details on ssh vendors.
957
754
        """
958
 
        self._real_medium = None
959
 
        self._ssh_params = ssh_params
 
755
        self._connected = False
 
756
        self._host = host
 
757
        self._password = password
 
758
        self._port = port
 
759
        self._username = username
960
760
        # for the benefit of progress making a short description of this
961
761
        # transport
962
762
        self._scheme = 'bzr+ssh'
964
764
        # _DebugCounter so we have to store all the values used in our repr
965
765
        # method before calling the super init.
966
766
        SmartClientStreamMedium.__init__(self, base)
 
767
        self._read_from = None
 
768
        self._ssh_connection = None
967
769
        self._vendor = vendor
968
 
        self._ssh_connection = None
 
770
        self._write_to = None
 
771
        self._bzr_remote_path = bzr_remote_path
969
772
 
970
773
    def __repr__(self):
971
 
        if self._ssh_params.port is None:
 
774
        if self._port is None:
972
775
            maybe_port = ''
973
776
        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/)" % (
 
777
            maybe_port = ':%s' % self._port
 
778
        return "%s(%s://%s@%s%s/)" % (
980
779
            self.__class__.__name__,
981
780
            self._scheme,
982
 
            maybe_user,
983
 
            self._ssh_params.host,
 
781
            self._username,
 
782
            self._host,
984
783
            maybe_port)
985
784
 
986
785
    def _accept_bytes(self, bytes):
987
786
        """See SmartClientStreamMedium.accept_bytes."""
988
787
        self._ensure_connection()
989
 
        self._real_medium.accept_bytes(bytes)
 
788
        self._write_to.write(bytes)
 
789
        self._report_activity(len(bytes), 'write')
990
790
 
991
791
    def disconnect(self):
992
792
        """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
 
793
        if not self._connected:
 
794
            return
 
795
        self._read_from.close()
 
796
        self._write_to.close()
 
797
        self._ssh_connection.close()
 
798
        self._connected = False
999
799
 
1000
800
    def _ensure_connection(self):
1001
801
        """Connect this medium if not already connected."""
1002
 
        if self._real_medium is not None:
 
802
        if self._connected:
1003
803
            return
1004
804
        if self._vendor is None:
1005
805
            vendor = ssh._get_ssh_vendor()
1006
806
        else:
1007
807
            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',
 
808
        self._ssh_connection = vendor.connect_ssh(self._username,
 
809
                self._password, self._host, self._port,
 
810
                command=[self._bzr_remote_path, 'serve', '--inet',
1012
811
                         '--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)
 
812
        self._read_from, self._write_to = \
 
813
            self._ssh_connection.get_filelike_channels()
 
814
        self._connected = True
1027
815
 
1028
816
    def _flush(self):
1029
817
        """See SmartClientStreamMedium._flush()."""
1030
 
        self._real_medium._flush()
 
818
        self._write_to.flush()
1031
819
 
1032
820
    def _read_bytes(self, count):
1033
821
        """See SmartClientStreamMedium.read_bytes."""
1034
 
        if self._real_medium is None:
 
822
        if not self._connected:
1035
823
            raise errors.MediumNotConnected(self)
1036
 
        return self._real_medium.read_bytes(count)
 
824
        bytes_to_read = min(count, _MAX_READ_SIZE)
 
825
        bytes = self._read_from.read(bytes_to_read)
 
826
        self._report_activity(len(bytes), 'read')
 
827
        return bytes
1037
828
 
1038
829
 
1039
830
# Port 4155 is the default port for bzr://, registered with IANA.
1041
832
BZR_DEFAULT_PORT = 4155
1042
833
 
1043
834
 
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
 
    """
 
835
class SmartTCPClientMedium(SmartClientStreamMedium):
 
836
    """A client medium using TCP."""
1049
837
 
1050
 
    def __init__(self, base):
 
838
    def __init__(self, host, port, base):
 
839
        """Creates a client that will connect on the first use."""
1051
840
        SmartClientStreamMedium.__init__(self, base)
 
841
        self._connected = False
 
842
        self._host = host
 
843
        self._port = port
1052
844
        self._socket = None
1053
 
        self._connected = False
1054
845
 
1055
846
    def _accept_bytes(self, bytes):
1056
847
        """See SmartClientMedium.accept_bytes."""
1057
848
        self._ensure_connection()
1058
849
        osutils.send_all(self._socket, bytes, self._report_activity)
1059
850
 
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
851
    def disconnect(self):
1080
852
        """See SmartClientMedium.disconnect()."""
1081
853
        if not self._connected:
1084
856
        self._socket = None
1085
857
        self._connected = False
1086
858
 
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
859
    def _ensure_connection(self):
1098
860
        """Connect this medium if not already connected."""
1099
861
        if self._connected:
1132
894
            raise errors.ConnectionError("failed to connect to %s:%d: %s" %
1133
895
                    (self._host, port, err_msg))
1134
896
        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
 
897
 
 
898
    def _flush(self):
 
899
        """See SmartClientStreamMedium._flush().
 
900
 
 
901
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and
 
902
        add a means to do a flush, but that can be done in the future.
 
903
        """
 
904
 
 
905
    def _read_bytes(self, count):
 
906
        """See SmartClientMedium.read_bytes."""
 
907
        if not self._connected:
 
908
            raise errors.MediumNotConnected(self)
 
909
        return osutils.read_bytes_from_socket(
 
910
            self._socket, self._report_activity)
1154
911
 
1155
912
 
1156
913
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
1191
948
        This invokes self._medium._flush to ensure all bytes are transmitted.
1192
949
        """
1193
950
        self._medium._flush()
 
951
 
 
952