/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: Colin D Bennett
  • Date: 2009-02-12 16:57:39 UTC
  • mto: This revision was merged to the branch mainline in revision 4008.
  • Revision ID: colin@gibibit.com-20090212165739-02xv63odccfmxomw
Generate PDF version of the User Guide.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
 
1
# Copyright (C) 2006 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""The 'medium' layer for the smart servers and clients.
18
18
 
24
24
bzrlib/transport/smart/__init__.py.
25
25
"""
26
26
 
 
27
import errno
27
28
import os
 
29
import socket
28
30
import sys
29
31
import urllib
30
32
 
31
 
import bzrlib
32
33
from bzrlib.lazy_import import lazy_import
33
34
lazy_import(globals(), """
34
 
import socket
35
 
import thread
 
35
import atexit
36
36
import weakref
37
 
 
38
37
from bzrlib import (
39
38
    debug,
40
39
    errors,
 
40
    osutils,
41
41
    symbol_versioning,
42
42
    trace,
43
 
    ui,
44
43
    urlutils,
45
44
    )
46
 
from bzrlib.smart import client, protocol, request, vfs
 
45
from bzrlib.smart import client, protocol
47
46
from bzrlib.transport import ssh
48
47
""")
49
 
from bzrlib import osutils
50
 
 
51
 
# Throughout this module buffer size parameters are either limited to be at
52
 
# most _MAX_READ_SIZE, or are ignored and _MAX_READ_SIZE is used instead.
53
 
# For this module's purposes, MAX_SOCKET_CHUNK is a reasonable size for reads
54
 
# from non-sockets as well.
55
 
_MAX_READ_SIZE = osutils.MAX_SOCKET_CHUNK
 
48
 
 
49
 
 
50
# We must not read any more than 64k at a time so we don't risk "no buffer
 
51
# space available" errors on some platforms.  Windows in particular is likely
 
52
# to give error 10053 or 10055 if we read more than 64k from a socket.
 
53
_MAX_READ_SIZE = 64 * 1024
 
54
 
56
55
 
57
56
def _get_protocol_factory_for_bytes(bytes):
58
57
    """Determine the right protocol factory for 'bytes'.
88
87
 
89
88
def _get_line(read_bytes_func):
90
89
    """Read bytes using read_bytes_func until a newline byte.
91
 
 
 
90
    
92
91
    This isn't particularly efficient, so should only be used when the
93
92
    expected size of the line is quite short.
94
 
 
 
93
    
95
94
    :returns: a tuple of two strs: (line, excess)
96
95
    """
97
96
    newline_pos = -1
113
112
 
114
113
    def __init__(self):
115
114
        self._push_back_buffer = None
116
 
 
 
115
        
117
116
    def _push_back(self, bytes):
118
117
        """Return unused bytes to the medium, because they belong to the next
119
118
        request(s).
153
152
 
154
153
    def _get_line(self):
155
154
        """Read bytes from this request's response until a newline byte.
156
 
 
 
155
        
157
156
        This isn't particularly efficient, so should only be used when the
158
157
        expected size of the line is quite short.
159
158
 
162
161
        line, excess = _get_line(self.read_bytes)
163
162
        self._push_back(excess)
164
163
        return line
165
 
 
166
 
    def _report_activity(self, bytes, direction):
167
 
        """Notify that this medium has activity.
168
 
 
169
 
        Implementations should call this from all methods that actually do IO.
170
 
        Be careful that it's not called twice, if one method is implemented on
171
 
        top of another.
172
 
 
173
 
        :param bytes: Number of bytes read or written.
174
 
        :param direction: 'read' or 'write' or None.
175
 
        """
176
 
        ui.ui_factory.report_transport_activity(self, bytes, direction)
177
 
 
 
164
 
178
165
 
179
166
class SmartServerStreamMedium(SmartMedium):
180
167
    """Handles smart commands coming over a stream.
185
172
    One instance is created for each connected client; it can serve multiple
186
173
    requests in the lifetime of the connection.
187
174
 
188
 
    The server passes requests through to an underlying backing transport,
 
175
    The server passes requests through to an underlying backing transport, 
189
176
    which will typically be a LocalTransport looking at the server's filesystem.
190
177
 
191
178
    :ivar _push_back_buffer: a str of bytes that have been read from the stream
236
223
 
237
224
    def _serve_one_request(self, protocol):
238
225
        """Read one request from input, process, send back a response.
239
 
 
 
226
        
240
227
        :param protocol: a SmartServerRequestProtocol.
241
228
        """
242
229
        try:
274
261
    def _serve_one_request_unguarded(self, protocol):
275
262
        while protocol.next_read_size():
276
263
            # We can safely try to read large chunks.  If there is less data
277
 
            # than MAX_SOCKET_CHUNK ready, the socket will just return a
278
 
            # short read immediately rather than block.
279
 
            bytes = self.read_bytes(osutils.MAX_SOCKET_CHUNK)
 
264
            # than _MAX_READ_SIZE ready, the socket wil just return a short
 
265
            # read immediately rather than block.
 
266
            bytes = self.read_bytes(_MAX_READ_SIZE)
280
267
            if bytes == '':
281
268
                self.finished = True
282
269
                return
283
270
            protocol.accept_bytes(bytes)
284
 
 
 
271
        
285
272
        self._push_back(protocol.unused_data)
286
273
 
287
274
    def _read_bytes(self, desired_count):
288
 
        return osutils.read_bytes_from_socket(
289
 
            self.socket, self._report_activity)
 
275
        # We ignore the desired_count because on sockets it's more efficient to
 
276
        # read large chunks (of _MAX_READ_SIZE bytes) at a time.
 
277
        return osutils.until_no_eintr(self.socket.recv, _MAX_READ_SIZE)
290
278
 
291
279
    def terminate_due_to_error(self):
292
280
        # TODO: This should log to a server log file, but no such thing
295
283
        self.finished = True
296
284
 
297
285
    def _write_out(self, bytes):
298
 
        tstart = osutils.timer_func()
299
 
        osutils.send_all(self.socket, bytes, self._report_activity)
300
 
        if 'hpss' in debug.debug_flags:
301
 
            thread_id = thread.get_ident()
302
 
            trace.mutter('%12s: [%s] %d bytes to the socket in %.3fs'
303
 
                         % ('wrote', thread_id, len(bytes),
304
 
                            osutils.timer_func() - tstart))
 
286
        osutils.send_all(self.socket, bytes)
305
287
 
306
288
 
307
289
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
368
350
    request.finished_reading()
369
351
 
370
352
    It is up to the individual SmartClientMedium whether multiple concurrent
371
 
    requests can exist. See SmartClientMedium.get_request to obtain instances
372
 
    of SmartClientMediumRequest, and the concrete Medium you are using for
 
353
    requests can exist. See SmartClientMedium.get_request to obtain instances 
 
354
    of SmartClientMediumRequest, and the concrete Medium you are using for 
373
355
    details on concurrency and pipelining.
374
356
    """
375
357
 
384
366
    def accept_bytes(self, bytes):
385
367
        """Accept bytes for inclusion in this request.
386
368
 
387
 
        This method may not be called after finished_writing() has been
 
369
        This method may not be be called after finished_writing() has been
388
370
        called.  It depends upon the Medium whether or not the bytes will be
389
371
        immediately transmitted. Message based Mediums will tend to buffer the
390
372
        bytes until finished_writing() is called.
421
403
    def _finished_reading(self):
422
404
        """Helper for finished_reading.
423
405
 
424
 
        finished_reading checks the state of the request to determine if
 
406
        finished_reading checks the state of the request to determine if 
425
407
        finished_reading is allowed, and if it is hands off to _finished_reading
426
408
        to perform the action.
427
409
        """
441
423
    def _finished_writing(self):
442
424
        """Helper for finished_writing.
443
425
 
444
 
        finished_writing checks the state of the request to determine if
 
426
        finished_writing checks the state of the request to determine if 
445
427
        finished_writing is allowed, and if it is hands off to _finished_writing
446
428
        to perform the action.
447
429
        """
467
449
        read_bytes checks the state of the request to determing if bytes
468
450
        should be read. After that it hands off to _read_bytes to do the
469
451
        actual read.
470
 
 
 
452
        
471
453
        By default this forwards to self._medium.read_bytes because we are
472
454
        operating on the medium's stream.
473
455
        """
478
460
        if not line.endswith('\n'):
479
461
            # end of file encountered reading from server
480
462
            raise errors.ConnectionReset(
481
 
                "Unexpected end of message. Please check connectivity "
482
 
                "and permissions, and report a bug if problems persist.")
 
463
                "please check connectivity and permissions",
 
464
                "(and try -Dhpss if further diagnosis is required)")
483
465
        return line
484
466
 
485
467
    def _read_line(self):
486
468
        """Helper for SmartClientMediumRequest.read_line.
487
 
 
 
469
        
488
470
        By default this forwards to self._medium._get_line because we are
489
471
        operating on the medium's stream.
490
472
        """
494
476
class _DebugCounter(object):
495
477
    """An object that counts the HPSS calls made to each client medium.
496
478
 
497
 
    When a medium is garbage-collected, or failing that when
498
 
    bzrlib.global_state exits, the total number of calls made on that medium
499
 
    are reported via trace.note.
 
479
    When a medium is garbage-collected, or failing that when atexit functions
 
480
    are run, the total number of calls made on that medium are reported via
 
481
    trace.note.
500
482
    """
501
483
 
502
484
    def __init__(self):
503
485
        self.counts = weakref.WeakKeyDictionary()
504
486
        client._SmartClient.hooks.install_named_hook(
505
487
            'call', self.increment_call_count, 'hpss call counter')
506
 
        bzrlib.global_state.cleanups.add_cleanup(self.flush_all)
 
488
        atexit.register(self.flush_all)
507
489
 
508
490
    def track(self, medium):
509
491
        """Start tracking calls made to a medium.
513
495
        """
514
496
        medium_repr = repr(medium)
515
497
        # Add this medium to the WeakKeyDictionary
516
 
        self.counts[medium] = dict(count=0, vfs_count=0,
517
 
                                   medium_repr=medium_repr)
 
498
        self.counts[medium] = [0, medium_repr]
518
499
        # Weakref callbacks are fired in reverse order of their association
519
500
        # with the referenced object.  So we add a weakref *after* adding to
520
501
        # the WeakKeyDict so that we can report the value from it before the
524
505
    def increment_call_count(self, params):
525
506
        # Increment the count in the WeakKeyDictionary
526
507
        value = self.counts[params.medium]
527
 
        value['count'] += 1
528
 
        try:
529
 
            request_method = request.request_handlers.get(params.method)
530
 
        except KeyError:
531
 
            # A method we don't know about doesn't count as a VFS method.
532
 
            return
533
 
        if issubclass(request_method, vfs.VfsRequest):
534
 
            value['vfs_count'] += 1
 
508
        value[0] += 1
535
509
 
536
510
    def done(self, ref):
537
511
        value = self.counts[ref]
538
 
        count, vfs_count, medium_repr = (
539
 
            value['count'], value['vfs_count'], value['medium_repr'])
 
512
        count, medium_repr = value
540
513
        # In case this callback is invoked for the same ref twice (by the
541
514
        # weakref callback and by the atexit function), set the call count back
542
515
        # to 0 so this item won't be reported twice.
543
 
        value['count'] = 0
544
 
        value['vfs_count'] = 0
 
516
        value[0] = 0
545
517
        if count != 0:
546
 
            trace.note('HPSS calls: %d (%d vfs) %s',
547
 
                       count, vfs_count, medium_repr)
548
 
 
 
518
            trace.note('HPSS calls: %d %s', count, medium_repr)
 
519
        
549
520
    def flush_all(self):
550
521
        for ref in list(self.counts.keys()):
551
522
            self.done(ref)
552
523
 
553
524
_debug_counter = None
554
 
 
555
 
 
 
525
  
 
526
  
556
527
class SmartClientMedium(SmartMedium):
557
528
    """Smart client is a medium for sending smart protocol requests over."""
558
529
 
603
574
        """
604
575
        if (self._remote_version_is_before is not None and
605
576
            version_tuple > self._remote_version_is_before):
606
 
            # We have been told that the remote side is older than some version
607
 
            # which is newer than a previously supplied older-than version.
608
 
            # This indicates that some smart verb call is not guarded
609
 
            # appropriately (it should simply not have been tried).
610
 
            trace.mutter(
 
577
            raise AssertionError(
611
578
                "_remember_remote_is_before(%r) called, but "
612
579
                "_remember_remote_is_before(%r) was called previously."
613
 
                , version_tuple, self._remote_version_is_before)
614
 
            if 'hpss' in debug.debug_flags:
615
 
                ui.ui_factory.show_warning(
616
 
                    "_remember_remote_is_before(%r) called, but "
617
 
                    "_remember_remote_is_before(%r) was called previously."
618
 
                    % (version_tuple, self._remote_version_is_before))
619
 
            return
 
580
                % (version_tuple, self._remote_version_is_before))
620
581
        self._remote_version_is_before = version_tuple
621
582
 
622
583
    def protocol_version(self):
656
617
 
657
618
    def disconnect(self):
658
619
        """If this medium maintains a persistent connection, close it.
659
 
 
 
620
        
660
621
        The default implementation does nothing.
661
622
        """
662
 
 
 
623
        
663
624
    def remote_path_from_transport(self, transport):
664
625
        """Convert transport into a path suitable for using in a request.
665
 
 
 
626
        
666
627
        Note that the resulting remote path doesn't encode the host name or
667
628
        anything but path, so it is only safe to use it in requests sent over
668
629
        the medium from the matching transport.
696
657
 
697
658
    def _flush(self):
698
659
        """Flush the output stream.
699
 
 
 
660
        
700
661
        This method is used by the SmartClientStreamMediumRequest to ensure that
701
662
        all data for a request is sent, to avoid long timeouts or deadlocks.
702
663
        """
713
674
 
714
675
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
715
676
    """A client medium using simple pipes.
716
 
 
 
677
    
717
678
    This client does not manage the pipes: it assumes they will always be open.
718
679
    """
719
680
 
725
686
    def _accept_bytes(self, bytes):
726
687
        """See SmartClientStreamMedium.accept_bytes."""
727
688
        self._writeable_pipe.write(bytes)
728
 
        self._report_activity(len(bytes), 'write')
729
689
 
730
690
    def _flush(self):
731
691
        """See SmartClientStreamMedium._flush()."""
733
693
 
734
694
    def _read_bytes(self, count):
735
695
        """See SmartClientStreamMedium._read_bytes."""
736
 
        bytes_to_read = min(count, _MAX_READ_SIZE)
737
 
        bytes = self._readable_pipe.read(bytes_to_read)
738
 
        self._report_activity(len(bytes), 'read')
739
 
        return bytes
740
 
 
741
 
 
742
 
class SSHParams(object):
743
 
    """A set of parameters for starting a remote bzr via SSH."""
744
 
 
 
696
        return self._readable_pipe.read(count)
 
697
 
 
698
 
 
699
class SmartSSHClientMedium(SmartClientStreamMedium):
 
700
    """A client medium using SSH."""
 
701
    
745
702
    def __init__(self, host, port=None, username=None, password=None,
746
 
            bzr_remote_path='bzr'):
747
 
        self.host = host
748
 
        self.port = port
749
 
        self.username = username
750
 
        self.password = password
751
 
        self.bzr_remote_path = bzr_remote_path
752
 
 
753
 
 
754
 
class SmartSSHClientMedium(SmartClientStreamMedium):
755
 
    """A client medium using SSH.
756
 
    
757
 
    It delegates IO to a SmartClientSocketMedium or
758
 
    SmartClientAlreadyConnectedSocketMedium (depending on platform).
759
 
    """
760
 
 
761
 
    def __init__(self, base, ssh_params, vendor=None):
 
703
            base=None, vendor=None, bzr_remote_path=None):
762
704
        """Creates a client that will connect on the first use.
763
 
 
764
 
        :param ssh_params: A SSHParams instance.
 
705
        
765
706
        :param vendor: An optional override for the ssh vendor to use. See
766
707
            bzrlib.transport.ssh for details on ssh vendors.
767
708
        """
768
 
        self._real_medium = None
769
 
        self._ssh_params = ssh_params
770
 
        # for the benefit of progress making a short description of this
771
 
        # transport
772
 
        self._scheme = 'bzr+ssh'
773
 
        # SmartClientStreamMedium stores the repr of this object in its
774
 
        # _DebugCounter so we have to store all the values used in our repr
775
 
        # method before calling the super init.
776
709
        SmartClientStreamMedium.__init__(self, base)
 
710
        self._connected = False
 
711
        self._host = host
 
712
        self._password = password
 
713
        self._port = port
 
714
        self._username = username
 
715
        self._read_from = None
 
716
        self._ssh_connection = None
777
717
        self._vendor = vendor
778
 
        self._ssh_connection = None
779
 
 
780
 
    def __repr__(self):
781
 
        if self._ssh_params.port is None:
782
 
            maybe_port = ''
783
 
        else:
784
 
            maybe_port = ':%s' % self._ssh_params.port
785
 
        return "%s(%s://%s@%s%s/)" % (
786
 
            self.__class__.__name__,
787
 
            self._scheme,
788
 
            self._ssh_params.username,
789
 
            self._ssh_params.host,
790
 
            maybe_port)
 
718
        self._write_to = None
 
719
        self._bzr_remote_path = bzr_remote_path
 
720
        if self._bzr_remote_path is None:
 
721
            symbol_versioning.warn(
 
722
                'bzr_remote_path is required as of bzr 0.92',
 
723
                DeprecationWarning, stacklevel=2)
 
724
            self._bzr_remote_path = os.environ.get('BZR_REMOTE_PATH', 'bzr')
791
725
 
792
726
    def _accept_bytes(self, bytes):
793
727
        """See SmartClientStreamMedium.accept_bytes."""
794
728
        self._ensure_connection()
795
 
        self._real_medium.accept_bytes(bytes)
 
729
        self._write_to.write(bytes)
796
730
 
797
731
    def disconnect(self):
798
732
        """See SmartClientMedium.disconnect()."""
799
 
        if self._real_medium is not None:
800
 
            self._real_medium.disconnect()
801
 
            self._real_medium = None
802
 
        if self._ssh_connection is not None:
803
 
            self._ssh_connection.close()
804
 
            self._ssh_connection = None
 
733
        if not self._connected:
 
734
            return
 
735
        self._read_from.close()
 
736
        self._write_to.close()
 
737
        self._ssh_connection.close()
 
738
        self._connected = False
805
739
 
806
740
    def _ensure_connection(self):
807
741
        """Connect this medium if not already connected."""
808
 
        if self._real_medium is not None:
 
742
        if self._connected:
809
743
            return
810
744
        if self._vendor is None:
811
745
            vendor = ssh._get_ssh_vendor()
812
746
        else:
813
747
            vendor = self._vendor
814
 
        self._ssh_connection = vendor.connect_ssh(self._ssh_params.username,
815
 
                self._ssh_params.password, self._ssh_params.host,
816
 
                self._ssh_params.port,
817
 
                command=[self._ssh_params.bzr_remote_path, 'serve', '--inet',
 
748
        self._ssh_connection = vendor.connect_ssh(self._username,
 
749
                self._password, self._host, self._port,
 
750
                command=[self._bzr_remote_path, 'serve', '--inet',
818
751
                         '--directory=/', '--allow-writes'])
819
 
        io_kind, io_object = self._ssh_connection.get_sock_or_pipes()
820
 
        if io_kind == 'socket':
821
 
            self._real_medium = SmartClientAlreadyConnectedSocketMedium(
822
 
                self.base, io_object)
823
 
        elif io_kind == 'pipes':
824
 
            read_from, write_to = io_object
825
 
            self._real_medium = SmartSimplePipesClientMedium(
826
 
                read_from, write_to, self.base)
827
 
        else:
828
 
            raise AssertionError(
829
 
                "Unexpected io_kind %r from %r"
830
 
                % (io_kind, self._ssh_connection))
 
752
        self._read_from, self._write_to = \
 
753
            self._ssh_connection.get_filelike_channels()
 
754
        self._connected = True
831
755
 
832
756
    def _flush(self):
833
757
        """See SmartClientStreamMedium._flush()."""
834
 
        self._real_medium._flush()
 
758
        self._write_to.flush()
835
759
 
836
760
    def _read_bytes(self, count):
837
761
        """See SmartClientStreamMedium.read_bytes."""
838
 
        if self._real_medium is None:
 
762
        if not self._connected:
839
763
            raise errors.MediumNotConnected(self)
840
 
        return self._real_medium.read_bytes(count)
 
764
        bytes_to_read = min(count, _MAX_READ_SIZE)
 
765
        return self._read_from.read(bytes_to_read)
841
766
 
842
767
 
843
768
# Port 4155 is the default port for bzr://, registered with IANA.
845
770
BZR_DEFAULT_PORT = 4155
846
771
 
847
772
 
848
 
class SmartClientSocketMedium(SmartClientStreamMedium):
849
 
    """A client medium using a socket.
 
773
class SmartTCPClientMedium(SmartClientStreamMedium):
 
774
    """A client medium using TCP."""
850
775
    
851
 
    This class isn't usable directly.  Use one of its subclasses instead.
852
 
    """
853
 
 
854
 
    def __init__(self, base):
 
776
    def __init__(self, host, port, base):
 
777
        """Creates a client that will connect on the first use."""
855
778
        SmartClientStreamMedium.__init__(self, base)
 
779
        self._connected = False
 
780
        self._host = host
 
781
        self._port = port
856
782
        self._socket = None
857
 
        self._connected = False
858
783
 
859
784
    def _accept_bytes(self, bytes):
860
785
        """See SmartClientMedium.accept_bytes."""
861
786
        self._ensure_connection()
862
 
        osutils.send_all(self._socket, bytes, self._report_activity)
863
 
 
864
 
    def _ensure_connection(self):
865
 
        """Connect this medium if not already connected."""
866
 
        raise NotImplementedError(self._ensure_connection)
867
 
 
868
 
    def _flush(self):
869
 
        """See SmartClientStreamMedium._flush().
870
 
 
871
 
        For sockets we do no flushing. For TCP sockets we may want to turn off
872
 
        TCP_NODELAY and add a means to do a flush, but that can be done in the
873
 
        future.
874
 
        """
875
 
 
876
 
    def _read_bytes(self, count):
877
 
        """See SmartClientMedium.read_bytes."""
878
 
        if not self._connected:
879
 
            raise errors.MediumNotConnected(self)
880
 
        return osutils.read_bytes_from_socket(
881
 
            self._socket, self._report_activity)
 
787
        osutils.send_all(self._socket, bytes)
882
788
 
883
789
    def disconnect(self):
884
790
        """See SmartClientMedium.disconnect()."""
888
794
        self._socket = None
889
795
        self._connected = False
890
796
 
891
 
 
892
 
class SmartTCPClientMedium(SmartClientSocketMedium):
893
 
    """A client medium that creates a TCP connection."""
894
 
 
895
 
    def __init__(self, host, port, base):
896
 
        """Creates a client that will connect on the first use."""
897
 
        SmartClientSocketMedium.__init__(self, base)
898
 
        self._host = host
899
 
        self._port = port
900
 
 
901
797
    def _ensure_connection(self):
902
798
        """Connect this medium if not already connected."""
903
799
        if self._connected:
907
803
        else:
908
804
            port = int(self._port)
909
805
        try:
910
 
            sockaddrs = socket.getaddrinfo(self._host, port, socket.AF_UNSPEC,
 
806
            sockaddrs = socket.getaddrinfo(self._host, port, socket.AF_UNSPEC, 
911
807
                socket.SOCK_STREAM, 0, 0)
912
808
        except socket.gaierror, (err_num, err_msg):
913
809
            raise errors.ConnectionError("failed to lookup %s:%d: %s" %
917
813
        for (family, socktype, proto, canonname, sockaddr) in sockaddrs:
918
814
            try:
919
815
                self._socket = socket.socket(family, socktype, proto)
920
 
                self._socket.setsockopt(socket.IPPROTO_TCP,
 
816
                self._socket.setsockopt(socket.IPPROTO_TCP, 
921
817
                                        socket.TCP_NODELAY, 1)
922
818
                self._socket.connect(sockaddr)
923
819
            except socket.error, err:
937
833
                    (self._host, port, err_msg))
938
834
        self._connected = True
939
835
 
940
 
 
941
 
class SmartClientAlreadyConnectedSocketMedium(SmartClientSocketMedium):
942
 
    """A client medium for an already connected socket.
943
 
    
944
 
    Note that this class will assume it "owns" the socket, so it will close it
945
 
    when its disconnect method is called.
946
 
    """
947
 
 
948
 
    def __init__(self, base, sock):
949
 
        SmartClientSocketMedium.__init__(self, base)
950
 
        self._socket = sock
951
 
        self._connected = True
952
 
 
953
 
    def _ensure_connection(self):
954
 
        # Already connected, by definition!  So nothing to do.
955
 
        pass
 
836
    def _flush(self):
 
837
        """See SmartClientStreamMedium._flush().
 
838
        
 
839
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and 
 
840
        add a means to do a flush, but that can be done in the future.
 
841
        """
 
842
 
 
843
    def _read_bytes(self, count):
 
844
        """See SmartClientMedium.read_bytes."""
 
845
        if not self._connected:
 
846
            raise errors.MediumNotConnected(self)
 
847
        # We ignore the desired_count because on sockets it's more efficient to
 
848
        # read large chunks (of _MAX_READ_SIZE bytes) at a time.
 
849
        try:
 
850
            return self._socket.recv(_MAX_READ_SIZE)
 
851
        except socket.error, e:
 
852
            if len(e.args) and e.args[0] == errno.ECONNRESET:
 
853
                # Callers expect an empty string in that case
 
854
                return ''
 
855
            else:
 
856
                raise
956
857
 
957
858
 
958
859
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
971
872
 
972
873
    def _accept_bytes(self, bytes):
973
874
        """See SmartClientMediumRequest._accept_bytes.
974
 
 
 
875
        
975
876
        This forwards to self._medium._accept_bytes because we are operating
976
877
        on the mediums stream.
977
878
        """
980
881
    def _finished_reading(self):
981
882
        """See SmartClientMediumRequest._finished_reading.
982
883
 
983
 
        This clears the _current_request on self._medium to allow a new
 
884
        This clears the _current_request on self._medium to allow a new 
984
885
        request to be created.
985
886
        """
986
887
        if self._medium._current_request is not self:
987
888
            raise AssertionError()
988
889
        self._medium._current_request = None
989
 
 
 
890
        
990
891
    def _finished_writing(self):
991
892
        """See SmartClientMediumRequest._finished_writing.
992
893
 
994
895
        """
995
896
        self._medium._flush()
996
897
 
997