~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/medium.py

  • Committer: Tarmac
  • Author(s): Vincent Ladeuil
  • Date: 2017-01-30 14:42:05 UTC
  • mfrom: (6620.1.1 trunk)
  • Revision ID: tarmac-20170130144205-r8fh2xpmiuxyozpv
Merge  2.7 into trunk including fix for bug #1657238 [r=vila]

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006 Canonical Ltd
 
1
# Copyright (C) 2006-2011 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., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 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
from __future__ import absolute_import
 
28
 
 
29
import errno
27
30
import os
 
31
import sys
 
32
import time
 
33
 
 
34
import bzrlib
 
35
from bzrlib.lazy_import import lazy_import
 
36
lazy_import(globals(), """
 
37
import select
28
38
import socket
29
 
import sys
 
39
import thread
 
40
import weakref
30
41
 
31
 
from bzrlib import errors
32
 
from bzrlib.smart.protocol import (
33
 
    REQUEST_VERSION_TWO,
34
 
    SmartServerRequestProtocolOne,
35
 
    SmartServerRequestProtocolTwo,
 
42
from bzrlib import (
 
43
    debug,
 
44
    errors,
 
45
    trace,
 
46
    transport,
 
47
    ui,
 
48
    urlutils,
36
49
    )
37
 
 
38
 
try:
39
 
    from bzrlib.transport import ssh
40
 
except errors.ParamikoNotPresent:
41
 
    # no paramiko.  SmartSSHClientMedium will break.
42
 
    pass
43
 
 
44
 
 
45
 
class SmartServerStreamMedium(object):
 
50
from bzrlib.i18n import gettext
 
51
from bzrlib.smart import client, protocol, request, signals, vfs
 
52
from bzrlib.transport import ssh
 
53
""")
 
54
from bzrlib import osutils
 
55
 
 
56
# Throughout this module buffer size parameters are either limited to be at
 
57
# most _MAX_READ_SIZE, or are ignored and _MAX_READ_SIZE is used instead.
 
58
# For this module's purposes, MAX_SOCKET_CHUNK is a reasonable size for reads
 
59
# from non-sockets as well.
 
60
_MAX_READ_SIZE = osutils.MAX_SOCKET_CHUNK
 
61
 
 
62
def _get_protocol_factory_for_bytes(bytes):
 
63
    """Determine the right protocol factory for 'bytes'.
 
64
 
 
65
    This will return an appropriate protocol factory depending on the version
 
66
    of the protocol being used, as determined by inspecting the given bytes.
 
67
    The bytes should have at least one newline byte (i.e. be a whole line),
 
68
    otherwise it's possible that a request will be incorrectly identified as
 
69
    version 1.
 
70
 
 
71
    Typical use would be::
 
72
 
 
73
         factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
 
74
         server_protocol = factory(transport, write_func, root_client_path)
 
75
         server_protocol.accept_bytes(unused_bytes)
 
76
 
 
77
    :param bytes: a str of bytes of the start of the request.
 
78
    :returns: 2-tuple of (protocol_factory, unused_bytes).  protocol_factory is
 
79
        a callable that takes three args: transport, write_func,
 
80
        root_client_path.  unused_bytes are any bytes that were not part of a
 
81
        protocol version marker.
 
82
    """
 
83
    if bytes.startswith(protocol.MESSAGE_VERSION_THREE):
 
84
        protocol_factory = protocol.build_server_protocol_three
 
85
        bytes = bytes[len(protocol.MESSAGE_VERSION_THREE):]
 
86
    elif bytes.startswith(protocol.REQUEST_VERSION_TWO):
 
87
        protocol_factory = protocol.SmartServerRequestProtocolTwo
 
88
        bytes = bytes[len(protocol.REQUEST_VERSION_TWO):]
 
89
    else:
 
90
        protocol_factory = protocol.SmartServerRequestProtocolOne
 
91
    return protocol_factory, bytes
 
92
 
 
93
 
 
94
def _get_line(read_bytes_func):
 
95
    """Read bytes using read_bytes_func until a newline byte.
 
96
 
 
97
    This isn't particularly efficient, so should only be used when the
 
98
    expected size of the line is quite short.
 
99
 
 
100
    :returns: a tuple of two strs: (line, excess)
 
101
    """
 
102
    newline_pos = -1
 
103
    bytes = ''
 
104
    while newline_pos == -1:
 
105
        new_bytes = read_bytes_func(1)
 
106
        bytes += new_bytes
 
107
        if new_bytes == '':
 
108
            # Ran out of bytes before receiving a complete line.
 
109
            return bytes, ''
 
110
        newline_pos = bytes.find('\n')
 
111
    line = bytes[:newline_pos+1]
 
112
    excess = bytes[newline_pos+1:]
 
113
    return line, excess
 
114
 
 
115
 
 
116
class SmartMedium(object):
 
117
    """Base class for smart protocol media, both client- and server-side."""
 
118
 
 
119
    def __init__(self):
 
120
        self._push_back_buffer = None
 
121
 
 
122
    def _push_back(self, bytes):
 
123
        """Return unused bytes to the medium, because they belong to the next
 
124
        request(s).
 
125
 
 
126
        This sets the _push_back_buffer to the given bytes.
 
127
        """
 
128
        if self._push_back_buffer is not None:
 
129
            raise AssertionError(
 
130
                "_push_back called when self._push_back_buffer is %r"
 
131
                % (self._push_back_buffer,))
 
132
        if bytes == '':
 
133
            return
 
134
        self._push_back_buffer = bytes
 
135
 
 
136
    def _get_push_back_buffer(self):
 
137
        if self._push_back_buffer == '':
 
138
            raise AssertionError(
 
139
                '%s._push_back_buffer should never be the empty string, '
 
140
                'which can be confused with EOF' % (self,))
 
141
        bytes = self._push_back_buffer
 
142
        self._push_back_buffer = None
 
143
        return bytes
 
144
 
 
145
    def read_bytes(self, desired_count):
 
146
        """Read some bytes from this medium.
 
147
 
 
148
        :returns: some bytes, possibly more or less than the number requested
 
149
            in 'desired_count' depending on the medium.
 
150
        """
 
151
        if self._push_back_buffer is not None:
 
152
            return self._get_push_back_buffer()
 
153
        bytes_to_read = min(desired_count, _MAX_READ_SIZE)
 
154
        return self._read_bytes(bytes_to_read)
 
155
 
 
156
    def _read_bytes(self, count):
 
157
        raise NotImplementedError(self._read_bytes)
 
158
 
 
159
    def _get_line(self):
 
160
        """Read bytes from this request's response until a newline byte.
 
161
 
 
162
        This isn't particularly efficient, so should only be used when the
 
163
        expected size of the line is quite short.
 
164
 
 
165
        :returns: a string of bytes ending in a newline (byte 0x0A).
 
166
        """
 
167
        line, excess = _get_line(self.read_bytes)
 
168
        self._push_back(excess)
 
169
        return line
 
170
 
 
171
    def _report_activity(self, bytes, direction):
 
172
        """Notify that this medium has activity.
 
173
 
 
174
        Implementations should call this from all methods that actually do IO.
 
175
        Be careful that it's not called twice, if one method is implemented on
 
176
        top of another.
 
177
 
 
178
        :param bytes: Number of bytes read or written.
 
179
        :param direction: 'read' or 'write' or None.
 
180
        """
 
181
        ui.ui_factory.report_transport_activity(self, bytes, direction)
 
182
 
 
183
 
 
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
class SmartServerStreamMedium(SmartMedium):
46
193
    """Handles smart commands coming over a stream.
47
194
 
48
195
    The stream may be a pipe connected to sshd, or a tcp socket, or an
51
198
    One instance is created for each connected client; it can serve multiple
52
199
    requests in the lifetime of the connection.
53
200
 
54
 
    The server passes requests through to an underlying backing transport, 
 
201
    The server passes requests through to an underlying backing transport,
55
202
    which will typically be a LocalTransport looking at the server's filesystem.
 
203
 
 
204
    :ivar _push_back_buffer: a str of bytes that have been read from the stream
 
205
        but not used yet, or None if there are no buffered bytes.  Subclasses
 
206
        should make sure to exhaust this buffer before reading more bytes from
 
207
        the stream.  See also the _push_back method.
56
208
    """
57
209
 
58
 
    def __init__(self, backing_transport):
 
210
    _timer = time.time
 
211
 
 
212
    def __init__(self, backing_transport, root_client_path='/', timeout=None):
59
213
        """Construct new server.
60
214
 
61
215
        :param backing_transport: Transport for the directory served.
62
216
        """
63
217
        # backing_transport could be passed to serve instead of __init__
64
218
        self.backing_transport = backing_transport
 
219
        self.root_client_path = root_client_path
65
220
        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
        SmartMedium.__init__(self)
66
226
 
67
227
    def serve(self):
68
228
        """Serve requests until the client disconnects."""
73
233
            while not self.finished:
74
234
                server_protocol = self._build_protocol()
75
235
                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
76
242
        except Exception, e:
77
243
            stderr.write("%s terminating on exception %s\n" % (self, e))
78
244
            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)
79
269
 
80
270
    def _build_protocol(self):
81
271
        """Identifies the version of the incoming request, and returns an
86
276
 
87
277
        :returns: a SmartServerRequestProtocol.
88
278
        """
89
 
        # Identify the protocol version.
 
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
90
283
        bytes = self._get_line()
91
 
        if bytes.startswith(REQUEST_VERSION_TWO):
92
 
            protocol_class = SmartServerRequestProtocolTwo
93
 
            bytes = bytes[len(REQUEST_VERSION_TWO):]
94
 
        else:
95
 
            protocol_class = SmartServerRequestProtocolOne
96
 
        protocol = protocol_class(self.backing_transport, self._write_out)
97
 
        protocol.accept_bytes(bytes)
 
284
        protocol_factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
 
285
        protocol = protocol_factory(
 
286
            self.backing_transport, self._write_out, self.root_client_path)
 
287
        protocol.accept_bytes(unused_bytes)
98
288
        return protocol
99
289
 
 
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
 
100
321
    def _serve_one_request(self, protocol):
101
322
        """Read one request from input, process, send back a response.
102
 
        
 
323
 
103
324
        :param protocol: a SmartServerRequestProtocol.
104
325
        """
 
326
        if protocol is None:
 
327
            return
105
328
        try:
106
329
            self._serve_one_request_unguarded(protocol)
107
330
        except KeyboardInterrupt:
113
336
        """Called when an unhandled exception from the protocol occurs."""
114
337
        raise NotImplementedError(self.terminate_due_to_error)
115
338
 
116
 
    def _get_bytes(self, desired_count):
 
339
    def _read_bytes(self, desired_count):
117
340
        """Get some bytes from the medium.
118
341
 
119
342
        :param desired_count: number of bytes we want to read.
120
343
        """
121
 
        raise NotImplementedError(self._get_bytes)
122
 
 
123
 
    def _get_line(self):
124
 
        """Read bytes from this request's response until a newline byte.
125
 
        
126
 
        This isn't particularly efficient, so should only be used when the
127
 
        expected size of the line is quite short.
128
 
 
129
 
        :returns: a string of bytes ending in a newline (byte 0x0A).
130
 
        """
131
 
        # XXX: this duplicates SmartClientRequestProtocolOne._recv_tuple
132
 
        line = ''
133
 
        while not line or line[-1] != '\n':
134
 
            new_char = self._get_bytes(1)
135
 
            line += new_char
136
 
            if new_char == '':
137
 
                # Ran out of bytes before receiving a complete line.
138
 
                break
139
 
        return line
 
344
        raise NotImplementedError(self._read_bytes)
140
345
 
141
346
 
142
347
class SmartServerSocketStreamMedium(SmartServerStreamMedium):
143
348
 
144
 
    def __init__(self, sock, backing_transport):
 
349
    def __init__(self, sock, backing_transport, root_client_path='/',
 
350
                 timeout=None):
145
351
        """Constructor.
146
352
 
147
353
        :param sock: the socket the server will read from.  It will be put
148
354
            into blocking mode.
149
355
        """
150
 
        SmartServerStreamMedium.__init__(self, backing_transport)
151
 
        self.push_back = ''
 
356
        SmartServerStreamMedium.__init__(
 
357
            self, backing_transport, root_client_path=root_client_path,
 
358
            timeout=timeout)
152
359
        sock.setblocking(True)
153
360
        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)
154
373
 
155
374
    def _serve_one_request_unguarded(self, protocol):
156
375
        while protocol.next_read_size():
157
 
            if self.push_back:
158
 
                protocol.accept_bytes(self.push_back)
159
 
                self.push_back = ''
160
 
            else:
161
 
                bytes = self._get_bytes(4096)
162
 
                if bytes == '':
163
 
                    self.finished = True
164
 
                    return
165
 
                protocol.accept_bytes(bytes)
166
 
        
167
 
        self.push_back = protocol.excess_buffer
168
 
 
169
 
    def _get_bytes(self, desired_count):
170
 
        # We ignore the desired_count because on sockets it's more efficient to
171
 
        # read 4k at a time.
172
 
        return self.socket.recv(4096)
173
 
    
 
376
            # We can safely try to read large chunks.  If there is less data
 
377
            # than MAX_SOCKET_CHUNK ready, the socket will just return a
 
378
            # short read immediately rather than block.
 
379
            bytes = self.read_bytes(osutils.MAX_SOCKET_CHUNK)
 
380
            if bytes == '':
 
381
                self.finished = True
 
382
                return
 
383
            protocol.accept_bytes(bytes)
 
384
 
 
385
        self._push_back(protocol.unused_data)
 
386
 
 
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
    def _read_bytes(self, desired_count):
 
404
        return osutils.read_bytes_from_socket(
 
405
            self.socket, self._report_activity)
 
406
 
174
407
    def terminate_due_to_error(self):
175
 
        """Called when an unhandled exception from the protocol occurs."""
176
408
        # TODO: This should log to a server log file, but no such thing
177
409
        # exists yet.  Andrew Bennetts 2006-09-29.
178
410
        self.socket.close()
179
411
        self.finished = True
180
412
 
181
413
    def _write_out(self, bytes):
182
 
        self.socket.sendall(bytes)
 
414
        tstart = osutils.timer_func()
 
415
        osutils.send_all(self.socket, bytes, self._report_activity)
 
416
        if 'hpss' in debug.debug_flags:
 
417
            thread_id = thread.get_ident()
 
418
            trace.mutter('%12s: [%s] %d bytes to the socket in %.3fs'
 
419
                         % ('wrote', thread_id, len(bytes),
 
420
                            osutils.timer_func() - tstart))
183
421
 
184
422
 
185
423
class SmartServerPipeStreamMedium(SmartServerStreamMedium):
186
424
 
187
 
    def __init__(self, in_file, out_file, backing_transport):
 
425
    def __init__(self, in_file, out_file, backing_transport, timeout=None):
188
426
        """Construct new server.
189
427
 
190
428
        :param in_file: Python file from which requests can be read.
191
429
        :param out_file: Python file to write responses.
192
430
        :param backing_transport: Transport for the directory served.
193
431
        """
194
 
        SmartServerStreamMedium.__init__(self, backing_transport)
 
432
        SmartServerStreamMedium.__init__(self, backing_transport,
 
433
            timeout=timeout)
195
434
        if sys.platform == 'win32':
196
435
            # force binary mode for files
197
436
            import msvcrt
202
441
        self._in = in_file
203
442
        self._out = out_file
204
443
 
 
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
 
205
455
    def _serve_one_request_unguarded(self, protocol):
206
456
        while True:
 
457
            # We need to be careful not to read past the end of the current
 
458
            # request, or else the read from the pipe will block, so we use
 
459
            # protocol.next_read_size().
207
460
            bytes_to_read = protocol.next_read_size()
208
461
            if bytes_to_read == 0:
209
462
                # Finished serving this request.
210
463
                self._out.flush()
211
464
                return
212
 
            bytes = self._get_bytes(bytes_to_read)
 
465
            bytes = self.read_bytes(bytes_to_read)
213
466
            if bytes == '':
214
467
                # Connection has been closed.
215
468
                self.finished = True
217
470
                return
218
471
            protocol.accept_bytes(bytes)
219
472
 
220
 
    def _get_bytes(self, desired_count):
 
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
    def _read_bytes(self, desired_count):
221
495
        return self._in.read(desired_count)
222
496
 
223
497
    def terminate_due_to_error(self):
243
517
    request.finished_reading()
244
518
 
245
519
    It is up to the individual SmartClientMedium whether multiple concurrent
246
 
    requests can exist. See SmartClientMedium.get_request to obtain instances 
247
 
    of SmartClientMediumRequest, and the concrete Medium you are using for 
 
520
    requests can exist. See SmartClientMedium.get_request to obtain instances
 
521
    of SmartClientMediumRequest, and the concrete Medium you are using for
248
522
    details on concurrency and pipelining.
249
523
    """
250
524
 
259
533
    def accept_bytes(self, bytes):
260
534
        """Accept bytes for inclusion in this request.
261
535
 
262
 
        This method may not be be called after finished_writing() has been
 
536
        This method may not be called after finished_writing() has been
263
537
        called.  It depends upon the Medium whether or not the bytes will be
264
538
        immediately transmitted. Message based Mediums will tend to buffer the
265
539
        bytes until finished_writing() is called.
296
570
    def _finished_reading(self):
297
571
        """Helper for finished_reading.
298
572
 
299
 
        finished_reading checks the state of the request to determine if 
 
573
        finished_reading checks the state of the request to determine if
300
574
        finished_reading is allowed, and if it is hands off to _finished_reading
301
575
        to perform the action.
302
576
        """
316
590
    def _finished_writing(self):
317
591
        """Helper for finished_writing.
318
592
 
319
 
        finished_writing checks the state of the request to determine if 
 
593
        finished_writing checks the state of the request to determine if
320
594
        finished_writing is allowed, and if it is hands off to _finished_writing
321
595
        to perform the action.
322
596
        """
337
611
        return self._read_bytes(count)
338
612
 
339
613
    def _read_bytes(self, count):
340
 
        """Helper for read_bytes.
 
614
        """Helper for SmartClientMediumRequest.read_bytes.
341
615
 
342
616
        read_bytes checks the state of the request to determing if bytes
343
617
        should be read. After that it hands off to _read_bytes to do the
344
618
        actual read.
 
619
 
 
620
        By default this forwards to self._medium.read_bytes because we are
 
621
        operating on the medium's stream.
345
622
        """
346
 
        raise NotImplementedError(self._read_bytes)
 
623
        return self._medium.read_bytes(count)
347
624
 
348
625
    def read_line(self):
349
 
        """Read bytes from this request's response until a newline byte.
350
 
        
351
 
        This isn't particularly efficient, so should only be used when the
352
 
        expected size of the line is quite short.
353
 
 
354
 
        :returns: a string of bytes ending in a newline (byte 0x0A).
355
 
        """
356
 
        # XXX: this duplicates SmartClientRequestProtocolOne._recv_tuple
357
 
        line = ''
358
 
        while not line or line[-1] != '\n':
359
 
            new_char = self.read_bytes(1)
360
 
            line += new_char
361
 
            if new_char == '':
362
 
                raise errors.SmartProtocolError(
363
 
                    'unexpected end of file reading from server')
 
626
        line = self._read_line()
 
627
        if not line.endswith('\n'):
 
628
            # end of file encountered reading from server
 
629
            raise errors.ConnectionReset(
 
630
                "Unexpected end of message. Please check connectivity "
 
631
                "and permissions, and report a bug if problems persist.")
364
632
        return line
365
633
 
366
 
 
367
 
class SmartClientMedium(object):
 
634
    def _read_line(self):
 
635
        """Helper for SmartClientMediumRequest.read_line.
 
636
 
 
637
        By default this forwards to self._medium._get_line because we are
 
638
        operating on the medium's stream.
 
639
        """
 
640
        return self._medium._get_line()
 
641
 
 
642
 
 
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
class _DebugCounter(object):
 
663
    """An object that counts the HPSS calls made to each client medium.
 
664
 
 
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.
 
668
    """
 
669
 
 
670
    def __init__(self):
 
671
        self.counts = weakref.WeakKeyDictionary()
 
672
        client._SmartClient.hooks.install_named_hook(
 
673
            'call', self.increment_call_count, 'hpss call counter')
 
674
        bzrlib.global_state.cleanups.add_cleanup(self.flush_all)
 
675
 
 
676
    def track(self, medium):
 
677
        """Start tracking calls made to a medium.
 
678
 
 
679
        This only keeps a weakref to the medium, so shouldn't affect the
 
680
        medium's lifetime.
 
681
        """
 
682
        medium_repr = repr(medium)
 
683
        # Add this medium to the WeakKeyDictionary
 
684
        self.counts[medium] = dict(count=0, vfs_count=0,
 
685
                                   medium_repr=medium_repr)
 
686
        # Weakref callbacks are fired in reverse order of their association
 
687
        # with the referenced object.  So we add a weakref *after* adding to
 
688
        # the WeakKeyDict so that we can report the value from it before the
 
689
        # entry is removed by the WeakKeyDict's own callback.
 
690
        ref = weakref.ref(medium, self.done)
 
691
 
 
692
    def increment_call_count(self, params):
 
693
        # Increment the count in the WeakKeyDictionary
 
694
        value = self.counts[params.medium]
 
695
        value['count'] += 1
 
696
        try:
 
697
            request_method = request.request_handlers.get(params.method)
 
698
        except KeyError:
 
699
            # A method we don't know about doesn't count as a VFS method.
 
700
            return
 
701
        if issubclass(request_method, vfs.VfsRequest):
 
702
            value['vfs_count'] += 1
 
703
 
 
704
    def done(self, ref):
 
705
        value = self.counts[ref]
 
706
        count, vfs_count, medium_repr = (
 
707
            value['count'], value['vfs_count'], value['medium_repr'])
 
708
        # In case this callback is invoked for the same ref twice (by the
 
709
        # weakref callback and by the atexit function), set the call count back
 
710
        # to 0 so this item won't be reported twice.
 
711
        value['count'] = 0
 
712
        value['vfs_count'] = 0
 
713
        if count != 0:
 
714
            trace.note(gettext('HPSS calls: {0} ({1} vfs) {2}').format(
 
715
                       count, vfs_count, medium_repr))
 
716
 
 
717
    def flush_all(self):
 
718
        for ref in list(self.counts.keys()):
 
719
            self.done(ref)
 
720
 
 
721
_debug_counter = None
 
722
_vfs_refuser = None
 
723
 
 
724
 
 
725
class SmartClientMedium(SmartMedium):
368
726
    """Smart client is a medium for sending smart protocol requests over."""
369
727
 
 
728
    def __init__(self, base):
 
729
        super(SmartClientMedium, self).__init__()
 
730
        self.base = base
 
731
        self._protocol_version_error = None
 
732
        self._protocol_version = None
 
733
        self._done_hello = False
 
734
        # Be optimistic: we assume the remote end can accept new remote
 
735
        # requests until we get an error saying otherwise.
 
736
        # _remote_version_is_before tracks the bzr version the remote side
 
737
        # can be based on what we've seen so far.
 
738
        self._remote_version_is_before = None
 
739
        # Install debug hook function if debug flag is set.
 
740
        if 'hpss' in debug.debug_flags:
 
741
            global _debug_counter
 
742
            if _debug_counter is None:
 
743
                _debug_counter = _DebugCounter()
 
744
            _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
 
 
750
    def _is_remote_before(self, version_tuple):
 
751
        """Is it possible the remote side supports RPCs for a given version?
 
752
 
 
753
        Typical use::
 
754
 
 
755
            needed_version = (1, 2)
 
756
            if medium._is_remote_before(needed_version):
 
757
                fallback_to_pre_1_2_rpc()
 
758
            else:
 
759
                try:
 
760
                    do_1_2_rpc()
 
761
                except UnknownSmartMethod:
 
762
                    medium._remember_remote_is_before(needed_version)
 
763
                    fallback_to_pre_1_2_rpc()
 
764
 
 
765
        :seealso: _remember_remote_is_before
 
766
        """
 
767
        if self._remote_version_is_before is None:
 
768
            # So far, the remote side seems to support everything
 
769
            return False
 
770
        return version_tuple >= self._remote_version_is_before
 
771
 
 
772
    def _remember_remote_is_before(self, version_tuple):
 
773
        """Tell this medium that the remote side is older the given version.
 
774
 
 
775
        :seealso: _is_remote_before
 
776
        """
 
777
        if (self._remote_version_is_before is not None and
 
778
            version_tuple > self._remote_version_is_before):
 
779
            # We have been told that the remote side is older than some version
 
780
            # which is newer than a previously supplied older-than version.
 
781
            # This indicates that some smart verb call is not guarded
 
782
            # appropriately (it should simply not have been tried).
 
783
            trace.mutter(
 
784
                "_remember_remote_is_before(%r) called, but "
 
785
                "_remember_remote_is_before(%r) was called previously."
 
786
                , version_tuple, self._remote_version_is_before)
 
787
            if 'hpss' in debug.debug_flags:
 
788
                ui.ui_factory.show_warning(
 
789
                    "_remember_remote_is_before(%r) called, but "
 
790
                    "_remember_remote_is_before(%r) was called previously."
 
791
                    % (version_tuple, self._remote_version_is_before))
 
792
            return
 
793
        self._remote_version_is_before = version_tuple
 
794
 
 
795
    def protocol_version(self):
 
796
        """Find out if 'hello' smart request works."""
 
797
        if self._protocol_version_error is not None:
 
798
            raise self._protocol_version_error
 
799
        if not self._done_hello:
 
800
            try:
 
801
                medium_request = self.get_request()
 
802
                # Send a 'hello' request in protocol version one, for maximum
 
803
                # backwards compatibility.
 
804
                client_protocol = protocol.SmartClientRequestProtocolOne(medium_request)
 
805
                client_protocol.query_version()
 
806
                self._done_hello = True
 
807
            except errors.SmartProtocolError, e:
 
808
                # Cache the error, just like we would cache a successful
 
809
                # result.
 
810
                self._protocol_version_error = e
 
811
                raise
 
812
        return '2'
 
813
 
 
814
    def should_probe(self):
 
815
        """Should RemoteBzrDirFormat.probe_transport send a smart request on
 
816
        this medium?
 
817
 
 
818
        Some transports are unambiguously smart-only; there's no need to check
 
819
        if the transport is able to carry smart requests, because that's all
 
820
        it is for.  In those cases, this method should return False.
 
821
 
 
822
        But some HTTP transports can sometimes fail to carry smart requests,
 
823
        but still be usuable for accessing remote bzrdirs via plain file
 
824
        accesses.  So for those transports, their media should return True here
 
825
        so that RemoteBzrDirFormat can determine if it is appropriate for that
 
826
        transport.
 
827
        """
 
828
        return False
 
829
 
370
830
    def disconnect(self):
371
831
        """If this medium maintains a persistent connection, close it.
372
 
        
 
832
 
373
833
        The default implementation does nothing.
374
834
        """
375
 
        
 
835
 
 
836
    def remote_path_from_transport(self, transport):
 
837
        """Convert transport into a path suitable for using in a request.
 
838
 
 
839
        Note that the resulting remote path doesn't encode the host name or
 
840
        anything but path, so it is only safe to use it in requests sent over
 
841
        the medium from the matching transport.
 
842
        """
 
843
        medium_base = urlutils.join(self.base, '/')
 
844
        rel_url = urlutils.relative_url(medium_base, transport.base)
 
845
        return urlutils.unquote(rel_url)
 
846
 
376
847
 
377
848
class SmartClientStreamMedium(SmartClientMedium):
378
849
    """Stream based medium common class.
383
854
    receive bytes.
384
855
    """
385
856
 
386
 
    def __init__(self):
 
857
    def __init__(self, base):
 
858
        SmartClientMedium.__init__(self, base)
387
859
        self._current_request = None
388
860
 
389
861
    def accept_bytes(self, bytes):
397
869
 
398
870
    def _flush(self):
399
871
        """Flush the output stream.
400
 
        
 
872
 
401
873
        This method is used by the SmartClientStreamMediumRequest to ensure that
402
874
        all data for a request is sent, to avoid long timeouts or deadlocks.
403
875
        """
411
883
        """
412
884
        return SmartClientStreamMediumRequest(self)
413
885
 
414
 
    def read_bytes(self, count):
415
 
        return self._read_bytes(count)
 
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
416
893
 
417
894
 
418
895
class SmartSimplePipesClientMedium(SmartClientStreamMedium):
419
896
    """A client medium using simple pipes.
420
 
    
 
897
 
421
898
    This client does not manage the pipes: it assumes they will always be open.
422
899
    """
423
900
 
424
 
    def __init__(self, readable_pipe, writeable_pipe):
425
 
        SmartClientStreamMedium.__init__(self)
 
901
    def __init__(self, readable_pipe, writeable_pipe, base):
 
902
        SmartClientStreamMedium.__init__(self, base)
426
903
        self._readable_pipe = readable_pipe
427
904
        self._writeable_pipe = writeable_pipe
428
905
 
429
906
    def _accept_bytes(self, bytes):
430
907
        """See SmartClientStreamMedium.accept_bytes."""
431
 
        self._writeable_pipe.write(bytes)
 
908
        try:
 
909
            self._writeable_pipe.write(bytes)
 
910
        except IOError, e:
 
911
            if e.errno in (errno.EINVAL, errno.EPIPE):
 
912
                raise errors.ConnectionReset(
 
913
                    "Error trying to write to subprocess", e)
 
914
            raise
 
915
        self._report_activity(len(bytes), 'write')
432
916
 
433
917
    def _flush(self):
434
918
        """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.
435
922
        self._writeable_pipe.flush()
436
923
 
437
924
    def _read_bytes(self, count):
438
925
        """See SmartClientStreamMedium._read_bytes."""
439
 
        return self._readable_pipe.read(count)
 
926
        bytes_to_read = min(count, _MAX_READ_SIZE)
 
927
        bytes = self._readable_pipe.read(bytes_to_read)
 
928
        self._report_activity(len(bytes), 'read')
 
929
        return bytes
 
930
 
 
931
 
 
932
class SSHParams(object):
 
933
    """A set of parameters for starting a remote bzr via SSH."""
 
934
 
 
935
    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
440
942
 
441
943
 
442
944
class SmartSSHClientMedium(SmartClientStreamMedium):
443
 
    """A client medium using SSH."""
444
 
    
445
 
    def __init__(self, host, port=None, username=None, password=None,
446
 
            vendor=None):
 
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):
447
952
        """Creates a client that will connect on the first use.
448
 
        
 
953
 
 
954
        :param ssh_params: A SSHParams instance.
449
955
        :param vendor: An optional override for the ssh vendor to use. See
450
956
            bzrlib.transport.ssh for details on ssh vendors.
451
957
        """
452
 
        SmartClientStreamMedium.__init__(self)
453
 
        self._connected = False
454
 
        self._host = host
455
 
        self._password = password
456
 
        self._port = port
457
 
        self._username = username
458
 
        self._read_from = None
 
958
        self._real_medium = None
 
959
        self._ssh_params = ssh_params
 
960
        # for the benefit of progress making a short description of this
 
961
        # transport
 
962
        self._scheme = 'bzr+ssh'
 
963
        # SmartClientStreamMedium stores the repr of this object in its
 
964
        # _DebugCounter so we have to store all the values used in our repr
 
965
        # method before calling the super init.
 
966
        SmartClientStreamMedium.__init__(self, base)
 
967
        self._vendor = vendor
459
968
        self._ssh_connection = None
460
 
        self._vendor = vendor
461
 
        self._write_to = None
 
969
 
 
970
    def __repr__(self):
 
971
        if self._ssh_params.port is None:
 
972
            maybe_port = ''
 
973
        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/)" % (
 
980
            self.__class__.__name__,
 
981
            self._scheme,
 
982
            maybe_user,
 
983
            self._ssh_params.host,
 
984
            maybe_port)
462
985
 
463
986
    def _accept_bytes(self, bytes):
464
987
        """See SmartClientStreamMedium.accept_bytes."""
465
988
        self._ensure_connection()
466
 
        self._write_to.write(bytes)
 
989
        self._real_medium.accept_bytes(bytes)
467
990
 
468
991
    def disconnect(self):
469
992
        """See SmartClientMedium.disconnect()."""
470
 
        if not self._connected:
471
 
            return
472
 
        self._read_from.close()
473
 
        self._write_to.close()
474
 
        self._ssh_connection.close()
475
 
        self._connected = False
 
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
476
999
 
477
1000
    def _ensure_connection(self):
478
1001
        """Connect this medium if not already connected."""
479
 
        if self._connected:
 
1002
        if self._real_medium is not None:
480
1003
            return
481
 
        executable = os.environ.get('BZR_REMOTE_PATH', 'bzr')
482
1004
        if self._vendor is None:
483
1005
            vendor = ssh._get_ssh_vendor()
484
1006
        else:
485
1007
            vendor = self._vendor
486
 
        self._ssh_connection = vendor.connect_ssh(self._username,
487
 
                self._password, self._host, self._port,
488
 
                command=[executable, 'serve', '--inet', '--directory=/',
489
 
                         '--allow-writes'])
490
 
        self._read_from, self._write_to = \
491
 
            self._ssh_connection.get_filelike_channels()
492
 
        self._connected = True
 
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',
 
1012
                         '--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)
493
1027
 
494
1028
    def _flush(self):
495
1029
        """See SmartClientStreamMedium._flush()."""
496
 
        self._write_to.flush()
 
1030
        self._real_medium._flush()
497
1031
 
498
1032
    def _read_bytes(self, count):
499
1033
        """See SmartClientStreamMedium.read_bytes."""
500
 
        if not self._connected:
 
1034
        if self._real_medium is None:
501
1035
            raise errors.MediumNotConnected(self)
502
 
        return self._read_from.read(count)
503
 
 
504
 
 
505
 
class SmartTCPClientMedium(SmartClientStreamMedium):
506
 
    """A client medium using TCP."""
507
 
    
508
 
    def __init__(self, host, port):
509
 
        """Creates a client that will connect on the first use."""
510
 
        SmartClientStreamMedium.__init__(self)
 
1036
        return self._real_medium.read_bytes(count)
 
1037
 
 
1038
 
 
1039
# Port 4155 is the default port for bzr://, registered with IANA.
 
1040
BZR_DEFAULT_INTERFACE = None
 
1041
BZR_DEFAULT_PORT = 4155
 
1042
 
 
1043
 
 
1044
class SmartClientSocketMedium(SmartClientStreamMedium):
 
1045
    """A client medium using a socket.
 
1046
 
 
1047
    This class isn't usable directly.  Use one of its subclasses instead.
 
1048
    """
 
1049
 
 
1050
    def __init__(self, base):
 
1051
        SmartClientStreamMedium.__init__(self, base)
 
1052
        self._socket = None
511
1053
        self._connected = False
512
 
        self._host = host
513
 
        self._port = port
514
 
        self._socket = None
515
1054
 
516
1055
    def _accept_bytes(self, bytes):
517
1056
        """See SmartClientMedium.accept_bytes."""
518
1057
        self._ensure_connection()
519
 
        self._socket.sendall(bytes)
 
1058
        osutils.send_all(self._socket, bytes, self._report_activity)
 
1059
 
 
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)
520
1078
 
521
1079
    def disconnect(self):
522
1080
        """See SmartClientMedium.disconnect()."""
526
1084
        self._socket = None
527
1085
        self._connected = False
528
1086
 
 
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
 
529
1097
    def _ensure_connection(self):
530
1098
        """Connect this medium if not already connected."""
531
1099
        if self._connected:
532
1100
            return
533
 
        self._socket = socket.socket()
534
 
        self._socket.setsockopt(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)
535
 
        result = self._socket.connect_ex((self._host, int(self._port)))
536
 
        if result:
 
1101
        if self._port is None:
 
1102
            port = BZR_DEFAULT_PORT
 
1103
        else:
 
1104
            port = int(self._port)
 
1105
        try:
 
1106
            sockaddrs = socket.getaddrinfo(self._host, port, socket.AF_UNSPEC,
 
1107
                socket.SOCK_STREAM, 0, 0)
 
1108
        except socket.gaierror, (err_num, err_msg):
 
1109
            raise errors.ConnectionError("failed to lookup %s:%d: %s" %
 
1110
                    (self._host, port, err_msg))
 
1111
        # Initialize err in case there are no addresses returned:
 
1112
        err = socket.error("no address found for %s" % self._host)
 
1113
        for (family, socktype, proto, canonname, sockaddr) in sockaddrs:
 
1114
            try:
 
1115
                self._socket = socket.socket(family, socktype, proto)
 
1116
                self._socket.setsockopt(socket.IPPROTO_TCP,
 
1117
                                        socket.TCP_NODELAY, 1)
 
1118
                self._socket.connect(sockaddr)
 
1119
            except socket.error, err:
 
1120
                if self._socket is not None:
 
1121
                    self._socket.close()
 
1122
                self._socket = None
 
1123
                continue
 
1124
            break
 
1125
        if self._socket is None:
 
1126
            # socket errors either have a (string) or (errno, string) as their
 
1127
            # args.
 
1128
            if type(err.args) is str:
 
1129
                err_msg = err.args
 
1130
            else:
 
1131
                err_msg = err.args[1]
537
1132
            raise errors.ConnectionError("failed to connect to %s:%d: %s" %
538
 
                    (self._host, self._port, os.strerror(result)))
539
 
        self._connected = True
540
 
 
541
 
    def _flush(self):
542
 
        """See SmartClientStreamMedium._flush().
543
 
        
544
 
        For TCP we do no flushing. We may want to turn off TCP_NODELAY and 
545
 
        add a means to do a flush, but that can be done in the future.
546
 
        """
547
 
 
548
 
    def _read_bytes(self, count):
549
 
        """See SmartClientMedium.read_bytes."""
550
 
        if not self._connected:
551
 
            raise errors.MediumNotConnected(self)
552
 
        return self._socket.recv(count)
 
1133
                    (self._host, port, err_msg))
 
1134
        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
553
1154
 
554
1155
 
555
1156
class SmartClientStreamMediumRequest(SmartClientMediumRequest):
568
1169
 
569
1170
    def _accept_bytes(self, bytes):
570
1171
        """See SmartClientMediumRequest._accept_bytes.
571
 
        
 
1172
 
572
1173
        This forwards to self._medium._accept_bytes because we are operating
573
1174
        on the mediums stream.
574
1175
        """
577
1178
    def _finished_reading(self):
578
1179
        """See SmartClientMediumRequest._finished_reading.
579
1180
 
580
 
        This clears the _current_request on self._medium to allow a new 
 
1181
        This clears the _current_request on self._medium to allow a new
581
1182
        request to be created.
582
1183
        """
583
 
        assert self._medium._current_request is self
 
1184
        if self._medium._current_request is not self:
 
1185
            raise AssertionError()
584
1186
        self._medium._current_request = None
585
 
        
 
1187
 
586
1188
    def _finished_writing(self):
587
1189
        """See SmartClientMediumRequest._finished_writing.
588
1190
 
589
1191
        This invokes self._medium._flush to ensure all bytes are transmitted.
590
1192
        """
591
1193
        self._medium._flush()
592
 
 
593
 
    def _read_bytes(self, count):
594
 
        """See SmartClientMediumRequest._read_bytes.
595
 
        
596
 
        This forwards to self._medium._read_bytes because we are operating
597
 
        on the mediums stream.
598
 
        """
599
 
        return self._medium._read_bytes(count)
600