5752.3.8
by John Arbash Meinel
Merge bzr.dev 5764 to resolve release-notes (aka NEWS) conflicts |
1 |
# Copyright (C) 2006-2011 Canonical Ltd
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
2 |
#
|
3 |
# This program is free software; you can redistribute it and/or modify
|
|
4 |
# it under the terms of the GNU General Public License as published by
|
|
5 |
# the Free Software Foundation; either version 2 of the License, or
|
|
6 |
# (at your option) any later version.
|
|
7 |
#
|
|
8 |
# This program is distributed in the hope that it will be useful,
|
|
9 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
10 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
11 |
# GNU General Public License for more details.
|
|
12 |
#
|
|
13 |
# You should have received a copy of the GNU General Public License
|
|
14 |
# along with this program; if not, write to the Free Software
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
15 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
16 |
|
2018.5.19
by Andrew Bennetts
Add docstrings to all the new modules, and a few other places. |
17 |
"""The 'medium' layer for the smart servers and clients.
|
18 |
||
19 |
"Medium" here is the noun meaning "a means of transmission", not the adjective
|
|
20 |
for "the quality between big and small."
|
|
21 |
||
22 |
Media carry the bytes of the requests somehow (e.g. via TCP, wrapped in HTTP, or
|
|
23 |
over SSH), and pass them to and from the protocol logic. See the overview in
|
|
24 |
bzrlib/transport/smart/__init__.py.
|
|
25 |
"""
|
|
26 |
||
6133.4.10
by John Arbash Meinel
Handle the win32 not-a-socket errors. I think this is ~ready. At least for feedback. |
27 |
import errno |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
28 |
import os |
2018.5.162
by Andrew Bennetts
Add some missing _ensure_real calls, and a missing import. |
29 |
import sys |
6133.4.8
by John Arbash Meinel
Refactor a bit. Use a common _wait_for_descriptor code. |
30 |
import time |
3431.3.11
by Andrew Bennetts
Push remote_path_from_transport logic into SmartClientMedium, removing special-casing of bzr+http from _SmartClient. |
31 |
import urllib |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
32 |
|
5222.2.9
by Robert Collins
Write up some doc about bzrlib.initialize. |
33 |
import bzrlib |
3530.1.1
by John Arbash Meinel
Make bzrlib.smart use lazy imports. |
34 |
from bzrlib.lazy_import import lazy_import |
35 |
lazy_import(globals(), """ |
|
6133.4.7
by John Arbash Meinel
Currently causes 5-10 failures in bt.per_branch.test_branch Remote out of 150 tests. |
36 |
import select
|
5011.3.11
by Andrew Bennetts
Consolidate changes, try to minimise unnecessary changes and tidy up those that kept. |
37 |
import socket
|
4913.1.1
by John Arbash Meinel
Switch to using thread.get_ident() which is available on all python versions. |
38 |
import thread
|
3731.2.4
by Andrew Bennetts
Minor tweaks. |
39 |
import weakref
|
4889.2.2
by John Arbash Meinel
Add a -Dhpssthread debug flag to include thread.ident info. |
40 |
|
1551.18.17
by Aaron Bentley
Introduce bzr_remote_path configuration variable |
41 |
from bzrlib import (
|
3731.2.1
by Andrew Bennetts
Show total HPSS calls (if any) on stderr when -Dhpss is active. |
42 |
debug,
|
1551.18.17
by Aaron Bentley
Introduce bzr_remote_path configuration variable |
43 |
errors,
|
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
44 |
trace,
|
3958.1.1
by Andrew Bennetts
Report traffic on smart media as transport activity. |
45 |
ui,
|
3431.3.11
by Andrew Bennetts
Push remote_path_from_transport logic into SmartClientMedium, removing special-casing of bzr+http from _SmartClient. |
46 |
urlutils,
|
1551.18.17
by Aaron Bentley
Introduce bzr_remote_path configuration variable |
47 |
)
|
6138.3.4
by Jonathan Riddell
add gettext() to uses of trace.note() |
48 |
from bzrlib.i18n import gettext
|
6133.4.61
by John Arbash Meinel
Expose infrastructure so that we can test the Inet portion of the server. |
49 |
from bzrlib.smart import client, protocol, request, signals, vfs
|
3066.2.1
by John Arbash Meinel
We don't require paramiko for bzr+ssh. |
50 |
from bzrlib.transport import ssh
|
3530.1.1
by John Arbash Meinel
Make bzrlib.smart use lazy imports. |
51 |
""") |
6133.4.55
by John Arbash Meinel
Now we've implemented the basic structure. |
52 |
from bzrlib import osutils |
2018.5.17
by Andrew Bennetts
Paramaterise the commands handled by SmartServerRequestHandler. |
53 |
|
5011.3.3
by Martin
Reintroduce EINTR handling only for socket object functions and general cleanup |
54 |
# Throughout this module buffer size parameters are either limited to be at
|
5011.3.11
by Andrew Bennetts
Consolidate changes, try to minimise unnecessary changes and tidy up those that kept. |
55 |
# most _MAX_READ_SIZE, or are ignored and _MAX_READ_SIZE is used instead.
|
56 |
# For this module's purposes, MAX_SOCKET_CHUNK is a reasonable size for reads
|
|
57 |
# from non-sockets as well.
|
|
58 |
_MAX_READ_SIZE = osutils.MAX_SOCKET_CHUNK |
|
3565.1.3
by Andrew Bennetts
Define a _MAX_READ_SIZE constant as suggested by John's review. |
59 |
|
3245.4.16
by Andrew Bennetts
Remove duplication of request version identification logic in wsgi.py |
60 |
def _get_protocol_factory_for_bytes(bytes): |
61 |
"""Determine the right protocol factory for 'bytes'.
|
|
62 |
||
63 |
This will return an appropriate protocol factory depending on the version
|
|
64 |
of the protocol being used, as determined by inspecting the given bytes.
|
|
65 |
The bytes should have at least one newline byte (i.e. be a whole line),
|
|
66 |
otherwise it's possible that a request will be incorrectly identified as
|
|
67 |
version 1.
|
|
68 |
||
69 |
Typical use would be::
|
|
70 |
||
71 |
factory, unused_bytes = _get_protocol_factory_for_bytes(bytes)
|
|
72 |
server_protocol = factory(transport, write_func, root_client_path)
|
|
73 |
server_protocol.accept_bytes(unused_bytes)
|
|
74 |
||
75 |
:param bytes: a str of bytes of the start of the request.
|
|
76 |
:returns: 2-tuple of (protocol_factory, unused_bytes). protocol_factory is
|
|
77 |
a callable that takes three args: transport, write_func,
|
|
78 |
root_client_path. unused_bytes are any bytes that were not part of a
|
|
79 |
protocol version marker.
|
|
80 |
"""
|
|
3530.1.1
by John Arbash Meinel
Make bzrlib.smart use lazy imports. |
81 |
if bytes.startswith(protocol.MESSAGE_VERSION_THREE): |
82 |
protocol_factory = protocol.build_server_protocol_three |
|
83 |
bytes = bytes[len(protocol.MESSAGE_VERSION_THREE):] |
|
84 |
elif bytes.startswith(protocol.REQUEST_VERSION_TWO): |
|
85 |
protocol_factory = protocol.SmartServerRequestProtocolTwo |
|
86 |
bytes = bytes[len(protocol.REQUEST_VERSION_TWO):] |
|
3245.4.16
by Andrew Bennetts
Remove duplication of request version identification logic in wsgi.py |
87 |
else: |
3530.1.1
by John Arbash Meinel
Make bzrlib.smart use lazy imports. |
88 |
protocol_factory = protocol.SmartServerRequestProtocolOne |
3245.4.16
by Andrew Bennetts
Remove duplication of request version identification logic in wsgi.py |
89 |
return protocol_factory, bytes |
90 |
||
91 |
||
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
92 |
def _get_line(read_bytes_func): |
93 |
"""Read bytes using read_bytes_func until a newline byte.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
94 |
|
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
95 |
This isn't particularly efficient, so should only be used when the
|
96 |
expected size of the line is quite short.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
97 |
|
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
98 |
:returns: a tuple of two strs: (line, excess)
|
99 |
"""
|
|
100 |
newline_pos = -1 |
|
101 |
bytes = '' |
|
102 |
while newline_pos == -1: |
|
103 |
new_bytes = read_bytes_func(1) |
|
104 |
bytes += new_bytes |
|
105 |
if new_bytes == '': |
|
106 |
# Ran out of bytes before receiving a complete line.
|
|
107 |
return bytes, '' |
|
108 |
newline_pos = bytes.find('\n') |
|
109 |
line = bytes[:newline_pos+1] |
|
110 |
excess = bytes[newline_pos+1:] |
|
111 |
return line, excess |
|
112 |
||
113 |
||
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
114 |
class SmartMedium(object): |
115 |
"""Base class for smart protocol media, both client- and server-side."""
|
|
116 |
||
117 |
def __init__(self): |
|
118 |
self._push_back_buffer = None |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
119 |
|
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
120 |
def _push_back(self, bytes): |
121 |
"""Return unused bytes to the medium, because they belong to the next
|
|
122 |
request(s).
|
|
123 |
||
124 |
This sets the _push_back_buffer to the given bytes.
|
|
125 |
"""
|
|
126 |
if self._push_back_buffer is not None: |
|
127 |
raise AssertionError( |
|
128 |
"_push_back called when self._push_back_buffer is %r" |
|
129 |
% (self._push_back_buffer,)) |
|
130 |
if bytes == '': |
|
131 |
return
|
|
132 |
self._push_back_buffer = bytes |
|
133 |
||
134 |
def _get_push_back_buffer(self): |
|
135 |
if self._push_back_buffer == '': |
|
136 |
raise AssertionError( |
|
137 |
'%s._push_back_buffer should never be the empty string, ' |
|
138 |
'which can be confused with EOF' % (self,)) |
|
139 |
bytes = self._push_back_buffer |
|
140 |
self._push_back_buffer = None |
|
141 |
return bytes |
|
142 |
||
143 |
def read_bytes(self, desired_count): |
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
144 |
"""Read some bytes from this medium.
|
145 |
||
146 |
:returns: some bytes, possibly more or less than the number requested
|
|
147 |
in 'desired_count' depending on the medium.
|
|
148 |
"""
|
|
149 |
if self._push_back_buffer is not None: |
|
150 |
return self._get_push_back_buffer() |
|
3565.1.3
by Andrew Bennetts
Define a _MAX_READ_SIZE constant as suggested by John's review. |
151 |
bytes_to_read = min(desired_count, _MAX_READ_SIZE) |
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
152 |
return self._read_bytes(bytes_to_read) |
153 |
||
154 |
def _read_bytes(self, count): |
|
155 |
raise NotImplementedError(self._read_bytes) |
|
156 |
||
157 |
def _get_line(self): |
|
158 |
"""Read bytes from this request's response until a newline byte.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
159 |
|
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
160 |
This isn't particularly efficient, so should only be used when the
|
161 |
expected size of the line is quite short.
|
|
162 |
||
163 |
:returns: a string of bytes ending in a newline (byte 0x0A).
|
|
164 |
"""
|
|
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
165 |
line, excess = _get_line(self.read_bytes) |
166 |
self._push_back(excess) |
|
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
167 |
return line |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
168 |
|
3958.1.1
by Andrew Bennetts
Report traffic on smart media as transport activity. |
169 |
def _report_activity(self, bytes, direction): |
170 |
"""Notify that this medium has activity.
|
|
171 |
||
172 |
Implementations should call this from all methods that actually do IO.
|
|
173 |
Be careful that it's not called twice, if one method is implemented on
|
|
174 |
top of another.
|
|
175 |
||
176 |
:param bytes: Number of bytes read or written.
|
|
177 |
:param direction: 'read' or 'write' or None.
|
|
178 |
"""
|
|
179 |
ui.ui_factory.report_transport_activity(self, bytes, direction) |
|
180 |
||
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
181 |
|
6133.4.10
by John Arbash Meinel
Handle the win32 not-a-socket errors. I think this is ~ready. At least for feedback. |
182 |
_bad_file_descriptor = (errno.EBADF,) |
183 |
if sys.platform == 'win32': |
|
184 |
# Given on Windows if you pass a closed socket to select.select. Probably
|
|
185 |
# also given if you pass a file handle to select.
|
|
186 |
WSAENOTSOCK = 10038 |
|
187 |
_bad_file_descriptor += (WSAENOTSOCK,) |
|
188 |
||
189 |
||
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
190 |
class SmartServerStreamMedium(SmartMedium): |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
191 |
"""Handles smart commands coming over a stream.
|
192 |
||
193 |
The stream may be a pipe connected to sshd, or a tcp socket, or an
|
|
194 |
in-process fifo for testing.
|
|
195 |
||
196 |
One instance is created for each connected client; it can serve multiple
|
|
197 |
requests in the lifetime of the connection.
|
|
198 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
199 |
The server passes requests through to an underlying backing transport,
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
200 |
which will typically be a LocalTransport looking at the server's filesystem.
|
3236.3.4
by Andrew Bennetts
Rename 'push_back' attribute to '_push_back_buffer', add some docstrings, and remove a little bit of redundant code from SmartServerSocketStreamMedium._serve_one_request_unguarded. |
201 |
|
202 |
:ivar _push_back_buffer: a str of bytes that have been read from the stream
|
|
203 |
but not used yet, or None if there are no buffered bytes. Subclasses
|
|
204 |
should make sure to exhaust this buffer before reading more bytes from
|
|
205 |
the stream. See also the _push_back method.
|
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
206 |
"""
|
207 |
||
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
208 |
_timer = time.time |
209 |
||
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
210 |
def __init__(self, backing_transport, root_client_path='/', timeout=None): |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
211 |
"""Construct new server.
|
212 |
||
213 |
:param backing_transport: Transport for the directory served.
|
|
214 |
"""
|
|
215 |
# backing_transport could be passed to serve instead of __init__
|
|
216 |
self.backing_transport = backing_transport |
|
2692.1.11
by Andrew Bennetts
Improve test coverage by making SmartTCPServer_for_testing by default create a server that does not serve the backing transport's root at its own root. This mirrors the way most HTTP smart servers are configured. |
217 |
self.root_client_path = root_client_path |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
218 |
self.finished = False |
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
219 |
if timeout is None: |
6133.4.26
by John Arbash Meinel
get rid of the default timeout parameters. |
220 |
raise AssertionError('You must supply a timeout.') |
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
221 |
self._client_timeout = timeout |
222 |
self._client_poll_timeout = min(timeout / 10.0, 1.0) |
|
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
223 |
SmartMedium.__init__(self) |
3236.3.5
by Andrew Bennetts
Add _get_push_back_buffer helper. |
224 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
225 |
def serve(self): |
226 |
"""Serve requests until the client disconnects."""
|
|
227 |
# Keep a reference to stderr because the sys module's globals get set to
|
|
228 |
# None during interpreter shutdown.
|
|
229 |
from sys import stderr |
|
230 |
try: |
|
231 |
while not self.finished: |
|
2432.2.3
by Andrew Bennetts
Merge from bzr.dev. |
232 |
server_protocol = self._build_protocol() |
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
233 |
# TODO: This seems inelegant:
|
234 |
if server_protocol is None: |
|
235 |
# We could 'continue' only to notice that self.finished is
|
|
236 |
# True...
|
|
237 |
break
|
|
2018.5.14
by Andrew Bennetts
Move SmartTCPServer to smart/server.py, and SmartServerRequestHandler to smart/request.py. |
238 |
self._serve_one_request(server_protocol) |
6133.4.11
by John Arbash Meinel
It turns out that if we don't explicitly close the socket, it hangs around somewhere. |
239 |
except errors.ConnectionTimeout, e: |
240 |
trace.note('%s' % (e,)) |
|
6133.4.12
by John Arbash Meinel
Don't traceback for timeouts, and give better documentation about why we don't just select. |
241 |
trace.log_exception_quietly() |
6133.5.2
by John Arbash Meinel
Rename _close to _disconnect_client. This makes it more obvious. |
242 |
self._disconnect_client() |
6133.4.12
by John Arbash Meinel
Don't traceback for timeouts, and give better documentation about why we don't just select. |
243 |
# We reported it, no reason to make a big fuss.
|
244 |
return
|
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
245 |
except Exception, e: |
246 |
stderr.write("%s terminating on exception %s\n" % (self, e)) |
|
247 |
raise
|
|
6133.4.61
by John Arbash Meinel
Expose infrastructure so that we can test the Inet portion of the server. |
248 |
self._disconnect_client() |
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
249 |
|
6133.4.44
by John Arbash Meinel
Move the code into bzrlib.smart.signals. |
250 |
def _stop_gracefully(self): |
251 |
"""When we finish this message, stop looking for more."""
|
|
252 |
trace.mutter('Stopping %s' % (self,)) |
|
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
253 |
self.finished = True |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
254 |
|
6133.5.2
by John Arbash Meinel
Rename _close to _disconnect_client. This makes it more obvious. |
255 |
def _disconnect_client(self): |
6133.4.11
by John Arbash Meinel
It turns out that if we don't explicitly close the socket, it hangs around somewhere. |
256 |
"""Close the current connection. We stopped due to a timeout/etc."""
|
257 |
# The default implementation is a no-op, because that is all we used to
|
|
258 |
# do when disconnecting from a client. I suppose we never had the
|
|
259 |
# *server* initiate a disconnect, before
|
|
260 |
||
261 |
def _wait_for_bytes_with_timeout(self, timeout_seconds): |
|
262 |
"""Wait for more bytes to be read, but timeout if none available.
|
|
263 |
||
264 |
This allows us to detect idle connections, and stop trying to read from
|
|
265 |
them, without setting the socket itself to non-blocking. This also
|
|
266 |
allows us to specify when we watch for idle timeouts.
|
|
267 |
||
268 |
:return: Did we timeout? (True if we timed out, False if there is data
|
|
269 |
to be read)
|
|
270 |
"""
|
|
271 |
raise NotImplementedError(self._wait_for_bytes_with_timeout) |
|
272 |
||
2432.2.2
by Andrew Bennetts
Smart server mediums now detect which protocol version a request is and dispatch accordingly. |
273 |
def _build_protocol(self): |
2432.2.8
by Andrew Bennetts
NEWS entry, greatly improved docstring in bzrlib.smart. |
274 |
"""Identifies the version of the incoming request, and returns an
|
275 |
a protocol object that can interpret it.
|
|
276 |
||
277 |
If more bytes than the version prefix of the request are read, they will
|
|
278 |
be fed into the protocol before it is returned.
|
|
279 |
||
280 |
:returns: a SmartServerRequestProtocol.
|
|
281 |
"""
|
|
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
282 |
self._wait_for_bytes_with_timeout(self._client_timeout) |
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
283 |
if self.finished: |
284 |
# We're stopping, so don't try to do any more work
|
|
285 |
return None |
|
2432.2.7
by Andrew Bennetts
Use less confusing version strings, and define REQUEST_VERSION_TWO/RESPONSE_VERSION_TWO constants for them. |
286 |
bytes = self._get_line() |
3245.4.16
by Andrew Bennetts
Remove duplication of request version identification logic in wsgi.py |
287 |
protocol_factory, unused_bytes = _get_protocol_factory_for_bytes(bytes) |
3245.4.14
by Andrew Bennetts
Merge from bzr.dev (via loom thread). |
288 |
protocol = protocol_factory( |
2692.1.11
by Andrew Bennetts
Improve test coverage by making SmartTCPServer_for_testing by default create a server that does not serve the backing transport's root at its own root. This mirrors the way most HTTP smart servers are configured. |
289 |
self.backing_transport, self._write_out, self.root_client_path) |
3245.4.16
by Andrew Bennetts
Remove duplication of request version identification logic in wsgi.py |
290 |
protocol.accept_bytes(unused_bytes) |
2432.2.2
by Andrew Bennetts
Smart server mediums now detect which protocol version a request is and dispatch accordingly. |
291 |
return protocol |
292 |
||
6133.4.8
by John Arbash Meinel
Refactor a bit. Use a common _wait_for_descriptor code. |
293 |
def _wait_on_descriptor(self, fd, timeout_seconds): |
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
294 |
"""select() on a file descriptor, waiting for nonblocking read()
|
295 |
||
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
296 |
This will raise a ConnectionTimeout exception if we do not get a
|
297 |
readable handle before timeout_seconds.
|
|
298 |
:return: None
|
|
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
299 |
"""
|
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
300 |
t_end = self._timer() + timeout_seconds |
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
301 |
poll_timeout = min(timeout_seconds, self._client_poll_timeout) |
302 |
rs = xs = None |
|
6133.4.43
by John Arbash Meinel
Initial implementation of SIGHUP for bzr serve. bug #795025. |
303 |
while not rs and not xs and self._timer() < t_end: |
304 |
if self.finished: |
|
305 |
return
|
|
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
306 |
try: |
6133.4.39
by John Arbash Meinel
pass fd in the 'check for errors' section of select. |
307 |
rs, _, xs = select.select([fd], [], [fd], poll_timeout) |
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
308 |
except (select.error, socket.error) as e: |
309 |
err = getattr(e, 'errno', None) |
|
310 |
if err is None and getattr(e, 'args', None) is not None: |
|
311 |
# select.error doesn't have 'errno', it just has args[0]
|
|
6133.4.8
by John Arbash Meinel
Refactor a bit. Use a common _wait_for_descriptor code. |
312 |
err = e.args[0] |
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
313 |
if err in _bad_file_descriptor: |
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
314 |
return # Not a socket indicates read() will fail |
6133.4.41
by John Arbash Meinel
Re-order the loop a bit. Move the exception handling into the loop, so that we can handle EINTR. |
315 |
elif err == errno.EINTR: |
316 |
# Interrupted, keep looping.
|
|
317 |
continue
|
|
318 |
raise
|
|
6133.4.40
by John Arbash Meinel
Treat an 'errfds' as just something we want to try and read from later. |
319 |
if rs or xs: |
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
320 |
return
|
321 |
raise errors.ConnectionTimeout('disconnecting client after %.1f seconds' |
|
322 |
% (timeout_seconds,)) |
|
6133.4.8
by John Arbash Meinel
Refactor a bit. Use a common _wait_for_descriptor code. |
323 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
324 |
def _serve_one_request(self, protocol): |
325 |
"""Read one request from input, process, send back a response.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
326 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
327 |
:param protocol: a SmartServerRequestProtocol.
|
328 |
"""
|
|
329 |
try: |
|
330 |
self._serve_one_request_unguarded(protocol) |
|
331 |
except KeyboardInterrupt: |
|
332 |
raise
|
|
333 |
except Exception, e: |
|
334 |
self.terminate_due_to_error() |
|
335 |
||
336 |
def terminate_due_to_error(self): |
|
337 |
"""Called when an unhandled exception from the protocol occurs."""
|
|
338 |
raise NotImplementedError(self.terminate_due_to_error) |
|
339 |
||
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
340 |
def _read_bytes(self, desired_count): |
2432.2.2
by Andrew Bennetts
Smart server mediums now detect which protocol version a request is and dispatch accordingly. |
341 |
"""Get some bytes from the medium.
|
342 |
||
343 |
:param desired_count: number of bytes we want to read.
|
|
344 |
"""
|
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
345 |
raise NotImplementedError(self._read_bytes) |
2432.2.2
by Andrew Bennetts
Smart server mediums now detect which protocol version a request is and dispatch accordingly. |
346 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
347 |
|
348 |
class SmartServerSocketStreamMedium(SmartServerStreamMedium): |
|
349 |
||
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
350 |
def __init__(self, sock, backing_transport, root_client_path='/', |
351 |
timeout=None): |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
352 |
"""Constructor.
|
353 |
||
354 |
:param sock: the socket the server will read from. It will be put
|
|
355 |
into blocking mode.
|
|
356 |
"""
|
|
2692.1.11
by Andrew Bennetts
Improve test coverage by making SmartTCPServer_for_testing by default create a server that does not serve the backing transport's root at its own root. This mirrors the way most HTTP smart servers are configured. |
357 |
SmartServerStreamMedium.__init__( |
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
358 |
self, backing_transport, root_client_path=root_client_path, |
359 |
timeout=timeout) |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
360 |
sock.setblocking(True) |
361 |
self.socket = sock |
|
6133.4.62
by John Arbash Meinel
Add a nicer repr for shutting down. |
362 |
# Get the getpeername now, as we might be closed later when we care.
|
363 |
try: |
|
364 |
self._client_info = sock.getpeername() |
|
365 |
except socket.error: |
|
366 |
self._client_info = '<unknown>' |
|
367 |
||
368 |
def __str__(self): |
|
369 |
return '%s(client=%s)' % (self.__class__.__name__, self._client_info) |
|
370 |
||
371 |
def __repr__(self): |
|
372 |
return '%s.%s(client=%s)' % (self.__module__, self.__class__.__name__, |
|
373 |
self._client_info) |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
374 |
|
375 |
def _serve_one_request_unguarded(self, protocol): |
|
376 |
while protocol.next_read_size(): |
|
3565.1.3
by Andrew Bennetts
Define a _MAX_READ_SIZE constant as suggested by John's review. |
377 |
# We can safely try to read large chunks. If there is less data
|
5011.3.11
by Andrew Bennetts
Consolidate changes, try to minimise unnecessary changes and tidy up those that kept. |
378 |
# than MAX_SOCKET_CHUNK ready, the socket will just return a
|
379 |
# short read immediately rather than block.
|
|
380 |
bytes = self.read_bytes(osutils.MAX_SOCKET_CHUNK) |
|
3236.3.4
by Andrew Bennetts
Rename 'push_back' attribute to '_push_back_buffer', add some docstrings, and remove a little bit of redundant code from SmartServerSocketStreamMedium._serve_one_request_unguarded. |
381 |
if bytes == '': |
382 |
self.finished = True |
|
383 |
return
|
|
384 |
protocol.accept_bytes(bytes) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
385 |
|
3245.4.21
by Andrew Bennetts
Remove 'excess_buffer' attribute and another crufty comment. |
386 |
self._push_back(protocol.unused_data) |
3195.3.18
by Andrew Bennetts
call_with_body_bytes now works with v3 (e.g. test_copy_content_remote_to_local passes). Lots of debugging cruft, though. |
387 |
|
6133.5.2
by John Arbash Meinel
Rename _close to _disconnect_client. This makes it more obvious. |
388 |
def _disconnect_client(self): |
6133.4.11
by John Arbash Meinel
It turns out that if we don't explicitly close the socket, it hangs around somewhere. |
389 |
"""Close the current connection. We stopped due to a timeout/etc."""
|
390 |
self.socket.close() |
|
391 |
||
6133.4.1
by John Arbash Meinel
Add a _wait_for_bytes_with_timeout helper function. |
392 |
def _wait_for_bytes_with_timeout(self, timeout_seconds): |
393 |
"""Wait for more bytes to be read, but timeout if none available.
|
|
394 |
||
395 |
This allows us to detect idle connections, and stop trying to read from
|
|
396 |
them, without setting the socket itself to non-blocking. This also
|
|
397 |
allows us to specify when we watch for idle timeouts.
|
|
6133.4.2
by John Arbash Meinel
Implement waiting for Pipe mediums, including handling ones |
398 |
|
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
399 |
:return: None, this will raise ConnectionTimeout if we time out before
|
400 |
data is available.
|
|
6133.4.1
by John Arbash Meinel
Add a _wait_for_bytes_with_timeout helper function. |
401 |
"""
|
6133.4.8
by John Arbash Meinel
Refactor a bit. Use a common _wait_for_descriptor code. |
402 |
return self._wait_on_descriptor(self.socket, timeout_seconds) |
6133.4.1
by John Arbash Meinel
Add a _wait_for_bytes_with_timeout helper function. |
403 |
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
404 |
def _read_bytes(self, desired_count): |
5011.3.11
by Andrew Bennetts
Consolidate changes, try to minimise unnecessary changes and tidy up those that kept. |
405 |
return osutils.read_bytes_from_socket( |
406 |
self.socket, self._report_activity) |
|
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
407 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
408 |
def terminate_due_to_error(self): |
3245.4.59
by Andrew Bennetts
Various tweaks in response to Martin's review. |
409 |
# TODO: This should log to a server log file, but no such thing
|
410 |
# exists yet. Andrew Bennetts 2006-09-29.
|
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
411 |
self.socket.close() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
412 |
self.finished = True |
413 |
||
414 |
def _write_out(self, bytes): |
|
4889.2.1
by John Arbash Meinel
Make -Dhpss log debug information for the server process. |
415 |
tstart = osutils.timer_func() |
5011.3.9
by Andrew Bennetts
Remove _send_bytes_chunked. |
416 |
osutils.send_all(self.socket, bytes, self._report_activity) |
4889.2.1
by John Arbash Meinel
Make -Dhpss log debug information for the server process. |
417 |
if 'hpss' in debug.debug_flags: |
4913.1.1
by John Arbash Meinel
Switch to using thread.get_ident() which is available on all python versions. |
418 |
thread_id = thread.get_ident() |
4889.2.3
by John Arbash Meinel
Get rid of -Dhpssthread, just always include it. |
419 |
trace.mutter('%12s: [%s] %d bytes to the socket in %.3fs' |
420 |
% ('wrote', thread_id, len(bytes), |
|
4889.2.2
by John Arbash Meinel
Add a -Dhpssthread debug flag to include thread.ident info. |
421 |
osutils.timer_func() - tstart)) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
422 |
|
423 |
||
424 |
class SmartServerPipeStreamMedium(SmartServerStreamMedium): |
|
425 |
||
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
426 |
def __init__(self, in_file, out_file, backing_transport, timeout=None): |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
427 |
"""Construct new server.
|
428 |
||
429 |
:param in_file: Python file from which requests can be read.
|
|
430 |
:param out_file: Python file to write responses.
|
|
431 |
:param backing_transport: Transport for the directory served.
|
|
432 |
"""
|
|
6133.4.15
by John Arbash Meinel
Start working on exposing timeout as a configuration item. |
433 |
SmartServerStreamMedium.__init__(self, backing_transport, |
434 |
timeout=timeout) |
|
2018.5.161
by Andrew Bennetts
Reinstate forcing binary mode on windows in SmartServerStreamMedium. |
435 |
if sys.platform == 'win32': |
436 |
# force binary mode for files
|
|
437 |
import msvcrt |
|
438 |
for f in (in_file, out_file): |
|
439 |
fileno = getattr(f, 'fileno', None) |
|
440 |
if fileno: |
|
441 |
msvcrt.setmode(fileno(), os.O_BINARY) |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
442 |
self._in = in_file |
443 |
self._out = out_file |
|
444 |
||
6133.4.61
by John Arbash Meinel
Expose infrastructure so that we can test the Inet portion of the server. |
445 |
def serve(self): |
446 |
"""See SmartServerStreamMedium.serve"""
|
|
447 |
# This is the regular serve, except it adds signal trapping for soft
|
|
448 |
# shutdown.
|
|
449 |
stop_gracefully = self._stop_gracefully |
|
450 |
signals.register_on_hangup(id(self), stop_gracefully) |
|
451 |
try: |
|
452 |
return super(SmartServerPipeStreamMedium, self).serve() |
|
453 |
finally: |
|
454 |
signals.unregister_on_hangup(id(self)) |
|
455 |
||
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
456 |
def _serve_one_request_unguarded(self, protocol): |
457 |
while True: |
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
458 |
# We need to be careful not to read past the end of the current
|
459 |
# request, or else the read from the pipe will block, so we use
|
|
460 |
# protocol.next_read_size().
|
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
461 |
bytes_to_read = protocol.next_read_size() |
462 |
if bytes_to_read == 0: |
|
463 |
# Finished serving this request.
|
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
464 |
self._out.flush() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
465 |
return
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
466 |
bytes = self.read_bytes(bytes_to_read) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
467 |
if bytes == '': |
468 |
# Connection has been closed.
|
|
469 |
self.finished = True |
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
470 |
self._out.flush() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
471 |
return
|
472 |
protocol.accept_bytes(bytes) |
|
473 |
||
6133.5.2
by John Arbash Meinel
Rename _close to _disconnect_client. This makes it more obvious. |
474 |
def _disconnect_client(self): |
6133.4.11
by John Arbash Meinel
It turns out that if we don't explicitly close the socket, it hangs around somewhere. |
475 |
self._in.close() |
6133.4.63
by John Arbash Meinel
Flush before we close _out. |
476 |
self._out.flush() |
6133.4.11
by John Arbash Meinel
It turns out that if we don't explicitly close the socket, it hangs around somewhere. |
477 |
self._out.close() |
478 |
||
6133.4.2
by John Arbash Meinel
Implement waiting for Pipe mediums, including handling ones |
479 |
def _wait_for_bytes_with_timeout(self, timeout_seconds): |
480 |
"""Wait for more bytes to be read, but timeout if none available.
|
|
481 |
||
482 |
This allows us to detect idle connections, and stop trying to read from
|
|
483 |
them, without setting the socket itself to non-blocking. This also
|
|
484 |
allows us to specify when we watch for idle timeouts.
|
|
485 |
||
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
486 |
:return: None, this will raise ConnectionTimeout if we time out before
|
487 |
data is available.
|
|
6133.4.2
by John Arbash Meinel
Implement waiting for Pipe mediums, including handling ones |
488 |
"""
|
6133.4.10
by John Arbash Meinel
Handle the win32 not-a-socket errors. I think this is ~ready. At least for feedback. |
489 |
if (getattr(self._in, 'fileno', None) is None |
490 |
or sys.platform == 'win32'): |
|
491 |
# You can't select() file descriptors on Windows.
|
|
6133.4.42
by John Arbash Meinel
Change the code to just raise ConnectionTimeout. |
492 |
return
|
6133.4.8
by John Arbash Meinel
Refactor a bit. Use a common _wait_for_descriptor code. |
493 |
return self._wait_on_descriptor(self._in, timeout_seconds) |
6133.4.2
by John Arbash Meinel
Implement waiting for Pipe mediums, including handling ones |
494 |
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
495 |
def _read_bytes(self, desired_count): |
5011.3.1
by Martin
Revert second-phase EINTR changes |
496 |
return self._in.read(desired_count) |
2432.2.2
by Andrew Bennetts
Smart server mediums now detect which protocol version a request is and dispatch accordingly. |
497 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
498 |
def terminate_due_to_error(self): |
499 |
# TODO: This should log to a server log file, but no such thing
|
|
500 |
# exists yet. Andrew Bennetts 2006-09-29.
|
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
501 |
self._out.close() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
502 |
self.finished = True |
503 |
||
504 |
def _write_out(self, bytes): |
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
505 |
self._out.write(bytes) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
506 |
|
507 |
||
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
508 |
class SmartClientMediumRequest(object): |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
509 |
"""A request on a SmartClientMedium.
|
510 |
||
511 |
Each request allows bytes to be provided to it via accept_bytes, and then
|
|
512 |
the response bytes to be read via read_bytes.
|
|
513 |
||
514 |
For instance:
|
|
515 |
request.accept_bytes('123')
|
|
516 |
request.finished_writing()
|
|
517 |
result = request.read_bytes(3)
|
|
518 |
request.finished_reading()
|
|
519 |
||
520 |
It is up to the individual SmartClientMedium whether multiple concurrent
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
521 |
requests can exist. See SmartClientMedium.get_request to obtain instances
|
522 |
of SmartClientMediumRequest, and the concrete Medium you are using for
|
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
523 |
details on concurrency and pipelining.
|
524 |
"""
|
|
525 |
||
526 |
def __init__(self, medium): |
|
527 |
"""Construct a SmartClientMediumRequest for the medium medium."""
|
|
528 |
self._medium = medium |
|
529 |
# we track state by constants - we may want to use the same
|
|
530 |
# pattern as BodyReader if it gets more complex.
|
|
531 |
# valid states are: "writing", "reading", "done"
|
|
532 |
self._state = "writing" |
|
533 |
||
534 |
def accept_bytes(self, bytes): |
|
535 |
"""Accept bytes for inclusion in this request.
|
|
536 |
||
4031.3.1
by Frank Aspell
Fixing various typos |
537 |
This method may not be called after finished_writing() has been
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
538 |
called. It depends upon the Medium whether or not the bytes will be
|
539 |
immediately transmitted. Message based Mediums will tend to buffer the
|
|
540 |
bytes until finished_writing() is called.
|
|
541 |
||
542 |
:param bytes: A bytestring.
|
|
543 |
"""
|
|
544 |
if self._state != "writing": |
|
545 |
raise errors.WritingCompleted(self) |
|
546 |
self._accept_bytes(bytes) |
|
547 |
||
548 |
def _accept_bytes(self, bytes): |
|
549 |
"""Helper for accept_bytes.
|
|
550 |
||
551 |
Accept_bytes checks the state of the request to determing if bytes
|
|
552 |
should be accepted. After that it hands off to _accept_bytes to do the
|
|
553 |
actual acceptance.
|
|
554 |
"""
|
|
555 |
raise NotImplementedError(self._accept_bytes) |
|
556 |
||
557 |
def finished_reading(self): |
|
558 |
"""Inform the request that all desired data has been read.
|
|
559 |
||
560 |
This will remove the request from the pipeline for its medium (if the
|
|
561 |
medium supports pipelining) and any further calls to methods on the
|
|
562 |
request will raise ReadingCompleted.
|
|
563 |
"""
|
|
564 |
if self._state == "writing": |
|
565 |
raise errors.WritingNotComplete(self) |
|
566 |
if self._state != "reading": |
|
567 |
raise errors.ReadingCompleted(self) |
|
568 |
self._state = "done" |
|
569 |
self._finished_reading() |
|
570 |
||
571 |
def _finished_reading(self): |
|
572 |
"""Helper for finished_reading.
|
|
573 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
574 |
finished_reading checks the state of the request to determine if
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
575 |
finished_reading is allowed, and if it is hands off to _finished_reading
|
576 |
to perform the action.
|
|
577 |
"""
|
|
578 |
raise NotImplementedError(self._finished_reading) |
|
579 |
||
580 |
def finished_writing(self): |
|
581 |
"""Finish the writing phase of this request.
|
|
582 |
||
583 |
This will flush all pending data for this request along the medium.
|
|
584 |
After calling finished_writing, you may not call accept_bytes anymore.
|
|
585 |
"""
|
|
586 |
if self._state != "writing": |
|
587 |
raise errors.WritingCompleted(self) |
|
588 |
self._state = "reading" |
|
589 |
self._finished_writing() |
|
590 |
||
591 |
def _finished_writing(self): |
|
592 |
"""Helper for finished_writing.
|
|
593 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
594 |
finished_writing checks the state of the request to determine if
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
595 |
finished_writing is allowed, and if it is hands off to _finished_writing
|
596 |
to perform the action.
|
|
597 |
"""
|
|
598 |
raise NotImplementedError(self._finished_writing) |
|
599 |
||
600 |
def read_bytes(self, count): |
|
601 |
"""Read bytes from this requests response.
|
|
602 |
||
603 |
This method will block and wait for count bytes to be read. It may not
|
|
604 |
be invoked until finished_writing() has been called - this is to ensure
|
|
2432.2.7
by Andrew Bennetts
Use less confusing version strings, and define REQUEST_VERSION_TWO/RESPONSE_VERSION_TWO constants for them. |
605 |
a message-based approach to requests, for compatibility with message
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
606 |
based mediums like HTTP.
|
607 |
"""
|
|
608 |
if self._state == "writing": |
|
609 |
raise errors.WritingNotComplete(self) |
|
610 |
if self._state != "reading": |
|
611 |
raise errors.ReadingCompleted(self) |
|
612 |
return self._read_bytes(count) |
|
613 |
||
614 |
def _read_bytes(self, count): |
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
615 |
"""Helper for SmartClientMediumRequest.read_bytes.
|
616 |
||
617 |
read_bytes checks the state of the request to determing if bytes
|
|
618 |
should be read. After that it hands off to _read_bytes to do the
|
|
619 |
actual read.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
620 |
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
621 |
By default this forwards to self._medium.read_bytes because we are
|
622 |
operating on the medium's stream.
|
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
623 |
"""
|
3565.1.2
by Andrew Bennetts
Delete some more code, fix some bugs, add more comments. |
624 |
return self._medium.read_bytes(count) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
625 |
|
2432.2.7
by Andrew Bennetts
Use less confusing version strings, and define REQUEST_VERSION_TWO/RESPONSE_VERSION_TWO constants for them. |
626 |
def read_line(self): |
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
627 |
line = self._read_line() |
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
628 |
if not line.endswith('\n'): |
629 |
# end of file encountered reading from server
|
|
630 |
raise errors.ConnectionReset( |
|
4509.2.3
by Martin Pool
Test tweaks for ConnectionReset message change |
631 |
"Unexpected end of message. Please check connectivity "
|
632 |
"and permissions, and report a bug if problems persist.") |
|
2432.2.7
by Andrew Bennetts
Use less confusing version strings, and define REQUEST_VERSION_TWO/RESPONSE_VERSION_TWO constants for them. |
633 |
return line |
634 |
||
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
635 |
def _read_line(self): |
636 |
"""Helper for SmartClientMediumRequest.read_line.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
637 |
|
3606.4.1
by Andrew Bennetts
Fix NotImplementedError when probing for smart protocol via HTTP. |
638 |
By default this forwards to self._medium._get_line because we are
|
639 |
operating on the medium's stream.
|
|
640 |
"""
|
|
641 |
return self._medium._get_line() |
|
642 |
||
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
643 |
|
6060.7.1
by Jelmer Vernooij
Add vfs refuser. |
644 |
class _VfsRefuser(object): |
645 |
"""An object that refuses all VFS requests.
|
|
646 |
||
647 |
"""
|
|
648 |
||
649 |
def __init__(self): |
|
650 |
client._SmartClient.hooks.install_named_hook( |
|
6060.7.2
by Jelmer Vernooij
Fix hook installation |
651 |
'call', self.check_vfs, 'vfs refuser') |
6060.7.1
by Jelmer Vernooij
Add vfs refuser. |
652 |
|
653 |
def check_vfs(self, params): |
|
654 |
try: |
|
655 |
request_method = request.request_handlers.get(params.method) |
|
656 |
except KeyError: |
|
657 |
# A method we don't know about doesn't count as a VFS method.
|
|
658 |
return
|
|
6060.7.4
by Jelmer Vernooij
Update NEWS |
659 |
if issubclass(request_method, vfs.VfsRequest): |
660 |
raise errors.HpssVfsRequestNotAllowed(params.method, params.args) |
|
6060.7.1
by Jelmer Vernooij
Add vfs refuser. |
661 |
|
662 |
||
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
663 |
class _DebugCounter(object): |
664 |
"""An object that counts the HPSS calls made to each client medium.
|
|
665 |
||
5222.2.9
by Robert Collins
Write up some doc about bzrlib.initialize. |
666 |
When a medium is garbage-collected, or failing that when
|
667 |
bzrlib.global_state exits, the total number of calls made on that medium
|
|
668 |
are reported via trace.note.
|
|
3731.2.1
by Andrew Bennetts
Show total HPSS calls (if any) on stderr when -Dhpss is active. |
669 |
"""
|
670 |
||
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
671 |
def __init__(self): |
672 |
self.counts = weakref.WeakKeyDictionary() |
|
673 |
client._SmartClient.hooks.install_named_hook( |
|
674 |
'call', self.increment_call_count, 'hpss call counter') |
|
5310.1.1
by Vincent Ladeuil
Fix typo but we may want to use addCleanup instead indeed. |
675 |
bzrlib.global_state.cleanups.add_cleanup(self.flush_all) |
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
676 |
|
677 |
def track(self, medium): |
|
678 |
"""Start tracking calls made to a medium.
|
|
679 |
||
680 |
This only keeps a weakref to the medium, so shouldn't affect the
|
|
681 |
medium's lifetime.
|
|
682 |
"""
|
|
683 |
medium_repr = repr(medium) |
|
684 |
# Add this medium to the WeakKeyDictionary
|
|
4326.2.3
by Jonathan Lange
Use as a dict. |
685 |
self.counts[medium] = dict(count=0, vfs_count=0, |
686 |
medium_repr=medium_repr) |
|
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
687 |
# Weakref callbacks are fired in reverse order of their association
|
688 |
# with the referenced object. So we add a weakref *after* adding to
|
|
689 |
# the WeakKeyDict so that we can report the value from it before the
|
|
690 |
# entry is removed by the WeakKeyDict's own callback.
|
|
691 |
ref = weakref.ref(medium, self.done) |
|
692 |
||
693 |
def increment_call_count(self, params): |
|
694 |
# Increment the count in the WeakKeyDictionary
|
|
695 |
value = self.counts[params.medium] |
|
4326.2.3
by Jonathan Lange
Use as a dict. |
696 |
value['count'] += 1 |
4476.3.15
by Andrew Bennetts
Partially working fallback for pre-1.17 servers. |
697 |
try: |
698 |
request_method = request.request_handlers.get(params.method) |
|
699 |
except KeyError: |
|
4547.3.1
by Andrew Bennetts
Fix minor bug in -Dhpss that would cause a KeyError when issuing a request for a method not registered in request_handlers. |
700 |
# A method we don't know about doesn't count as a VFS method.
|
4476.3.15
by Andrew Bennetts
Partially working fallback for pre-1.17 servers. |
701 |
return
|
4326.2.1
by Jonathan Lange
Show the number of VFS calls in -Dhpss output. |
702 |
if issubclass(request_method, vfs.VfsRequest): |
4326.2.3
by Jonathan Lange
Use as a dict. |
703 |
value['vfs_count'] += 1 |
3731.2.1
by Andrew Bennetts
Show total HPSS calls (if any) on stderr when -Dhpss is active. |
704 |
|
705 |
def done(self, ref): |
|
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
706 |
value = self.counts[ref] |
4326.2.3
by Jonathan Lange
Use as a dict. |
707 |
count, vfs_count, medium_repr = ( |
708 |
value['count'], value['vfs_count'], value['medium_repr']) |
|
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
709 |
# In case this callback is invoked for the same ref twice (by the
|
710 |
# weakref callback and by the atexit function), set the call count back
|
|
711 |
# to 0 so this item won't be reported twice.
|
|
4326.2.3
by Jonathan Lange
Use as a dict. |
712 |
value['count'] = 0 |
713 |
value['vfs_count'] = 0 |
|
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
714 |
if count != 0: |
6138.3.4
by Jonathan Riddell
add gettext() to uses of trace.note() |
715 |
trace.note(gettext('HPSS calls: {0} ({1} vfs) {2}').format( |
716 |
count, vfs_count, medium_repr)) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
717 |
|
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
718 |
def flush_all(self): |
719 |
for ref in list(self.counts.keys()): |
|
720 |
self.done(ref) |
|
721 |
||
722 |
_debug_counter = None |
|
6060.7.2
by Jelmer Vernooij
Fix hook installation |
723 |
_vfs_refuser = None |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
724 |
|
725 |
||
3565.1.1
by Andrew Bennetts
Read no more then 64k at a time in the smart protocol code. |
726 |
class SmartClientMedium(SmartMedium): |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
727 |
"""Smart client is a medium for sending smart protocol requests over."""
|
728 |
||
3431.3.3
by Andrew Bennetts
Set 'base' in SmartClientMedium base class. |
729 |
def __init__(self, base): |
3241.1.1
by Andrew Bennetts
Shift protocol version querying from RemoteBzrDirFormat into SmartClientMedium. |
730 |
super(SmartClientMedium, self).__init__() |
3431.3.3
by Andrew Bennetts
Set 'base' in SmartClientMedium base class. |
731 |
self.base = base |
3241.1.4
by Andrew Bennetts
Use get_smart_medium as suggested by Robert, and deal with the fallout. |
732 |
self._protocol_version_error = None |
3241.1.1
by Andrew Bennetts
Shift protocol version querying from RemoteBzrDirFormat into SmartClientMedium. |
733 |
self._protocol_version = None |
3245.4.47
by Andrew Bennetts
Don't automatically send 'hello' requests from RemoteBzrDirFormat.probe_transport unless we have to (i.e. the transport is HTTP). |
734 |
self._done_hello = False |
3435.1.1
by Andrew Bennetts
Define _remote_is_at_least_1_2 on SmartClientMedium base class, rather than just SmartClientStreamMedium. |
735 |
# Be optimistic: we assume the remote end can accept new remote
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
736 |
# requests until we get an error saying otherwise.
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
737 |
# _remote_version_is_before tracks the bzr version the remote side
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
738 |
# can be based on what we've seen so far.
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
739 |
self._remote_version_is_before = None |
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
740 |
# Install debug hook function if debug flag is set.
|
3731.2.1
by Andrew Bennetts
Show total HPSS calls (if any) on stderr when -Dhpss is active. |
741 |
if 'hpss' in debug.debug_flags: |
3731.2.5
by Andrew Bennetts
Rework hpss call counter. |
742 |
global _debug_counter |
743 |
if _debug_counter is None: |
|
744 |
_debug_counter = _DebugCounter() |
|
745 |
_debug_counter.track(self) |
|
6060.7.5
by Jelmer Vernooij
Rename hpss_no_vfs to hpss_client_no_vfs. |
746 |
if 'hpss_client_no_vfs' in debug.debug_flags: |
6060.7.1
by Jelmer Vernooij
Add vfs refuser. |
747 |
global _vfs_refuser |
748 |
if _vfs_refuser is None: |
|
749 |
_vfs_refuser = _VfsRefuser() |
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
750 |
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
751 |
def _is_remote_before(self, version_tuple): |
3502.1.1
by Matt Nordhoff
Fix a docstring typo, and a two-expression ``raise`` statement |
752 |
"""Is it possible the remote side supports RPCs for a given version?
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
753 |
|
754 |
Typical use::
|
|
755 |
||
756 |
needed_version = (1, 2)
|
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
757 |
if medium._is_remote_before(needed_version):
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
758 |
fallback_to_pre_1_2_rpc()
|
759 |
else:
|
|
760 |
try:
|
|
761 |
do_1_2_rpc()
|
|
762 |
except UnknownSmartMethod:
|
|
3453.4.9
by Andrew Bennetts
Rename _remote_is_not to _remember_remote_is_before. |
763 |
medium._remember_remote_is_before(needed_version)
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
764 |
fallback_to_pre_1_2_rpc()
|
765 |
||
3453.4.9
by Andrew Bennetts
Rename _remote_is_not to _remember_remote_is_before. |
766 |
:seealso: _remember_remote_is_before
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
767 |
"""
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
768 |
if self._remote_version_is_before is None: |
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
769 |
# So far, the remote side seems to support everything
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
770 |
return False |
771 |
return version_tuple >= self._remote_version_is_before |
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
772 |
|
3453.4.9
by Andrew Bennetts
Rename _remote_is_not to _remember_remote_is_before. |
773 |
def _remember_remote_is_before(self, version_tuple): |
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
774 |
"""Tell this medium that the remote side is older the given version.
|
775 |
||
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
776 |
:seealso: _is_remote_before
|
3453.4.1
by Andrew Bennetts
Better infrastructure on SmartClientMedium for tracking the remote version. |
777 |
"""
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
778 |
if (self._remote_version_is_before is not None and |
779 |
version_tuple > self._remote_version_is_before): |
|
4017.3.3
by Robert Collins
Review feedback - make RemoteRepository.initialize use helpers, and version-lock the new method to not attempt the method on older servers. |
780 |
# We have been told that the remote side is older than some version
|
781 |
# which is newer than a previously supplied older-than version.
|
|
782 |
# This indicates that some smart verb call is not guarded
|
|
783 |
# appropriately (it should simply not have been tried).
|
|
4797.49.1
by Andrew Bennetts
First, fix _remember_remote_is_before to never raise AssertionError for what is a very minor bug. |
784 |
trace.mutter( |
3453.4.9
by Andrew Bennetts
Rename _remote_is_not to _remember_remote_is_before. |
785 |
"_remember_remote_is_before(%r) called, but " |
786 |
"_remember_remote_is_before(%r) was called previously." |
|
4797.49.1
by Andrew Bennetts
First, fix _remember_remote_is_before to never raise AssertionError for what is a very minor bug. |
787 |
, version_tuple, self._remote_version_is_before) |
788 |
if 'hpss' in debug.debug_flags: |
|
789 |
ui.ui_factory.show_warning( |
|
790 |
"_remember_remote_is_before(%r) called, but " |
|
791 |
"_remember_remote_is_before(%r) was called previously." |
|
792 |
% (version_tuple, self._remote_version_is_before)) |
|
793 |
return
|
|
3453.4.10
by Andrew Bennetts
Change _is_remote_at_least to _is_remote_before. |
794 |
self._remote_version_is_before = version_tuple |
3241.1.1
by Andrew Bennetts
Shift protocol version querying from RemoteBzrDirFormat into SmartClientMedium. |
795 |
|
796 |
def protocol_version(self): |
|
3245.4.47
by Andrew Bennetts
Don't automatically send 'hello' requests from RemoteBzrDirFormat.probe_transport unless we have to (i.e. the transport is HTTP). |
797 |
"""Find out if 'hello' smart request works."""
|
3241.1.4
by Andrew Bennetts
Use get_smart_medium as suggested by Robert, and deal with the fallout. |
798 |
if self._protocol_version_error is not None: |
799 |
raise self._protocol_version_error |
|
3245.4.47
by Andrew Bennetts
Don't automatically send 'hello' requests from RemoteBzrDirFormat.probe_transport unless we have to (i.e. the transport is HTTP). |
800 |
if not self._done_hello: |
3241.1.4
by Andrew Bennetts
Use get_smart_medium as suggested by Robert, and deal with the fallout. |
801 |
try: |
802 |
medium_request = self.get_request() |
|
803 |
# Send a 'hello' request in protocol version one, for maximum
|
|
804 |
# backwards compatibility.
|
|
3530.1.2
by John Arbash Meinel
missed one of the imports |
805 |
client_protocol = protocol.SmartClientRequestProtocolOne(medium_request) |
3245.4.47
by Andrew Bennetts
Don't automatically send 'hello' requests from RemoteBzrDirFormat.probe_transport unless we have to (i.e. the transport is HTTP). |
806 |
client_protocol.query_version() |
807 |
self._done_hello = True |
|
3241.1.4
by Andrew Bennetts
Use get_smart_medium as suggested by Robert, and deal with the fallout. |
808 |
except errors.SmartProtocolError, e: |
809 |
# Cache the error, just like we would cache a successful
|
|
810 |
# result.
|
|
811 |
self._protocol_version_error = e |
|
812 |
raise
|
|
3245.4.47
by Andrew Bennetts
Don't automatically send 'hello' requests from RemoteBzrDirFormat.probe_transport unless we have to (i.e. the transport is HTTP). |
813 |
return '2' |
814 |
||
815 |
def should_probe(self): |
|
816 |
"""Should RemoteBzrDirFormat.probe_transport send a smart request on
|
|
817 |
this medium?
|
|
818 |
||
819 |
Some transports are unambiguously smart-only; there's no need to check
|
|
820 |
if the transport is able to carry smart requests, because that's all
|
|
821 |
it is for. In those cases, this method should return False.
|
|
822 |
||
823 |
But some HTTP transports can sometimes fail to carry smart requests,
|
|
824 |
but still be usuable for accessing remote bzrdirs via plain file
|
|
825 |
accesses. So for those transports, their media should return True here
|
|
826 |
so that RemoteBzrDirFormat can determine if it is appropriate for that
|
|
827 |
transport.
|
|
828 |
"""
|
|
829 |
return False |
|
3241.1.1
by Andrew Bennetts
Shift protocol version querying from RemoteBzrDirFormat into SmartClientMedium. |
830 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
831 |
def disconnect(self): |
832 |
"""If this medium maintains a persistent connection, close it.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
833 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
834 |
The default implementation does nothing.
|
835 |
"""
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
836 |
|
3431.3.11
by Andrew Bennetts
Push remote_path_from_transport logic into SmartClientMedium, removing special-casing of bzr+http from _SmartClient. |
837 |
def remote_path_from_transport(self, transport): |
838 |
"""Convert transport into a path suitable for using in a request.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
839 |
|
3431.3.11
by Andrew Bennetts
Push remote_path_from_transport logic into SmartClientMedium, removing special-casing of bzr+http from _SmartClient. |
840 |
Note that the resulting remote path doesn't encode the host name or
|
841 |
anything but path, so it is only safe to use it in requests sent over
|
|
842 |
the medium from the matching transport.
|
|
843 |
"""
|
|
844 |
medium_base = urlutils.join(self.base, '/') |
|
845 |
rel_url = urlutils.relative_url(medium_base, transport.base) |
|
846 |
return urllib.unquote(rel_url) |
|
847 |
||
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
848 |
|
849 |
class SmartClientStreamMedium(SmartClientMedium): |
|
850 |
"""Stream based medium common class.
|
|
851 |
||
852 |
SmartClientStreamMediums operate on a stream. All subclasses use a common
|
|
853 |
SmartClientStreamMediumRequest for their requests, and should implement
|
|
854 |
_accept_bytes and _read_bytes to allow the request objects to send and
|
|
855 |
receive bytes.
|
|
856 |
"""
|
|
857 |
||
3431.3.3
by Andrew Bennetts
Set 'base' in SmartClientMedium base class. |
858 |
def __init__(self, base): |
859 |
SmartClientMedium.__init__(self, base) |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
860 |
self._current_request = None |
861 |
||
862 |
def accept_bytes(self, bytes): |
|
863 |
self._accept_bytes(bytes) |
|
864 |
||
865 |
def __del__(self): |
|
866 |
"""The SmartClientStreamMedium knows how to close the stream when it is
|
|
867 |
finished with it.
|
|
868 |
"""
|
|
869 |
self.disconnect() |
|
870 |
||
871 |
def _flush(self): |
|
872 |
"""Flush the output stream.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
873 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
874 |
This method is used by the SmartClientStreamMediumRequest to ensure that
|
875 |
all data for a request is sent, to avoid long timeouts or deadlocks.
|
|
876 |
"""
|
|
877 |
raise NotImplementedError(self._flush) |
|
878 |
||
879 |
def get_request(self): |
|
880 |
"""See SmartClientMedium.get_request().
|
|
881 |
||
882 |
SmartClientStreamMedium always returns a SmartClientStreamMediumRequest
|
|
883 |
for get_request.
|
|
884 |
"""
|
|
885 |
return SmartClientStreamMediumRequest(self) |
|
886 |
||
887 |
||
888 |
class SmartSimplePipesClientMedium(SmartClientStreamMedium): |
|
889 |
"""A client medium using simple pipes.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
890 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
891 |
This client does not manage the pipes: it assumes they will always be open.
|
892 |
"""
|
|
893 |
||
3431.3.1
by Andrew Bennetts
First rough cut of a fix for bug #230550, by adding .base to SmartClientMedia rather than relying on other objects to track this accurately while reusing client media. |
894 |
def __init__(self, readable_pipe, writeable_pipe, base): |
3431.3.3
by Andrew Bennetts
Set 'base' in SmartClientMedium base class. |
895 |
SmartClientStreamMedium.__init__(self, base) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
896 |
self._readable_pipe = readable_pipe |
897 |
self._writeable_pipe = writeable_pipe |
|
898 |
||
899 |
def _accept_bytes(self, bytes): |
|
900 |
"""See SmartClientStreamMedium.accept_bytes."""
|
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
901 |
self._writeable_pipe.write(bytes) |
3958.1.2
by Andrew Bennetts
Report network activity from more client medium implementations. |
902 |
self._report_activity(len(bytes), 'write') |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
903 |
|
904 |
def _flush(self): |
|
905 |
"""See SmartClientStreamMedium._flush()."""
|
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
906 |
self._writeable_pipe.flush() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
907 |
|
908 |
def _read_bytes(self, count): |
|
909 |
"""See SmartClientStreamMedium._read_bytes."""
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
910 |
bytes_to_read = min(count, _MAX_READ_SIZE) |
911 |
bytes = self._readable_pipe.read(bytes_to_read) |
|
3958.1.2
by Andrew Bennetts
Report network activity from more client medium implementations. |
912 |
self._report_activity(len(bytes), 'read') |
913 |
return bytes |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
914 |
|
915 |
||
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
916 |
class SSHParams(object): |
5284.5.5
by Andrew Bennetts
Fix SSHParams docs. |
917 |
"""A set of parameters for starting a remote bzr via SSH."""
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
918 |
|
919 |
def __init__(self, host, port=None, username=None, password=None, |
|
920 |
bzr_remote_path='bzr'): |
|
921 |
self.host = host |
|
922 |
self.port = port |
|
923 |
self.username = username |
|
924 |
self.password = password |
|
925 |
self.bzr_remote_path = bzr_remote_path |
|
926 |
||
927 |
||
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
928 |
class SmartSSHClientMedium(SmartClientStreamMedium): |
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
929 |
"""A client medium using SSH.
|
930 |
|
|
931 |
It delegates IO to a SmartClientSocketMedium or
|
|
932 |
SmartClientAlreadyConnectedSocketMedium (depending on platform).
|
|
933 |
"""
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
934 |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
935 |
def __init__(self, base, ssh_params, vendor=None): |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
936 |
"""Creates a client that will connect on the first use.
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
937 |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
938 |
:param ssh_params: A SSHParams instance.
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
939 |
:param vendor: An optional override for the ssh vendor to use. See
|
940 |
bzrlib.transport.ssh for details on ssh vendors.
|
|
941 |
"""
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
942 |
self._real_medium = None |
943 |
self._ssh_params = ssh_params |
|
4964.2.5
by Martin Pool
Make sure variables used in repr are set early in initialization |
944 |
# for the benefit of progress making a short description of this
|
945 |
# transport
|
|
946 |
self._scheme = 'bzr+ssh' |
|
4100.1.5
by Martin Pool
Fix crash in SSHSmartClientStreamMedium repr. |
947 |
# SmartClientStreamMedium stores the repr of this object in its
|
948 |
# _DebugCounter so we have to store all the values used in our repr
|
|
949 |
# method before calling the super init.
|
|
950 |
SmartClientStreamMedium.__init__(self, base) |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
951 |
self._vendor = vendor |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
952 |
self._ssh_connection = None |
4100.1.1
by Martin Pool
Cleanup and add SmartSSHClientMedium repr |
953 |
|
954 |
def __repr__(self): |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
955 |
if self._ssh_params.port is None: |
4964.2.4
by Martin Pool
Tweak SSHSmartClientMedium to look better when there's no port |
956 |
maybe_port = '' |
957 |
else: |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
958 |
maybe_port = ':%s' % self._ssh_params.port |
4964.2.4
by Martin Pool
Tweak SSHSmartClientMedium to look better when there's no port |
959 |
return "%s(%s://%s@%s%s/)" % ( |
4100.1.1
by Martin Pool
Cleanup and add SmartSSHClientMedium repr |
960 |
self.__class__.__name__, |
4964.2.3
by Martin Pool
Tweak SmartSSHClientMedium repr |
961 |
self._scheme, |
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
962 |
self._ssh_params.username, |
963 |
self._ssh_params.host, |
|
4964.2.4
by Martin Pool
Tweak SSHSmartClientMedium to look better when there's no port |
964 |
maybe_port) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
965 |
|
966 |
def _accept_bytes(self, bytes): |
|
967 |
"""See SmartClientStreamMedium.accept_bytes."""
|
|
968 |
self._ensure_connection() |
|
5284.5.6
by Andrew Bennetts
Tweaks prompted by Robert's review. |
969 |
self._real_medium.accept_bytes(bytes) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
970 |
|
971 |
def disconnect(self): |
|
972 |
"""See SmartClientMedium.disconnect()."""
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
973 |
if self._real_medium is not None: |
974 |
self._real_medium.disconnect() |
|
975 |
self._real_medium = None |
|
976 |
if self._ssh_connection is not None: |
|
977 |
self._ssh_connection.close() |
|
978 |
self._ssh_connection = None |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
979 |
|
980 |
def _ensure_connection(self): |
|
981 |
"""Connect this medium if not already connected."""
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
982 |
if self._real_medium is not None: |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
983 |
return
|
984 |
if self._vendor is None: |
|
985 |
vendor = ssh._get_ssh_vendor() |
|
986 |
else: |
|
987 |
vendor = self._vendor |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
988 |
self._ssh_connection = vendor.connect_ssh(self._ssh_params.username, |
989 |
self._ssh_params.password, self._ssh_params.host, |
|
990 |
self._ssh_params.port, |
|
991 |
command=[self._ssh_params.bzr_remote_path, 'serve', '--inet', |
|
1551.18.17
by Aaron Bentley
Introduce bzr_remote_path configuration variable |
992 |
'--directory=/', '--allow-writes']) |
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
993 |
io_kind, io_object = self._ssh_connection.get_sock_or_pipes() |
994 |
if io_kind == 'socket': |
|
995 |
self._real_medium = SmartClientAlreadyConnectedSocketMedium( |
|
996 |
self.base, io_object) |
|
997 |
elif io_kind == 'pipes': |
|
998 |
read_from, write_to = io_object |
|
999 |
self._real_medium = SmartSimplePipesClientMedium( |
|
1000 |
read_from, write_to, self.base) |
|
1001 |
else: |
|
1002 |
raise AssertionError( |
|
1003 |
"Unexpected io_kind %r from %r" |
|
1004 |
% (io_kind, self._ssh_connection)) |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1005 |
|
1006 |
def _flush(self): |
|
1007 |
"""See SmartClientStreamMedium._flush()."""
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1008 |
self._real_medium._flush() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1009 |
|
1010 |
def _read_bytes(self, count): |
|
1011 |
"""See SmartClientStreamMedium.read_bytes."""
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1012 |
if self._real_medium is None: |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1013 |
raise errors.MediumNotConnected(self) |
5284.5.6
by Andrew Bennetts
Tweaks prompted by Robert's review. |
1014 |
return self._real_medium.read_bytes(count) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1015 |
|
1016 |
||
3004.2.1
by Vincent Ladeuil
Fix 150860 by leaving port as user specified it. |
1017 |
# Port 4155 is the default port for bzr://, registered with IANA.
|
3665.4.1
by Jelmer Vernooij
Support IPv6 in the smart server. |
1018 |
BZR_DEFAULT_INTERFACE = None |
3004.2.1
by Vincent Ladeuil
Fix 150860 by leaving port as user specified it. |
1019 |
BZR_DEFAULT_PORT = 4155 |
1020 |
||
1021 |
||
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1022 |
class SmartClientSocketMedium(SmartClientStreamMedium): |
5284.5.6
by Andrew Bennetts
Tweaks prompted by Robert's review. |
1023 |
"""A client medium using a socket.
|
1024 |
|
|
1025 |
This class isn't usable directly. Use one of its subclasses instead.
|
|
1026 |
"""
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1027 |
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1028 |
def __init__(self, base): |
3431.3.3
by Andrew Bennetts
Set 'base' in SmartClientMedium base class. |
1029 |
SmartClientStreamMedium.__init__(self, base) |
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1030 |
self._socket = None |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1031 |
self._connected = False |
1032 |
||
1033 |
def _accept_bytes(self, bytes): |
|
1034 |
"""See SmartClientMedium.accept_bytes."""
|
|
1035 |
self._ensure_connection() |
|
5011.3.9
by Andrew Bennetts
Remove _send_bytes_chunked. |
1036 |
osutils.send_all(self._socket, bytes, self._report_activity) |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1037 |
|
5284.5.6
by Andrew Bennetts
Tweaks prompted by Robert's review. |
1038 |
def _ensure_connection(self): |
1039 |
"""Connect this medium if not already connected."""
|
|
1040 |
raise NotImplementedError(self._ensure_connection) |
|
1041 |
||
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1042 |
def _flush(self): |
1043 |
"""See SmartClientStreamMedium._flush().
|
|
1044 |
||
5284.5.6
by Andrew Bennetts
Tweaks prompted by Robert's review. |
1045 |
For sockets we do no flushing. For TCP sockets we may want to turn off
|
1046 |
TCP_NODELAY and add a means to do a flush, but that can be done in the
|
|
1047 |
future.
|
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1048 |
"""
|
1049 |
||
1050 |
def _read_bytes(self, count): |
|
1051 |
"""See SmartClientMedium.read_bytes."""
|
|
1052 |
if not self._connected: |
|
1053 |
raise errors.MediumNotConnected(self) |
|
1054 |
return osutils.read_bytes_from_socket( |
|
1055 |
self._socket, self._report_activity) |
|
1056 |
||
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1057 |
def disconnect(self): |
1058 |
"""See SmartClientMedium.disconnect()."""
|
|
1059 |
if not self._connected: |
|
1060 |
return
|
|
5011.3.1
by Martin
Revert second-phase EINTR changes |
1061 |
self._socket.close() |
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1062 |
self._socket = None |
1063 |
self._connected = False |
|
1064 |
||
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1065 |
|
1066 |
class SmartTCPClientMedium(SmartClientSocketMedium): |
|
5284.5.6
by Andrew Bennetts
Tweaks prompted by Robert's review. |
1067 |
"""A client medium that creates a TCP connection."""
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1068 |
|
1069 |
def __init__(self, host, port, base): |
|
1070 |
"""Creates a client that will connect on the first use."""
|
|
1071 |
SmartClientSocketMedium.__init__(self, base) |
|
1072 |
self._host = host |
|
1073 |
self._port = port |
|
1074 |
||
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1075 |
def _ensure_connection(self): |
1076 |
"""Connect this medium if not already connected."""
|
|
1077 |
if self._connected: |
|
1078 |
return
|
|
3004.2.1
by Vincent Ladeuil
Fix 150860 by leaving port as user specified it. |
1079 |
if self._port is None: |
1080 |
port = BZR_DEFAULT_PORT |
|
1081 |
else: |
|
1082 |
port = int(self._port) |
|
3711.2.2
by Jelmer Vernooij
Avoid using AI_ADDRCONFIG since it's not portable. |
1083 |
try: |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1084 |
sockaddrs = socket.getaddrinfo(self._host, port, socket.AF_UNSPEC, |
3711.2.2
by Jelmer Vernooij
Avoid using AI_ADDRCONFIG since it's not portable. |
1085 |
socket.SOCK_STREAM, 0, 0) |
1086 |
except socket.gaierror, (err_num, err_msg): |
|
1087 |
raise errors.ConnectionError("failed to lookup %s:%d: %s" % |
|
1088 |
(self._host, port, err_msg)) |
|
3711.2.3
by Jelmer Vernooij
Add comment. |
1089 |
# Initialize err in case there are no addresses returned:
|
3665.4.2
by Jelmer Vernooij
Fall through to next available address if previous fails. |
1090 |
err = socket.error("no address found for %s" % self._host) |
3665.4.1
by Jelmer Vernooij
Support IPv6 in the smart server. |
1091 |
for (family, socktype, proto, canonname, sockaddr) in sockaddrs: |
1092 |
try: |
|
3665.4.2
by Jelmer Vernooij
Fall through to next available address if previous fails. |
1093 |
self._socket = socket.socket(family, socktype, proto) |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1094 |
self._socket.setsockopt(socket.IPPROTO_TCP, |
3665.4.2
by Jelmer Vernooij
Fall through to next available address if previous fails. |
1095 |
socket.TCP_NODELAY, 1) |
3665.4.1
by Jelmer Vernooij
Support IPv6 in the smart server. |
1096 |
self._socket.connect(sockaddr) |
1097 |
except socket.error, err: |
|
3665.4.2
by Jelmer Vernooij
Fall through to next available address if previous fails. |
1098 |
if self._socket is not None: |
1099 |
self._socket.close() |
|
1100 |
self._socket = None |
|
1101 |
continue
|
|
1102 |
break
|
|
1103 |
if self._socket is None: |
|
1104 |
# socket errors either have a (string) or (errno, string) as their
|
|
1105 |
# args.
|
|
1106 |
if type(err.args) is str: |
|
1107 |
err_msg = err.args |
|
1108 |
else: |
|
1109 |
err_msg = err.args[1] |
|
1110 |
raise errors.ConnectionError("failed to connect to %s:%d: %s" % |
|
1111 |
(self._host, port, err_msg)) |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1112 |
self._connected = True |
1113 |
||
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1114 |
|
1115 |
class SmartClientAlreadyConnectedSocketMedium(SmartClientSocketMedium): |
|
5284.5.3
by Andrew Bennetts
Docstring tweaks. |
1116 |
"""A client medium for an already connected socket.
|
5284.5.1
by Andrew Bennetts
Use socketpairs (rather than pipes) for SSH subprocesses where possible, and formalise some internal APIs a little more. |
1117 |
|
1118 |
Note that this class will assume it "owns" the socket, so it will close it
|
|
1119 |
when its disconnect method is called.
|
|
1120 |
"""
|
|
1121 |
||
1122 |
def __init__(self, base, sock): |
|
1123 |
SmartClientSocketMedium.__init__(self, base) |
|
1124 |
self._socket = sock |
|
1125 |
self._connected = True |
|
1126 |
||
1127 |
def _ensure_connection(self): |
|
1128 |
# Already connected, by definition! So nothing to do.
|
|
1129 |
pass
|
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1130 |
|
1131 |
||
1132 |
class SmartClientStreamMediumRequest(SmartClientMediumRequest): |
|
1133 |
"""A SmartClientMediumRequest that works with an SmartClientStreamMedium."""
|
|
1134 |
||
1135 |
def __init__(self, medium): |
|
1136 |
SmartClientMediumRequest.__init__(self, medium) |
|
1137 |
# check that we are safe concurrency wise. If some streams start
|
|
1138 |
# allowing concurrent requests - i.e. via multiplexing - then this
|
|
1139 |
# assert should be moved to SmartClientStreamMedium.get_request,
|
|
1140 |
# and the setting/unsetting of _current_request likewise moved into
|
|
1141 |
# that class : but its unneeded overhead for now. RBC 20060922
|
|
1142 |
if self._medium._current_request is not None: |
|
1143 |
raise errors.TooManyConcurrentRequests(self._medium) |
|
1144 |
self._medium._current_request = self |
|
1145 |
||
1146 |
def _accept_bytes(self, bytes): |
|
1147 |
"""See SmartClientMediumRequest._accept_bytes.
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1148 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1149 |
This forwards to self._medium._accept_bytes because we are operating
|
1150 |
on the mediums stream.
|
|
1151 |
"""
|
|
1152 |
self._medium._accept_bytes(bytes) |
|
1153 |
||
1154 |
def _finished_reading(self): |
|
1155 |
"""See SmartClientMediumRequest._finished_reading.
|
|
1156 |
||
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1157 |
This clears the _current_request on self._medium to allow a new
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1158 |
request to be created.
|
1159 |
"""
|
|
3376.2.4
by Martin Pool
Remove every assert statement from bzrlib! |
1160 |
if self._medium._current_request is not self: |
1161 |
raise AssertionError() |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1162 |
self._medium._current_request = None |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1163 |
|
2018.5.2
by Andrew Bennetts
Start splitting bzrlib/transport/smart.py into a package. |
1164 |
def _finished_writing(self): |
1165 |
"""See SmartClientMediumRequest._finished_writing.
|
|
1166 |
||
1167 |
This invokes self._medium._flush to ensure all bytes are transmitted.
|
|
1168 |
"""
|
|
1169 |
self._medium._flush() |
|
1170 |
||
4382.4.2
by Andrew Bennetts
Refactor duplicated SmartServerSocketStreamMedium._read_bytes and SmartTCPClientMedium._read_bytes to share a common implementation with the best parts of both. Includes Robert's review feedback. |
1171 |