~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.py

  • Committer: John Arbash Meinel
  • Author(s): Mark Hammond
  • Date: 2008-09-09 17:02:21 UTC
  • mto: This revision was merged to the branch mainline in revision 3697.
  • Revision ID: john@arbash-meinel.com-20080909170221-svim3jw2mrz0amp3
An updated transparent icon for bzr.

Show diffs side-by-side

added added

removed removed

Lines of Context:
27
27
import os
28
28
import socket
29
29
import sys
 
30
import urllib
30
31
 
 
32
from bzrlib.lazy_import import lazy_import
 
33
lazy_import(globals(), """
31
34
from bzrlib import (
32
35
    errors,
 
36
    osutils,
33
37
    symbol_versioning,
34
 
    )
35
 
from bzrlib.smart.protocol import (
36
 
    REQUEST_VERSION_TWO,
37
 
    SmartServerRequestProtocolOne,
38
 
    SmartServerRequestProtocolTwo,
39
 
    )
40
 
 
41
 
try:
42
 
    from bzrlib.transport import ssh
43
 
except errors.ParamikoNotPresent:
44
 
    # no paramiko.  SmartSSHClientMedium will break.
45
 
    pass
46
 
 
47
 
 
48
 
class SmartServerStreamMedium(object):
 
38
    urlutils,
 
39
    )
 
40
from bzrlib.smart import protocol
 
41
from bzrlib.transport import ssh
 
42
""")
 
43
 
 
44
 
 
45
# We must not read any more than 64k at a time so we don't risk "no buffer
 
46
# space available" errors on some platforms.  Windows in particular is likely
 
47
# to give error 10053 or 10055 if we read more than 64k from a socket.
 
48
_MAX_READ_SIZE = 64 * 1024
 
49
 
 
50
 
 
51
def _get_protocol_factory_for_bytes(bytes):
 
52
    """Determine the right protocol factory for 'bytes'.
 
53
 
 
54
    This will return an appropriate protocol factory depending on the version
 
55
    of the protocol being used, as determined by inspecting the given bytes.
 
56
    The bytes should have at least one newline byte (i.e. be a whole line),
 
57
    otherwise it's possible that a request will be incorrectly identified as
 
58
    version 1.
 
59
 
 
60
    Typical use would be::
 
61
 
 
62
         factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
 
63
         server_protocol = factory(transport, write_func, root_client_path)
 
64
         server_protocol.accept_bytes(unused_bytes)
 
65
 
 
66
    :param bytes: a str of bytes of the start of the request.
 
67
    :returns: 2-tuple of (protocol_factory, unused_bytes).  protocol_factory is
 
68
        a callable that takes three args: transport, write_func,
 
69
        root_client_path.  unused_bytes are any bytes that were not part of a
 
70
        protocol version marker.
 
71
    """
 
72
    if bytes.startswith(protocol.MESSAGE_VERSION_THREE):
 
73
        protocol_factory = protocol.build_server_protocol_three
 
74
        bytes = bytes[len(protocol.MESSAGE_VERSION_THREE):]
 
75
    elif bytes.startswith(protocol.REQUEST_VERSION_TWO):
 
76
        protocol_factory = protocol.SmartServerRequestProtocolTwo
 
77
        bytes = bytes[len(protocol.REQUEST_VERSION_TWO):]
 
78
    else:
 
79
        protocol_factory = protocol.SmartServerRequestProtocolOne
 
80
    return protocol_factory, bytes
 
81
 
 
82
 
 
83
def _get_line(read_bytes_func):
 
84
    """Read bytes using read_bytes_func until a newline byte.
 
85
    
 
86
    This isn't particularly efficient, so should only be used when the
 
87
    expected size of the line is quite short.
 
88
    
 
89
    :returns: a tuple of two strs: (line, excess)
 
90
    """
 
91
    newline_pos = -1
 
92
    bytes = ''
 
93
    while newline_pos == -1:
 
94
        new_bytes = read_bytes_func(1)
 
95
        bytes += new_bytes
 
96
        if new_bytes == '':
 
97
            # Ran out of bytes before receiving a complete line.
 
98
            return bytes, ''
 
99
        newline_pos = bytes.find('\n')
 
100
    line = bytes[:newline_pos+1]
 
101
    excess = bytes[newline_pos+1:]
 
102
    return line, excess
 
103
 
 
104
 
 
105
class SmartMedium(object):
 
106
    """Base class for smart protocol media, both client- and server-side."""
 
107
 
 
108
    def __init__(self):
 
109
        self._push_back_buffer = None
 
110
        
 
111
    def _push_back(self, bytes):
 
112
        """Return unused bytes to the medium, because they belong to the next
 
113
        request(s).
 
114
 
 
115
        This sets the _push_back_buffer to the given bytes.
 
116
        """
 
117
        if self._push_back_buffer is not None:
 
118
            raise AssertionError(
 
119
                "_push_back called when self._push_back_buffer is %r"
 
120
                % (self._push_back_buffer,))
 
121
        if bytes == '':
 
122
            return
 
123
        self._push_back_buffer = bytes
 
124
 
 
125
    def _get_push_back_buffer(self):
 
126
        if self._push_back_buffer == '':
 
127
            raise AssertionError(
 
128
                '%s._push_back_buffer should never be the empty string, '
 
129
                'which can be confused with EOF' % (self,))
 
130
        bytes = self._push_back_buffer
 
131
        self._push_back_buffer = None
 
132
        return bytes
 
133
 
 
134
    def read_bytes(self, desired_count):
 
135
        """Read some bytes from this medium.
 
136
 
 
137
        :returns: some bytes, possibly more or less than the number requested
 
138
            in 'desired_count' depending on the medium.
 
139
        """
 
140
        if self._push_back_buffer is not None:
 
141
            return self._get_push_back_buffer()
 
142
        bytes_to_read = min(desired_count, _MAX_READ_SIZE)
 
143
        return self._read_bytes(bytes_to_read)
 
144
 
 
145
    def _read_bytes(self, count):
 
146
        raise NotImplementedError(self._read_bytes)
 
147
 
 
148
    def _get_line(self):
 
149
        """Read bytes from this request's response until a newline byte.
 
150
        
 
151
        This isn't particularly efficient, so should only be used when the
 
152
        expected size of the line is quite short.
 
153
 
 
154
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
155
        """
 
156
        line, excess = _get_line(self.read_bytes)
 
157
        self._push_back(excess)
 
158
        return line
 
159
 
 
160
 
 
161
class SmartServerStreamMedium(SmartMedium):
49
162
    """Handles smart commands coming over a stream.
50
163
 
51
164
    The stream may be a pipe connected to sshd, or a tcp socket, or an
56
169
 
57
170
    The server passes requests through to an underlying backing transport, 
58
171
    which will typically be a LocalTransport looking at the server's filesystem.
 
172
 
 
173
    :ivar _push_back_buffer: a str of bytes that have been read from the stream
 
174
        but not used yet, or None if there are no buffered bytes.  Subclasses
 
175
        should make sure to exhaust this buffer before reading more bytes from
 
176
        the stream.  See also the _push_back method.
59
177
    """
60
178
 
61
 
    def __init__(self, backing_transport):
 
179
    def __init__(self, backing_transport, root_client_path='/'):
62
180
        """Construct new server.
63
181
 
64
182
        :param backing_transport: Transport for the directory served.
65
183
        """
66
184
        # backing_transport could be passed to serve instead of __init__
67
185
        self.backing_transport = backing_transport
 
186
        self.root_client_path = root_client_path
68
187
        self.finished = False
 
188
        SmartMedium.__init__(self)
69
189
 
70
190
    def serve(self):
71
191
        """Serve requests until the client disconnects."""
89
209
 
90
210
        :returns: a SmartServerRequestProtocol.
91
211
        """
92
 
        # Identify the protocol version.
93
212
        bytes = self._get_line()
94
 
        if bytes.startswith(REQUEST_VERSION_TWO):
95
 
            protocol_class = SmartServerRequestProtocolTwo
96
 
            bytes = bytes[len(REQUEST_VERSION_TWO):]
97
 
        else:
98
 
            protocol_class = SmartServerRequestProtocolOne
99
 
        protocol = protocol_class(self.backing_transport, self._write_out)
100
 
        protocol.accept_bytes(bytes)
 
213
        protocol_factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
 
214
        protocol = protocol_factory(
 
215
            self.backing_transport, self._write_out, self.root_client_path)
 
216
        protocol.accept_bytes(unused_bytes)
101
217
        return protocol
102
218
 
103
219
    def _serve_one_request(self, protocol):
116
232
        """Called when an unhandled exception from the protocol occurs."""
117
233
        raise NotImplementedError(self.terminate_due_to_error)
118
234
 
119
 
    def _get_bytes(self, desired_count):
 
235
    def _read_bytes(self, desired_count):
120
236
        """Get some bytes from the medium.
121
237
 
122
238
        :param desired_count: number of bytes we want to read.
123
239
        """
124
 
        raise NotImplementedError(self._get_bytes)
125
 
 
126
 
    def _get_line(self):
127
 
        """Read bytes from this request's response until a newline byte.
128
 
        
129
 
        This isn't particularly efficient, so should only be used when the
130
 
        expected size of the line is quite short.
131
 
 
132
 
        :returns: a string of bytes ending in a newline (byte 0x0A).
133
 
        """
134
 
        # XXX: this duplicates SmartClientRequestProtocolOne._recv_tuple
135
 
        line = ''
136
 
        while not line or line[-1] != '\n':
137
 
            new_char = self._get_bytes(1)
138
 
            line += new_char
139
 
            if new_char == '':
140
 
                # Ran out of bytes before receiving a complete line.
141
 
                break
142
 
        return line
 
240
        raise NotImplementedError(self._read_bytes)
143
241
 
144
242
 
145
243
class SmartServerSocketStreamMedium(SmartServerStreamMedium):
146
244
 
147
 
    def __init__(self, sock, backing_transport):
 
245
    def __init__(self, sock, backing_transport, root_client_path='/'):
148
246
        """Constructor.
149
247
 
150
248
        :param sock: the socket the server will read from.  It will be put
151
249
            into blocking mode.
152
250
        """
153
 
        SmartServerStreamMedium.__init__(self, backing_transport)
154
 
        self.push_back = ''
 
251
        SmartServerStreamMedium.__init__(
 
252
            self, backing_transport, root_client_path=root_client_path)
155
253
        sock.setblocking(True)
156
254
        self.socket = sock
157
255
 
158
256
    def _serve_one_request_unguarded(self, protocol):
159
257
        while protocol.next_read_size():
160
 
            if self.push_back:
161
 
                protocol.accept_bytes(self.push_back)
162
 
                self.push_back = ''
163
 
            else:
164
 
                bytes = self._get_bytes(4096)
165
 
                if bytes == '':
166
 
                    self.finished = True
167
 
                    return
168
 
                protocol.accept_bytes(bytes)
 
258
            # We can safely try to read large chunks.  If there is less data
 
259
            # than _MAX_READ_SIZE ready, the socket wil just return a short
 
260
            # read immediately rather than block.
 
261
            bytes = self.read_bytes(_MAX_READ_SIZE)
 
262
            if bytes == '':
 
263
                self.finished = True
 
264
                return
 
265
            protocol.accept_bytes(bytes)
169
266
        
170
 
        self.push_back = protocol.excess_buffer
 
267
        self._push_back(protocol.unused_data)
171
268
 
172
 
    def _get_bytes(self, desired_count):
 
269
    def _read_bytes(self, desired_count):
173
270
        # We ignore the desired_count because on sockets it's more efficient to
174
 
        # read 4k at a time.
175
 
        return self.socket.recv(4096)
176
 
    
 
271
        # read large chunks (of _MAX_READ_SIZE bytes) at a time.
 
272
        return self.socket.recv(_MAX_READ_SIZE)
 
273
 
177
274
    def terminate_due_to_error(self):
178
 
        """Called when an unhandled exception from the protocol occurs."""
179
275
        # TODO: This should log to a server log file, but no such thing
180
276
        # exists yet.  Andrew Bennetts 2006-09-29.
181
277
        self.socket.close()
182
278
        self.finished = True
183
279
 
184
280
    def _write_out(self, bytes):
185
 
        self.socket.sendall(bytes)
 
281
        osutils.send_all(self.socket, bytes)
186
282
 
187
283
 
188
284
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
207
303
 
208
304
    def _serve_one_request_unguarded(self, protocol):
209
305
        while True:
 
306
            # We need to be careful not to read past the end of the current
 
307
            # request, or else the read from the pipe will block, so we use
 
308
            # protocol.next_read_size().
210
309
            bytes_to_read = protocol.next_read_size()
211
310
            if bytes_to_read == 0:
212
311
                # Finished serving this request.
213
312
                self._out.flush()
214
313
                return
215
 
            bytes = self._get_bytes(bytes_to_read)
 
314
            bytes = self.read_bytes(bytes_to_read)
216
315
            if bytes == '':
217
316
                # Connection has been closed.
218
317
                self.finished = True
220
319
                return
221
320
            protocol.accept_bytes(bytes)
222
321
 
223
 
    def _get_bytes(self, desired_count):
 
322
    def _read_bytes(self, desired_count):
224
323
        return self._in.read(desired_count)
225
324
 
226
325
    def terminate_due_to_error(self):
340
439
        return self._read_bytes(count)
341
440
 
342
441
    def _read_bytes(self, count):
343
 
        """Helper for read_bytes.
 
442
        """Helper for SmartClientMediumRequest.read_bytes.
344
443
 
345
444
        read_bytes checks the state of the request to determing if bytes
346
445
        should be read. After that it hands off to _read_bytes to do the
347
446
        actual read.
 
447
        
 
448
        By default this forwards to self._medium.read_bytes because we are
 
449
        operating on the medium's stream.
348
450
        """
349
 
        raise NotImplementedError(self._read_bytes)
 
451
        return self._medium.read_bytes(count)
350
452
 
351
453
    def read_line(self):
352
 
        """Read bytes from this request's response until a newline byte.
 
454
        line = self._read_line()
 
455
        if not line.endswith('\n'):
 
456
            # end of file encountered reading from server
 
457
            raise errors.ConnectionReset(
 
458
                "please check connectivity and permissions",
 
459
                "(and try -Dhpss if further diagnosis is required)")
 
460
        return line
 
461
 
 
462
    def _read_line(self):
 
463
        """Helper for SmartClientMediumRequest.read_line.
353
464
        
354
 
        This isn't particularly efficient, so should only be used when the
355
 
        expected size of the line is quite short.
356
 
 
357
 
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
465
        By default this forwards to self._medium._get_line because we are
 
466
        operating on the medium's stream.
358
467
        """
359
 
        # XXX: this duplicates SmartClientRequestProtocolOne._recv_tuple
360
 
        line = ''
361
 
        while not line or line[-1] != '\n':
362
 
            new_char = self.read_bytes(1)
363
 
            line += new_char
364
 
            if new_char == '':
365
 
                raise errors.SmartProtocolError(
366
 
                    'unexpected end of file reading from server')
367
 
        return line
368
 
 
369
 
 
370
 
class SmartClientMedium(object):
 
468
        return self._medium._get_line()
 
469
 
 
470
 
 
471
class SmartClientMedium(SmartMedium):
371
472
    """Smart client is a medium for sending smart protocol requests over."""
372
473
 
 
474
    def __init__(self, base):
 
475
        super(SmartClientMedium, self).__init__()
 
476
        self.base = base
 
477
        self._protocol_version_error = None
 
478
        self._protocol_version = None
 
479
        self._done_hello = False
 
480
        # Be optimistic: we assume the remote end can accept new remote
 
481
        # requests until we get an error saying otherwise.
 
482
        # _remote_version_is_before tracks the bzr version the remote side
 
483
        # can be based on what we've seen so far.
 
484
        self._remote_version_is_before = None
 
485
 
 
486
    def _is_remote_before(self, version_tuple):
 
487
        """Is it possible the remote side supports RPCs for a given version?
 
488
 
 
489
        Typical use::
 
490
 
 
491
            needed_version = (1, 2)
 
492
            if medium._is_remote_before(needed_version):
 
493
                fallback_to_pre_1_2_rpc()
 
494
            else:
 
495
                try:
 
496
                    do_1_2_rpc()
 
497
                except UnknownSmartMethod:
 
498
                    medium._remember_remote_is_before(needed_version)
 
499
                    fallback_to_pre_1_2_rpc()
 
500
 
 
501
        :seealso: _remember_remote_is_before
 
502
        """
 
503
        if self._remote_version_is_before is None:
 
504
            # So far, the remote side seems to support everything
 
505
            return False
 
506
        return version_tuple >= self._remote_version_is_before
 
507
 
 
508
    def _remember_remote_is_before(self, version_tuple):
 
509
        """Tell this medium that the remote side is older the given version.
 
510
 
 
511
        :seealso: _is_remote_before
 
512
        """
 
513
        if (self._remote_version_is_before is not None and
 
514
            version_tuple > self._remote_version_is_before):
 
515
            raise AssertionError(
 
516
                "_remember_remote_is_before(%r) called, but "
 
517
                "_remember_remote_is_before(%r) was called previously."
 
518
                % (version_tuple, self._remote_version_is_before))
 
519
        self._remote_version_is_before = version_tuple
 
520
 
 
521
    def protocol_version(self):
 
522
        """Find out if 'hello' smart request works."""
 
523
        if self._protocol_version_error is not None:
 
524
            raise self._protocol_version_error
 
525
        if not self._done_hello:
 
526
            try:
 
527
                medium_request = self.get_request()
 
528
                # Send a 'hello' request in protocol version one, for maximum
 
529
                # backwards compatibility.
 
530
                client_protocol = protocol.SmartClientRequestProtocolOne(medium_request)
 
531
                client_protocol.query_version()
 
532
                self._done_hello = True
 
533
            except errors.SmartProtocolError, e:
 
534
                # Cache the error, just like we would cache a successful
 
535
                # result.
 
536
                self._protocol_version_error = e
 
537
                raise
 
538
        return '2'
 
539
 
 
540
    def should_probe(self):
 
541
        """Should RemoteBzrDirFormat.probe_transport send a smart request on
 
542
        this medium?
 
543
 
 
544
        Some transports are unambiguously smart-only; there's no need to check
 
545
        if the transport is able to carry smart requests, because that's all
 
546
        it is for.  In those cases, this method should return False.
 
547
 
 
548
        But some HTTP transports can sometimes fail to carry smart requests,
 
549
        but still be usuable for accessing remote bzrdirs via plain file
 
550
        accesses.  So for those transports, their media should return True here
 
551
        so that RemoteBzrDirFormat can determine if it is appropriate for that
 
552
        transport.
 
553
        """
 
554
        return False
 
555
 
373
556
    def disconnect(self):
374
557
        """If this medium maintains a persistent connection, close it.
375
558
        
376
559
        The default implementation does nothing.
377
560
        """
378
561
        
 
562
    def remote_path_from_transport(self, transport):
 
563
        """Convert transport into a path suitable for using in a request.
 
564
        
 
565
        Note that the resulting remote path doesn't encode the host name or
 
566
        anything but path, so it is only safe to use it in requests sent over
 
567
        the medium from the matching transport.
 
568
        """
 
569
        medium_base = urlutils.join(self.base, '/')
 
570
        rel_url = urlutils.relative_url(medium_base, transport.base)
 
571
        return urllib.unquote(rel_url)
 
572
 
379
573
 
380
574
class SmartClientStreamMedium(SmartClientMedium):
381
575
    """Stream based medium common class.
386
580
    receive bytes.
387
581
    """
388
582
 
389
 
    def __init__(self):
 
583
    def __init__(self, base):
 
584
        SmartClientMedium.__init__(self, base)
390
585
        self._current_request = None
391
586
 
392
587
    def accept_bytes(self, bytes):
414
609
        """
415
610
        return SmartClientStreamMediumRequest(self)
416
611
 
417
 
    def read_bytes(self, count):
418
 
        return self._read_bytes(count)
419
 
 
420
612
 
421
613
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
422
614
    """A client medium using simple pipes.
424
616
    This client does not manage the pipes: it assumes they will always be open.
425
617
    """
426
618
 
427
 
    def __init__(self, readable_pipe, writeable_pipe):
428
 
        SmartClientStreamMedium.__init__(self)
 
619
    def __init__(self, readable_pipe, writeable_pipe, base):
 
620
        SmartClientStreamMedium.__init__(self, base)
429
621
        self._readable_pipe = readable_pipe
430
622
        self._writeable_pipe = writeable_pipe
431
623
 
446
638
    """A client medium using SSH."""
447
639
    
448
640
    def __init__(self, host, port=None, username=None, password=None,
449
 
            vendor=None, bzr_remote_path=None):
 
641
            base=None, vendor=None, bzr_remote_path=None):
450
642
        """Creates a client that will connect on the first use.
451
643
        
452
644
        :param vendor: An optional override for the ssh vendor to use. See
453
645
            bzrlib.transport.ssh for details on ssh vendors.
454
646
        """
455
 
        SmartClientStreamMedium.__init__(self)
 
647
        SmartClientStreamMedium.__init__(self, base)
456
648
        self._connected = False
457
649
        self._host = host
458
650
        self._password = password
507
699
        """See SmartClientStreamMedium.read_bytes."""
508
700
        if not self._connected:
509
701
            raise errors.MediumNotConnected(self)
510
 
        return self._read_from.read(count)
 
702
        bytes_to_read = min(count, _MAX_READ_SIZE)
 
703
        return self._read_from.read(bytes_to_read)
 
704
 
 
705
 
 
706
# Port 4155 is the default port for bzr://, registered with IANA.
 
707
BZR_DEFAULT_INTERFACE = '0.0.0.0'
 
708
BZR_DEFAULT_PORT = 4155
511
709
 
512
710
 
513
711
class SmartTCPClientMedium(SmartClientStreamMedium):
514
712
    """A client medium using TCP."""
515
713
    
516
 
    def __init__(self, host, port):
 
714
    def __init__(self, host, port, base):
517
715
        """Creates a client that will connect on the first use."""
518
 
        SmartClientStreamMedium.__init__(self)
 
716
        SmartClientStreamMedium.__init__(self, base)
519
717
        self._connected = False
520
718
        self._host = host
521
719
        self._port = port
524
722
    def _accept_bytes(self, bytes):
525
723
        """See SmartClientMedium.accept_bytes."""
526
724
        self._ensure_connection()
527
 
        self._socket.sendall(bytes)
 
725
        osutils.send_all(self._socket, bytes)
528
726
 
529
727
    def disconnect(self):
530
728
        """See SmartClientMedium.disconnect()."""
540
738
            return
541
739
        self._socket = socket.socket()
542
740
        self._socket.setsockopt(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)
543
 
        result = self._socket.connect_ex((self._host, int(self._port)))
544
 
        if result:
 
741
        if self._port is None:
 
742
            port = BZR_DEFAULT_PORT
 
743
        else:
 
744
            port = int(self._port)
 
745
        try:
 
746
            self._socket.connect((self._host, port))
 
747
        except socket.error, err:
 
748
            # socket errors either have a (string) or (errno, string) as their
 
749
            # args.
 
750
            if type(err.args) is str:
 
751
                err_msg = err.args
 
752
            else:
 
753
                err_msg = err.args[1]
545
754
            raise errors.ConnectionError("failed to connect to %s:%d: %s" %
546
 
                    (self._host, self._port, os.strerror(result)))
 
755
                    (self._host, port, err_msg))
547
756
        self._connected = True
548
757
 
549
758
    def _flush(self):
557
766
        """See SmartClientMedium.read_bytes."""
558
767
        if not self._connected:
559
768
            raise errors.MediumNotConnected(self)
560
 
        return self._socket.recv(count)
 
769
        # We ignore the desired_count because on sockets it's more efficient to
 
770
        # read large chunks (of _MAX_READ_SIZE bytes) at a time.
 
771
        return self._socket.recv(_MAX_READ_SIZE)
561
772
 
562
773
 
563
774
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
588
799
        This clears the _current_request on self._medium to allow a new 
589
800
        request to be created.
590
801
        """
591
 
        assert self._medium._current_request is self
 
802
        if self._medium._current_request is not self:
 
803
            raise AssertionError()
592
804
        self._medium._current_request = None
593
805
        
594
806
    def _finished_writing(self):
598
810
        """
599
811
        self._medium._flush()
600
812
 
601
 
    def _read_bytes(self, count):
602
 
        """See SmartClientMediumRequest._read_bytes.
603
 
        
604
 
        This forwards to self._medium._read_bytes because we are operating
605
 
        on the mediums stream.
606
 
        """
607
 
        return self._medium._read_bytes(count)
608