~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.py

  • Committer: Ian Clatworthy
  • Date: 2008-07-17 01:08:21 UTC
  • mto: This revision was merged to the branch mainline in revision 3546.
  • Revision ID: ian.clatworthy@canonical.com-20080717010821-qg05ah7iobvl8hin
workaround docutils dot-in-option name bug (Mark Hammond)

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
 
25
25
"""
26
26
 
27
27
import os
 
28
import socket
28
29
import sys
29
30
import urllib
30
31
 
31
32
from bzrlib.lazy_import import lazy_import
32
33
lazy_import(globals(), """
33
 
import atexit
34
 
import socket
35
 
import thread
36
 
import weakref
37
 
 
38
34
from bzrlib import (
39
 
    debug,
40
35
    errors,
 
36
    osutils,
41
37
    symbol_versioning,
42
 
    trace,
43
 
    ui,
44
38
    urlutils,
45
39
    )
46
 
from bzrlib.smart import client, protocol, request, vfs
 
40
from bzrlib.smart import protocol
47
41
from bzrlib.transport import ssh
48
42
""")
49
 
from bzrlib import osutils
50
43
 
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
56
44
 
57
45
def _get_protocol_factory_for_bytes(bytes):
58
46
    """Determine the right protocol factory for 'bytes'.
86
74
    return protocol_factory, bytes
87
75
 
88
76
 
89
 
def _get_line(read_bytes_func):
90
 
    """Read bytes using read_bytes_func until a newline byte.
91
 
 
92
 
    This isn't particularly efficient, so should only be used when the
93
 
    expected size of the line is quite short.
94
 
 
95
 
    :returns: a tuple of two strs: (line, excess)
96
 
    """
97
 
    newline_pos = -1
98
 
    bytes = ''
99
 
    while newline_pos == -1:
100
 
        new_bytes = read_bytes_func(1)
101
 
        bytes += new_bytes
102
 
        if new_bytes == '':
103
 
            # Ran out of bytes before receiving a complete line.
104
 
            return bytes, ''
105
 
        newline_pos = bytes.find('\n')
106
 
    line = bytes[:newline_pos+1]
107
 
    excess = bytes[newline_pos+1:]
108
 
    return line, excess
109
 
 
110
 
 
111
 
class SmartMedium(object):
112
 
    """Base class for smart protocol media, both client- and server-side."""
113
 
 
114
 
    def __init__(self):
115
 
        self._push_back_buffer = None
116
 
 
117
 
    def _push_back(self, bytes):
118
 
        """Return unused bytes to the medium, because they belong to the next
119
 
        request(s).
120
 
 
121
 
        This sets the _push_back_buffer to the given bytes.
122
 
        """
123
 
        if self._push_back_buffer is not None:
124
 
            raise AssertionError(
125
 
                "_push_back called when self._push_back_buffer is %r"
126
 
                % (self._push_back_buffer,))
127
 
        if bytes == '':
128
 
            return
129
 
        self._push_back_buffer = bytes
130
 
 
131
 
    def _get_push_back_buffer(self):
132
 
        if self._push_back_buffer == '':
133
 
            raise AssertionError(
134
 
                '%s._push_back_buffer should never be the empty string, '
135
 
                'which can be confused with EOF' % (self,))
136
 
        bytes = self._push_back_buffer
137
 
        self._push_back_buffer = None
138
 
        return bytes
139
 
 
140
 
    def read_bytes(self, desired_count):
141
 
        """Read some bytes from this medium.
142
 
 
143
 
        :returns: some bytes, possibly more or less than the number requested
144
 
            in 'desired_count' depending on the medium.
145
 
        """
146
 
        if self._push_back_buffer is not None:
147
 
            return self._get_push_back_buffer()
148
 
        bytes_to_read = min(desired_count, _MAX_READ_SIZE)
149
 
        return self._read_bytes(bytes_to_read)
150
 
 
151
 
    def _read_bytes(self, count):
152
 
        raise NotImplementedError(self._read_bytes)
153
 
 
154
 
    def _get_line(self):
155
 
        """Read bytes from this request's response until a newline byte.
156
 
 
157
 
        This isn't particularly efficient, so should only be used when the
158
 
        expected size of the line is quite short.
159
 
 
160
 
        :returns: a string of bytes ending in a newline (byte 0x0A).
161
 
        """
162
 
        line, excess = _get_line(self.read_bytes)
163
 
        self._push_back(excess)
164
 
        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
 
 
178
 
 
179
 
class SmartServerStreamMedium(SmartMedium):
 
77
class SmartServerStreamMedium(object):
180
78
    """Handles smart commands coming over a stream.
181
79
 
182
80
    The stream may be a pipe connected to sshd, or a tcp socket, or an
185
83
    One instance is created for each connected client; it can serve multiple
186
84
    requests in the lifetime of the connection.
187
85
 
188
 
    The server passes requests through to an underlying backing transport,
 
86
    The server passes requests through to an underlying backing transport, 
189
87
    which will typically be a LocalTransport looking at the server's filesystem.
190
88
 
191
89
    :ivar _push_back_buffer: a str of bytes that have been read from the stream
203
101
        self.backing_transport = backing_transport
204
102
        self.root_client_path = root_client_path
205
103
        self.finished = False
206
 
        SmartMedium.__init__(self)
 
104
        self._push_back_buffer = None
 
105
 
 
106
    def _push_back(self, bytes):
 
107
        """Return unused bytes to the medium, because they belong to the next
 
108
        request(s).
 
109
 
 
110
        This sets the _push_back_buffer to the given bytes.
 
111
        """
 
112
        if self._push_back_buffer is not None:
 
113
            raise AssertionError(
 
114
                "_push_back called when self._push_back_buffer is %r"
 
115
                % (self._push_back_buffer,))
 
116
        if bytes == '':
 
117
            return
 
118
        self._push_back_buffer = bytes
 
119
 
 
120
    def _get_push_back_buffer(self):
 
121
        if self._push_back_buffer == '':
 
122
            raise AssertionError(
 
123
                '%s._push_back_buffer should never be the empty string, '
 
124
                'which can be confused with EOF' % (self,))
 
125
        bytes = self._push_back_buffer
 
126
        self._push_back_buffer = None
 
127
        return bytes
207
128
 
208
129
    def serve(self):
209
130
        """Serve requests until the client disconnects."""
236
157
 
237
158
    def _serve_one_request(self, protocol):
238
159
        """Read one request from input, process, send back a response.
239
 
 
 
160
        
240
161
        :param protocol: a SmartServerRequestProtocol.
241
162
        """
242
163
        try:
250
171
        """Called when an unhandled exception from the protocol occurs."""
251
172
        raise NotImplementedError(self.terminate_due_to_error)
252
173
 
253
 
    def _read_bytes(self, desired_count):
 
174
    def _get_bytes(self, desired_count):
254
175
        """Get some bytes from the medium.
255
176
 
256
177
        :param desired_count: number of bytes we want to read.
257
178
        """
258
 
        raise NotImplementedError(self._read_bytes)
259
 
 
 
179
        raise NotImplementedError(self._get_bytes)
 
180
 
 
181
    def _get_line(self):
 
182
        """Read bytes from this request's response until a newline byte.
 
183
        
 
184
        This isn't particularly efficient, so should only be used when the
 
185
        expected size of the line is quite short.
 
186
 
 
187
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
188
        """
 
189
        newline_pos = -1
 
190
        bytes = ''
 
191
        while newline_pos == -1:
 
192
            new_bytes = self._get_bytes(1)
 
193
            bytes += new_bytes
 
194
            if new_bytes == '':
 
195
                # Ran out of bytes before receiving a complete line.
 
196
                return bytes
 
197
            newline_pos = bytes.find('\n')
 
198
        line = bytes[:newline_pos+1]
 
199
        self._push_back(bytes[newline_pos+1:])
 
200
        return line
 
201
 
260
202
 
261
203
class SmartServerSocketStreamMedium(SmartServerStreamMedium):
262
204
 
273
215
 
274
216
    def _serve_one_request_unguarded(self, protocol):
275
217
        while protocol.next_read_size():
276
 
            # 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)
 
218
            bytes = self._get_bytes(4096)
280
219
            if bytes == '':
281
220
                self.finished = True
282
221
                return
283
222
            protocol.accept_bytes(bytes)
284
 
 
 
223
        
285
224
        self._push_back(protocol.unused_data)
286
225
 
287
 
    def _read_bytes(self, desired_count):
288
 
        return osutils.read_bytes_from_socket(
289
 
            self.socket, self._report_activity)
290
 
 
 
226
    def _get_bytes(self, desired_count):
 
227
        if self._push_back_buffer is not None:
 
228
            return self._get_push_back_buffer()
 
229
        # We ignore the desired_count because on sockets it's more efficient to
 
230
        # read 4k at a time.
 
231
        return self.socket.recv(4096)
 
232
    
291
233
    def terminate_due_to_error(self):
292
234
        # TODO: This should log to a server log file, but no such thing
293
235
        # exists yet.  Andrew Bennetts 2006-09-29.
295
237
        self.finished = True
296
238
 
297
239
    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))
 
240
        osutils.send_all(self.socket, bytes)
305
241
 
306
242
 
307
243
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
326
262
 
327
263
    def _serve_one_request_unguarded(self, protocol):
328
264
        while True:
329
 
            # We need to be careful not to read past the end of the current
330
 
            # request, or else the read from the pipe will block, so we use
331
 
            # protocol.next_read_size().
332
265
            bytes_to_read = protocol.next_read_size()
333
266
            if bytes_to_read == 0:
334
267
                # Finished serving this request.
335
268
                self._out.flush()
336
269
                return
337
 
            bytes = self.read_bytes(bytes_to_read)
 
270
            bytes = self._get_bytes(bytes_to_read)
338
271
            if bytes == '':
339
272
                # Connection has been closed.
340
273
                self.finished = True
342
275
                return
343
276
            protocol.accept_bytes(bytes)
344
277
 
345
 
    def _read_bytes(self, desired_count):
 
278
    def _get_bytes(self, desired_count):
 
279
        if self._push_back_buffer is not None:
 
280
            return self._get_push_back_buffer()
346
281
        return self._in.read(desired_count)
347
282
 
348
283
    def terminate_due_to_error(self):
368
303
    request.finished_reading()
369
304
 
370
305
    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
 
306
    requests can exist. See SmartClientMedium.get_request to obtain instances 
 
307
    of SmartClientMediumRequest, and the concrete Medium you are using for 
373
308
    details on concurrency and pipelining.
374
309
    """
375
310
 
384
319
    def accept_bytes(self, bytes):
385
320
        """Accept bytes for inclusion in this request.
386
321
 
387
 
        This method may not be called after finished_writing() has been
 
322
        This method may not be be called after finished_writing() has been
388
323
        called.  It depends upon the Medium whether or not the bytes will be
389
324
        immediately transmitted. Message based Mediums will tend to buffer the
390
325
        bytes until finished_writing() is called.
421
356
    def _finished_reading(self):
422
357
        """Helper for finished_reading.
423
358
 
424
 
        finished_reading checks the state of the request to determine if
 
359
        finished_reading checks the state of the request to determine if 
425
360
        finished_reading is allowed, and if it is hands off to _finished_reading
426
361
        to perform the action.
427
362
        """
441
376
    def _finished_writing(self):
442
377
        """Helper for finished_writing.
443
378
 
444
 
        finished_writing checks the state of the request to determine if
 
379
        finished_writing checks the state of the request to determine if 
445
380
        finished_writing is allowed, and if it is hands off to _finished_writing
446
381
        to perform the action.
447
382
        """
462
397
        return self._read_bytes(count)
463
398
 
464
399
    def _read_bytes(self, count):
465
 
        """Helper for SmartClientMediumRequest.read_bytes.
 
400
        """Helper for read_bytes.
466
401
 
467
402
        read_bytes checks the state of the request to determing if bytes
468
403
        should be read. After that it hands off to _read_bytes to do the
469
404
        actual read.
470
 
 
471
 
        By default this forwards to self._medium.read_bytes because we are
472
 
        operating on the medium's stream.
473
405
        """
474
 
        return self._medium.read_bytes(count)
 
406
        raise NotImplementedError(self._read_bytes)
475
407
 
476
408
    def read_line(self):
477
 
        line = self._read_line()
478
 
        if not line.endswith('\n'):
479
 
            # end of file encountered reading from server
480
 
            raise errors.ConnectionReset(
481
 
                "Unexpected end of message. Please check connectivity "
482
 
                "and permissions, and report a bug if problems persist.")
 
409
        """Read bytes from this request's response until a newline byte.
 
410
        
 
411
        This isn't particularly efficient, so should only be used when the
 
412
        expected size of the line is quite short.
 
413
 
 
414
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
415
        """
 
416
        # XXX: this duplicates SmartClientRequestProtocolOne._recv_tuple
 
417
        line = ''
 
418
        while not line or line[-1] != '\n':
 
419
            new_char = self.read_bytes(1)
 
420
            line += new_char
 
421
            if new_char == '':
 
422
                # end of file encountered reading from server
 
423
                raise errors.ConnectionReset(
 
424
                    "please check connectivity and permissions",
 
425
                    "(and try -Dhpss if further diagnosis is required)")
483
426
        return line
484
427
 
485
 
    def _read_line(self):
486
 
        """Helper for SmartClientMediumRequest.read_line.
487
 
 
488
 
        By default this forwards to self._medium._get_line because we are
489
 
        operating on the medium's stream.
490
 
        """
491
 
        return self._medium._get_line()
492
 
 
493
 
 
494
 
class _DebugCounter(object):
495
 
    """An object that counts the HPSS calls made to each client medium.
496
 
 
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.
500
 
    """
501
 
 
502
 
    def __init__(self):
503
 
        self.counts = weakref.WeakKeyDictionary()
504
 
        client._SmartClient.hooks.install_named_hook(
505
 
            'call', self.increment_call_count, 'hpss call counter')
506
 
        atexit.register(self.flush_all)
507
 
 
508
 
    def track(self, medium):
509
 
        """Start tracking calls made to a medium.
510
 
 
511
 
        This only keeps a weakref to the medium, so shouldn't affect the
512
 
        medium's lifetime.
513
 
        """
514
 
        medium_repr = repr(medium)
515
 
        # Add this medium to the WeakKeyDictionary
516
 
        self.counts[medium] = dict(count=0, vfs_count=0,
517
 
                                   medium_repr=medium_repr)
518
 
        # Weakref callbacks are fired in reverse order of their association
519
 
        # with the referenced object.  So we add a weakref *after* adding to
520
 
        # the WeakKeyDict so that we can report the value from it before the
521
 
        # entry is removed by the WeakKeyDict's own callback.
522
 
        ref = weakref.ref(medium, self.done)
523
 
 
524
 
    def increment_call_count(self, params):
525
 
        # Increment the count in the WeakKeyDictionary
526
 
        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
535
 
 
536
 
    def done(self, ref):
537
 
        value = self.counts[ref]
538
 
        count, vfs_count, medium_repr = (
539
 
            value['count'], value['vfs_count'], value['medium_repr'])
540
 
        # In case this callback is invoked for the same ref twice (by the
541
 
        # weakref callback and by the atexit function), set the call count back
542
 
        # to 0 so this item won't be reported twice.
543
 
        value['count'] = 0
544
 
        value['vfs_count'] = 0
545
 
        if count != 0:
546
 
            trace.note('HPSS calls: %d (%d vfs) %s',
547
 
                       count, vfs_count, medium_repr)
548
 
 
549
 
    def flush_all(self):
550
 
        for ref in list(self.counts.keys()):
551
 
            self.done(ref)
552
 
 
553
 
_debug_counter = None
554
 
 
555
 
 
556
 
class SmartClientMedium(SmartMedium):
 
428
 
 
429
class SmartClientMedium(object):
557
430
    """Smart client is a medium for sending smart protocol requests over."""
558
431
 
559
432
    def __init__(self, base):
567
440
        # _remote_version_is_before tracks the bzr version the remote side
568
441
        # can be based on what we've seen so far.
569
442
        self._remote_version_is_before = None
570
 
        # Install debug hook function if debug flag is set.
571
 
        if 'hpss' in debug.debug_flags:
572
 
            global _debug_counter
573
 
            if _debug_counter is None:
574
 
                _debug_counter = _DebugCounter()
575
 
            _debug_counter.track(self)
576
443
 
577
444
    def _is_remote_before(self, version_tuple):
578
445
        """Is it possible the remote side supports RPCs for a given version?
603
470
        """
604
471
        if (self._remote_version_is_before is not None and
605
472
            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(
 
473
            raise AssertionError(
611
474
                "_remember_remote_is_before(%r) called, but "
612
475
                "_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
 
476
                % (version_tuple, self._remote_version_is_before))
620
477
        self._remote_version_is_before = version_tuple
621
478
 
622
479
    def protocol_version(self):
656
513
 
657
514
    def disconnect(self):
658
515
        """If this medium maintains a persistent connection, close it.
659
 
 
 
516
        
660
517
        The default implementation does nothing.
661
518
        """
662
 
 
 
519
        
663
520
    def remote_path_from_transport(self, transport):
664
521
        """Convert transport into a path suitable for using in a request.
665
 
 
 
522
        
666
523
        Note that the resulting remote path doesn't encode the host name or
667
524
        anything but path, so it is only safe to use it in requests sent over
668
525
        the medium from the matching transport.
696
553
 
697
554
    def _flush(self):
698
555
        """Flush the output stream.
699
 
 
 
556
        
700
557
        This method is used by the SmartClientStreamMediumRequest to ensure that
701
558
        all data for a request is sent, to avoid long timeouts or deadlocks.
702
559
        """
710
567
        """
711
568
        return SmartClientStreamMediumRequest(self)
712
569
 
 
570
    def read_bytes(self, count):
 
571
        return self._read_bytes(count)
 
572
 
713
573
 
714
574
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
715
575
    """A client medium using simple pipes.
716
 
 
 
576
    
717
577
    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.
722
578
    """
723
579
 
724
580
    def __init__(self, readable_pipe, writeable_pipe, base):
729
585
    def _accept_bytes(self, bytes):
730
586
        """See SmartClientStreamMedium.accept_bytes."""
731
587
        self._writeable_pipe.write(bytes)
732
 
        self._report_activity(len(bytes), 'write')
733
588
 
734
589
    def _flush(self):
735
590
        """See SmartClientStreamMedium._flush()."""
737
592
 
738
593
    def _read_bytes(self, count):
739
594
        """See SmartClientStreamMedium._read_bytes."""
740
 
        bytes = osutils.until_no_eintr(self._readable_pipe.read, count)
741
 
        self._report_activity(len(bytes), 'read')
742
 
        return bytes
 
595
        return self._readable_pipe.read(count)
743
596
 
744
597
 
745
598
class SmartSSHClientMedium(SmartClientStreamMedium):
746
599
    """A client medium using SSH."""
747
 
 
 
600
    
748
601
    def __init__(self, host, port=None, username=None, password=None,
749
602
            base=None, vendor=None, bzr_remote_path=None):
750
603
        """Creates a client that will connect on the first use.
751
 
 
 
604
        
752
605
        :param vendor: An optional override for the ssh vendor to use. See
753
606
            bzrlib.transport.ssh for details on ssh vendors.
754
607
        """
 
608
        SmartClientStreamMedium.__init__(self, base)
755
609
        self._connected = False
756
610
        self._host = host
757
611
        self._password = password
758
612
        self._port = port
759
613
        self._username = username
760
 
        # for the benefit of progress making a short description of this
761
 
        # transport
762
 
        self._scheme = 'bzr+ssh'
763
 
        # SmartClientStreamMedium stores the repr of this object in its
764
 
        # _DebugCounter so we have to store all the values used in our repr
765
 
        # method before calling the super init.
766
 
        SmartClientStreamMedium.__init__(self, base)
767
614
        self._read_from = None
768
615
        self._ssh_connection = None
769
616
        self._vendor = vendor
770
617
        self._write_to = None
771
618
        self._bzr_remote_path = bzr_remote_path
772
 
 
773
 
    def __repr__(self):
774
 
        if self._port is None:
775
 
            maybe_port = ''
776
 
        else:
777
 
            maybe_port = ':%s' % self._port
778
 
        return "%s(%s://%s@%s%s/)" % (
779
 
            self.__class__.__name__,
780
 
            self._scheme,
781
 
            self._username,
782
 
            self._host,
783
 
            maybe_port)
 
619
        if self._bzr_remote_path is None:
 
620
            symbol_versioning.warn(
 
621
                'bzr_remote_path is required as of bzr 0.92',
 
622
                DeprecationWarning, stacklevel=2)
 
623
            self._bzr_remote_path = os.environ.get('BZR_REMOTE_PATH', 'bzr')
784
624
 
785
625
    def _accept_bytes(self, bytes):
786
626
        """See SmartClientStreamMedium.accept_bytes."""
787
627
        self._ensure_connection()
788
628
        self._write_to.write(bytes)
789
 
        self._report_activity(len(bytes), 'write')
790
629
 
791
630
    def disconnect(self):
792
631
        """See SmartClientMedium.disconnect()."""
821
660
        """See SmartClientStreamMedium.read_bytes."""
822
661
        if not self._connected:
823
662
            raise errors.MediumNotConnected(self)
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
 
663
        return self._read_from.read(count)
828
664
 
829
665
 
830
666
# Port 4155 is the default port for bzr://, registered with IANA.
831
 
BZR_DEFAULT_INTERFACE = None
 
667
BZR_DEFAULT_INTERFACE = '0.0.0.0'
832
668
BZR_DEFAULT_PORT = 4155
833
669
 
834
670
 
835
671
class SmartTCPClientMedium(SmartClientStreamMedium):
836
672
    """A client medium using TCP."""
837
 
 
 
673
    
838
674
    def __init__(self, host, port, base):
839
675
        """Creates a client that will connect on the first use."""
840
676
        SmartClientStreamMedium.__init__(self, base)
846
682
    def _accept_bytes(self, bytes):
847
683
        """See SmartClientMedium.accept_bytes."""
848
684
        self._ensure_connection()
849
 
        osutils.send_all(self._socket, bytes, self._report_activity)
 
685
        osutils.send_all(self._socket, bytes)
850
686
 
851
687
    def disconnect(self):
852
688
        """See SmartClientMedium.disconnect()."""
860
696
        """Connect this medium if not already connected."""
861
697
        if self._connected:
862
698
            return
 
699
        self._socket = socket.socket()
 
700
        self._socket.setsockopt(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)
863
701
        if self._port is None:
864
702
            port = BZR_DEFAULT_PORT
865
703
        else:
866
704
            port = int(self._port)
867
705
        try:
868
 
            sockaddrs = socket.getaddrinfo(self._host, port, socket.AF_UNSPEC,
869
 
                socket.SOCK_STREAM, 0, 0)
870
 
        except socket.gaierror, (err_num, err_msg):
871
 
            raise errors.ConnectionError("failed to lookup %s:%d: %s" %
872
 
                    (self._host, port, err_msg))
873
 
        # Initialize err in case there are no addresses returned:
874
 
        err = socket.error("no address found for %s" % self._host)
875
 
        for (family, socktype, proto, canonname, sockaddr) in sockaddrs:
876
 
            try:
877
 
                self._socket = socket.socket(family, socktype, proto)
878
 
                self._socket.setsockopt(socket.IPPROTO_TCP,
879
 
                                        socket.TCP_NODELAY, 1)
880
 
                self._socket.connect(sockaddr)
881
 
            except socket.error, err:
882
 
                if self._socket is not None:
883
 
                    self._socket.close()
884
 
                self._socket = None
885
 
                continue
886
 
            break
887
 
        if self._socket is None:
 
706
            self._socket.connect((self._host, port))
 
707
        except socket.error, err:
888
708
            # socket errors either have a (string) or (errno, string) as their
889
709
            # args.
890
710
            if type(err.args) is str:
897
717
 
898
718
    def _flush(self):
899
719
        """See SmartClientStreamMedium._flush().
900
 
 
901
 
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and
 
720
        
 
721
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and 
902
722
        add a means to do a flush, but that can be done in the future.
903
723
        """
904
724
 
906
726
        """See SmartClientMedium.read_bytes."""
907
727
        if not self._connected:
908
728
            raise errors.MediumNotConnected(self)
909
 
        return osutils.read_bytes_from_socket(
910
 
            self._socket, self._report_activity)
 
729
        return self._socket.recv(count)
911
730
 
912
731
 
913
732
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
926
745
 
927
746
    def _accept_bytes(self, bytes):
928
747
        """See SmartClientMediumRequest._accept_bytes.
929
 
 
 
748
        
930
749
        This forwards to self._medium._accept_bytes because we are operating
931
750
        on the mediums stream.
932
751
        """
935
754
    def _finished_reading(self):
936
755
        """See SmartClientMediumRequest._finished_reading.
937
756
 
938
 
        This clears the _current_request on self._medium to allow a new
 
757
        This clears the _current_request on self._medium to allow a new 
939
758
        request to be created.
940
759
        """
941
760
        if self._medium._current_request is not self:
942
761
            raise AssertionError()
943
762
        self._medium._current_request = None
944
 
 
 
763
        
945
764
    def _finished_writing(self):
946
765
        """See SmartClientMediumRequest._finished_writing.
947
766
 
949
768
        """
950
769
        self._medium._flush()
951
770
 
 
771
    def _read_bytes(self, count):
 
772
        """See SmartClientMediumRequest._read_bytes.
 
773
        
 
774
        This forwards to self._medium._read_bytes because we are operating
 
775
        on the mediums stream.
 
776
        """
 
777
        return self._medium._read_bytes(count)
952
778