~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/HttpServer.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2007-03-28 06:58:22 UTC
  • mfrom: (2379.2.3 hpss-chroot)
  • Revision ID: pqm@pqm.ubuntu.com-20070328065822-999550a858a3ced3
(robertc) Fix chroot urls to not expose the url of the transport they are protecting, allowing regular url operations to work on them. (Robert Collins, Andrew Bennetts)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2006 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
 
17
import BaseHTTPServer
17
18
import errno
18
 
import httplib
19
19
import os
 
20
from SimpleHTTPServer import SimpleHTTPRequestHandler
 
21
import socket
20
22
import posixpath
21
23
import random
22
24
import re
23
 
import SimpleHTTPServer
24
 
import socket
 
25
import sys
 
26
import threading
 
27
import time
 
28
import urllib
25
29
import urlparse
26
30
 
27
 
from bzrlib import urlutils
28
 
from bzrlib.tests import test_server
 
31
from bzrlib.transport import Server
 
32
 
 
33
 
 
34
class WebserverNotAvailable(Exception):
 
35
    pass
29
36
 
30
37
 
31
38
class BadWebserverPath(ValueError):
33
40
        return 'path %s is not in %s' % self.args
34
41
 
35
42
 
36
 
class TestingHTTPRequestHandler(SimpleHTTPServer.SimpleHTTPRequestHandler):
37
 
    """Handles one request.
38
 
 
39
 
    A TestingHTTPRequestHandler is instantiated for every request received by
40
 
    the associated server. Note that 'request' here is inherited from the base
41
 
    TCPServer class, for the HTTP server it is really a connection which itself
42
 
    will handle one or several HTTP requests.
43
 
    """
44
 
    # Default protocol version
45
 
    protocol_version = 'HTTP/1.1'
46
 
 
47
 
    # The Message-like class used to parse the request headers
48
 
    MessageClass = httplib.HTTPMessage
49
 
 
50
 
    def setup(self):
51
 
        SimpleHTTPServer.SimpleHTTPRequestHandler.setup(self)
52
 
        self._cwd = self.server._home_dir
53
 
        tcs = self.server.test_case_server
54
 
        if tcs.protocol_version is not None:
55
 
            # If the test server forced a protocol version, use it
56
 
            self.protocol_version = tcs.protocol_version
 
43
class TestingHTTPRequestHandler(SimpleHTTPRequestHandler):
57
44
 
58
45
    def log_message(self, format, *args):
59
 
        tcs = self.server.test_case_server
60
 
        tcs.log('webserver - %s - - [%s] %s "%s" "%s"',
61
 
                self.address_string(),
62
 
                self.log_date_time_string(),
63
 
                format % args,
64
 
                self.headers.get('referer', '-'),
65
 
                self.headers.get('user-agent', '-'))
66
 
 
67
 
    def handle(self):
68
 
        SimpleHTTPServer.SimpleHTTPRequestHandler.handle(self)
69
 
        # Some client (pycurl, I'm looking at you) are more picky than others
70
 
        # and require that the socket itself is closed
71
 
        # (SocketServer.StreamRequestHandler only close the two associated
72
 
        # 'makefile' objects)
73
 
        self.connection.close()
 
46
        self.server.test_case.log('webserver - %s - - [%s] %s "%s" "%s"',
 
47
                                  self.address_string(),
 
48
                                  self.log_date_time_string(),
 
49
                                  format % args,
 
50
                                  self.headers.get('referer', '-'),
 
51
                                  self.headers.get('user-agent', '-'))
74
52
 
75
53
    def handle_one_request(self):
76
54
        """Handle a single HTTP request.
77
55
 
78
 
        We catch all socket errors occurring when the client close the
79
 
        connection early to avoid polluting the test results.
 
56
        You normally don't need to override this method; see the class
 
57
        __doc__ string for information on how to handle specific HTTP
 
58
        commands such as GET and POST.
 
59
 
80
60
        """
81
 
        try:
82
 
            self._handle_one_request()
83
 
        except socket.error, e:
84
 
            # Any socket error should close the connection, but some errors are
85
 
            # due to the client closing early and we don't want to pollute test
86
 
            # results, so we raise only the others.
 
61
        for i in xrange(1,11): # Don't try more than 10 times
 
62
            try:
 
63
                self.raw_requestline = self.rfile.readline()
 
64
            except socket.error, e:
 
65
                if e.args[0] in (errno.EAGAIN, errno.EWOULDBLOCK):
 
66
                    # omitted for now because some tests look at the log of
 
67
                    # the server and expect to see no errors.  see recent
 
68
                    # email thread. -- mbp 20051021. 
 
69
                    ## self.log_message('EAGAIN (%d) while reading from raw_requestline' % i)
 
70
                    time.sleep(0.01)
 
71
                    continue
 
72
                raise
 
73
            else:
 
74
                break
 
75
        if not self.raw_requestline:
87
76
            self.close_connection = 1
88
 
            if (len(e.args) == 0
89
 
                or e.args[0] not in (errno.EPIPE, errno.ECONNRESET,
90
 
                                     errno.ECONNABORTED, errno.EBADF)):
91
 
                raise
92
 
 
93
 
    error_content_type = 'text/plain'
94
 
    error_message_format = '''\
95
 
Error code: %(code)s.
96
 
Message: %(message)s.
97
 
'''
98
 
 
99
 
    def send_error(self, code, message=None):
100
 
        """Send and log an error reply.
101
 
 
102
 
        We redefine the python-provided version to be able to set a 
103
 
        ``Content-Length`` header as some http/1.1 clients complain otherwise
104
 
        (see bug #568421).
105
 
 
106
 
        :param code: The HTTP error code.
107
 
 
108
 
        :param message: The explanation of the error code, Defaults to a short
109
 
             entry.
110
 
        """
111
 
 
112
 
        if message is None:
113
 
            try:
114
 
                message = self.responses[code][0]
115
 
            except KeyError:
116
 
                message = '???'
117
 
        self.log_error("code %d, message %s", code, message)
118
 
        content = (self.error_message_format %
119
 
                   {'code': code, 'message': message})
120
 
        self.send_response(code, message)
121
 
        self.send_header("Content-Type", self.error_content_type)
122
 
        self.send_header("Content-Length", "%d" % len(content))
123
 
        self.send_header('Connection', 'close')
124
 
        self.end_headers()
125
 
        if self.command != 'HEAD' and code >= 200 and code not in (204, 304):
126
 
            self.wfile.write(content)
127
 
 
128
 
    def _handle_one_request(self):
129
 
        SimpleHTTPServer.SimpleHTTPRequestHandler.handle_one_request(self)
130
 
 
131
 
    _range_regexp = re.compile(r'^(?P<start>\d+)-(?P<end>\d+)?$')
 
77
            return
 
78
        if not self.parse_request(): # An error code has been sent, just exit
 
79
            return
 
80
        mname = 'do_' + self.command
 
81
        if getattr(self, mname, None) is None:
 
82
            self.send_error(501, "Unsupported method (%r)" % self.command)
 
83
            return
 
84
        method = getattr(self, mname)
 
85
        method()
 
86
 
 
87
    _range_regexp = re.compile(r'^(?P<start>\d+)-(?P<end>\d+)$')
132
88
    _tail_regexp = re.compile(r'^-(?P<tail>\d+)$')
133
89
 
134
 
    def _parse_ranges(self, ranges_header, file_size):
135
 
        """Parse the range header value and returns ranges.
136
 
 
137
 
        RFC2616 14.35 says that syntactically invalid range specifiers MUST be
138
 
        ignored. In that case, we return None instead of a range list.
139
 
 
140
 
        :param ranges_header: The 'Range' header value.
141
 
 
142
 
        :param file_size: The size of the requested file.
143
 
 
144
 
        :return: A list of (start, end) tuples or None if some invalid range
145
 
            specifier is encountered.
 
90
    def parse_ranges(self, ranges_header):
 
91
        """Parse the range header value and returns ranges and tail.
 
92
 
 
93
        RFC2616 14.35 says that syntactically invalid range
 
94
        specifiers MUST be ignored. In that case, we return 0 for
 
95
        tail and [] for ranges.
146
96
        """
 
97
        tail = 0
 
98
        ranges = []
147
99
        if not ranges_header.startswith('bytes='):
148
100
            # Syntactically invalid header
149
 
            return None
 
101
            return 0, []
150
102
 
151
 
        tail = None
152
 
        ranges = []
153
103
        ranges_header = ranges_header[len('bytes='):]
154
104
        for range_str in ranges_header.split(','):
 
105
            # FIXME: RFC2616 says end is optional and default to file_size
155
106
            range_match = self._range_regexp.match(range_str)
156
107
            if range_match is not None:
157
108
                start = int(range_match.group('start'))
158
 
                end_match = range_match.group('end')
159
 
                if end_match is None:
160
 
                    # RFC2616 says end is optional and default to file_size
161
 
                    end = file_size
162
 
                else:
163
 
                    end = int(end_match)
 
109
                end = int(range_match.group('end'))
164
110
                if start > end:
165
111
                    # Syntactically invalid range
166
 
                    return None
 
112
                    return 0, []
167
113
                ranges.append((start, end))
168
114
            else:
169
115
                tail_match = self._tail_regexp.match(range_str)
171
117
                    tail = int(tail_match.group('tail'))
172
118
                else:
173
119
                    # Syntactically invalid range
174
 
                    return None
175
 
        if tail is not None:
176
 
            # Normalize tail into ranges
177
 
            ranges.append((max(0, file_size - tail), file_size))
178
 
 
179
 
        checked_ranges = []
180
 
        for start, end in ranges:
181
 
            if start >= file_size:
182
 
                # RFC2616 14.35, ranges are invalid if start >= file_size
183
 
                return None
184
 
            # RFC2616 14.35, end values should be truncated
185
 
            # to file_size -1 if they exceed it
186
 
            end = min(end, file_size - 1)
187
 
            checked_ranges.append((start, end))
188
 
        return checked_ranges
189
 
 
190
 
    def _header_line_length(self, keyword, value):
191
 
        header_line = '%s: %s\r\n' % (keyword, value)
192
 
        return len(header_line)
193
 
 
194
 
    def send_head(self):
195
 
        """Overrides base implementation to work around a bug in python2.5."""
196
 
        path = self.translate_path(self.path)
197
 
        if os.path.isdir(path) and not self.path.endswith('/'):
198
 
            # redirect browser - doing basically what apache does when
199
 
            # DirectorySlash option is On which is quite common (braindead, but
200
 
            # common)
201
 
            self.send_response(301)
202
 
            self.send_header("Location", self.path + "/")
203
 
            # Indicates that the body is empty for HTTP/1.1 clients
204
 
            self.send_header('Content-Length', '0')
205
 
            self.end_headers()
206
 
            return None
207
 
 
208
 
        return SimpleHTTPServer.SimpleHTTPRequestHandler.send_head(self)
 
120
                    return 0, []
 
121
        return tail, ranges
209
122
 
210
123
    def send_range_content(self, file, start, length):
211
124
        file.seek(start)
227
140
    def get_multiple_ranges(self, file, file_size, ranges):
228
141
        self.send_response(206)
229
142
        self.send_header('Accept-Ranges', 'bytes')
230
 
        boundary = '%d' % random.randint(0,0x7FFFFFFF)
231
 
        self.send_header('Content-Type',
232
 
                         'multipart/byteranges; boundary=%s' % boundary)
233
 
        boundary_line = '--%s\r\n' % boundary
234
 
        # Calculate the Content-Length
235
 
        content_length = 0
236
 
        for (start, end) in ranges:
237
 
            content_length += len(boundary_line)
238
 
            content_length += self._header_line_length(
239
 
                'Content-type', 'application/octet-stream')
240
 
            content_length += self._header_line_length(
241
 
                'Content-Range', 'bytes %d-%d/%d' % (start, end, file_size))
242
 
            content_length += len('\r\n') # end headers
243
 
            content_length += end - start + 1
244
 
        content_length += len(boundary_line)
245
 
        self.send_header('Content-length', content_length)
 
143
        boundary = "%d" % random.randint(0,0x7FFFFFFF)
 
144
        self.send_header("Content-Type",
 
145
                         "multipart/byteranges; boundary=%s" % boundary)
246
146
        self.end_headers()
247
 
 
248
 
        # Send the multipart body
249
147
        for (start, end) in ranges:
250
 
            self.wfile.write(boundary_line)
251
 
            self.send_header('Content-type', 'application/octet-stream')
252
 
            self.send_header('Content-Range', 'bytes %d-%d/%d'
253
 
                             % (start, end, file_size))
 
148
            self.wfile.write("--%s\r\n" % boundary)
 
149
            self.send_header("Content-type", 'application/octet-stream')
 
150
            self.send_header("Content-Range", "bytes %d-%d/%d" % (start,
 
151
                                                                  end,
 
152
                                                                  file_size))
254
153
            self.end_headers()
255
154
            self.send_range_content(file, start, end - start + 1)
256
 
        # Final boundary
257
 
        self.wfile.write(boundary_line)
 
155
            self.wfile.write("--%s\r\n" % boundary)
 
156
            pass
258
157
 
259
158
    def do_GET(self):
260
159
        """Serve a GET request.
261
160
 
262
161
        Handles the Range header.
263
162
        """
264
 
        # Update statistics
265
 
        self.server.test_case_server.GET_request_nb += 1
266
163
 
267
164
        path = self.translate_path(self.path)
268
165
        ranges_header_value = self.headers.get('Range')
269
166
        if ranges_header_value is None or os.path.isdir(path):
270
167
            # Let the mother class handle most cases
271
 
            return SimpleHTTPServer.SimpleHTTPRequestHandler.do_GET(self)
 
168
            return SimpleHTTPRequestHandler.do_GET(self)
272
169
 
273
170
        try:
274
171
            # Always read in binary mode. Opening files in text
275
172
            # mode may cause newline translations, making the
276
173
            # actual size of the content transmitted *less* than
277
174
            # the content-length!
278
 
            f = open(path, 'rb')
 
175
            file = open(path, 'rb')
279
176
        except IOError:
280
177
            self.send_error(404, "File not found")
281
178
            return
282
179
 
283
 
        file_size = os.fstat(f.fileno())[6]
284
 
        ranges = self._parse_ranges(ranges_header_value, file_size)
285
 
        if not ranges:
 
180
        file_size = os.fstat(file.fileno())[6]
 
181
        tail, ranges = self.parse_ranges(ranges_header_value)
 
182
        # Normalize tail into ranges
 
183
        if tail != 0:
 
184
            ranges.append((file_size - tail, file_size))
 
185
 
 
186
        self._satisfiable_ranges = True
 
187
        if len(ranges) == 0:
 
188
            self._satisfiable_ranges = False
 
189
        else:
 
190
            def check_range(range_specifier):
 
191
                start, end = range_specifier
 
192
                # RFC2616 14.35, ranges are invalid if start >= file_size
 
193
                if start >= file_size:
 
194
                    self._satisfiable_ranges = False # Side-effect !
 
195
                    return 0, 0
 
196
                # RFC2616 14.35, end values should be truncated
 
197
                # to file_size -1 if they exceed it
 
198
                end = min(end, file_size - 1)
 
199
                return start, end
 
200
 
 
201
            ranges = map(check_range, ranges)
 
202
 
 
203
        if not self._satisfiable_ranges:
286
204
            # RFC2616 14.16 and 14.35 says that when a server
287
205
            # encounters unsatisfiable range specifiers, it
288
206
            # SHOULD return a 416.
289
 
            f.close()
 
207
            file.close()
290
208
            # FIXME: We SHOULD send a Content-Range header too,
291
209
            # but the implementation of send_error does not
292
210
            # allows that. So far.
295
213
 
296
214
        if len(ranges) == 1:
297
215
            (start, end) = ranges[0]
298
 
            self.get_single_range(f, file_size, start, end)
 
216
            self.get_single_range(file, file_size, start, end)
299
217
        else:
300
 
            self.get_multiple_ranges(f, file_size, ranges)
301
 
        f.close()
302
 
 
303
 
    def translate_path(self, path):
304
 
        """Translate a /-separated PATH to the local filename syntax.
305
 
 
306
 
        If the server requires it, proxy the path before the usual translation
307
 
        """
308
 
        if self.server.test_case_server.proxy_requests:
309
 
            # We need to act as a proxy and accept absolute urls,
310
 
            # which SimpleHTTPRequestHandler (parent) is not
311
 
            # ready for. So we just drop the protocol://host:port
312
 
            # part in front of the request-url (because we know
313
 
            # we would not forward the request to *another*
314
 
            # proxy).
315
 
 
316
 
            # So we do what SimpleHTTPRequestHandler.translate_path
317
 
            # do beginning with python 2.4.3: abandon query
318
 
            # parameters, scheme, host port, etc (which ensure we
319
 
            # provide the right behaviour on all python versions).
 
218
            self.get_multiple_ranges(file, file_size, ranges)
 
219
        file.close()
 
220
 
 
221
    if sys.platform == 'win32':
 
222
        # On win32 you cannot access non-ascii filenames without
 
223
        # decoding them into unicode first.
 
224
        # However, under Linux, you can access bytestream paths
 
225
        # without any problems. If this function was always active
 
226
        # it would probably break tests when LANG=C was set
 
227
        def translate_path(self, path):
 
228
            """Translate a /-separated PATH to the local filename syntax.
 
229
 
 
230
            For bzr, all url paths are considered to be utf8 paths.
 
231
            On Linux, you can access these paths directly over the bytestream
 
232
            request, but on win32, you must decode them, and access them
 
233
            as Unicode files.
 
234
            """
 
235
            # abandon query parameters
320
236
            path = urlparse.urlparse(path)[2]
321
 
            # And now, we can apply *our* trick to proxy files
322
 
            path += '-proxied'
323
 
 
324
 
        return self._translate_path(path)
325
 
 
326
 
    def _translate_path(self, path):
327
 
        """Translate a /-separated PATH to the local filename syntax.
328
 
 
329
 
        Note that we're translating http URLs here, not file URLs.
330
 
        The URL root location is the server's startup directory.
331
 
        Components that mean special things to the local file system
332
 
        (e.g. drive or directory names) are ignored.  (XXX They should
333
 
        probably be diagnosed.)
334
 
 
335
 
        Override from python standard library to stop it calling os.getcwd()
336
 
        """
337
 
        # abandon query parameters
338
 
        path = urlparse.urlparse(path)[2]
339
 
        path = posixpath.normpath(urlutils.unquote(path))
340
 
        path = path.decode('utf-8')
341
 
        words = path.split('/')
342
 
        words = filter(None, words)
343
 
        path = self._cwd
344
 
        for num, word in enumerate(words):
345
 
            if num == 0:
 
237
            path = posixpath.normpath(urllib.unquote(path))
 
238
            path = path.decode('utf-8')
 
239
            words = path.split('/')
 
240
            words = filter(None, words)
 
241
            path = os.getcwdu()
 
242
            for word in words:
346
243
                drive, word = os.path.splitdrive(word)
347
 
            head, word = os.path.split(word)
348
 
            if word in (os.curdir, os.pardir): continue
349
 
            path = os.path.join(path, word)
350
 
        return path
351
 
 
352
 
 
353
 
class TestingHTTPServerMixin:
354
 
 
355
 
    def __init__(self, test_case_server):
356
 
        # test_case_server can be used to communicate between the
357
 
        # tests and the server (or the request handler and the
358
 
        # server), allowing dynamic behaviors to be defined from
359
 
        # the tests cases.
360
 
        self.test_case_server = test_case_server
361
 
        self._home_dir = test_case_server._home_dir
362
 
 
363
 
 
364
 
class TestingHTTPServer(test_server.TestingTCPServer, TestingHTTPServerMixin):
365
 
 
366
 
    def __init__(self, server_address, request_handler_class,
367
 
                 test_case_server):
368
 
        test_server.TestingTCPServer.__init__(self, server_address,
369
 
                                              request_handler_class)
370
 
        TestingHTTPServerMixin.__init__(self, test_case_server)
371
 
 
372
 
 
373
 
class TestingThreadingHTTPServer(test_server.TestingThreadingTCPServer,
374
 
                                 TestingHTTPServerMixin):
375
 
    """A threading HTTP test server for HTTP 1.1.
376
 
 
377
 
    Since tests can initiate several concurrent connections to the same http
378
 
    server, we need an independent connection for each of them. We achieve that
379
 
    by spawning a new thread for each connection.
380
 
    """
381
 
    def __init__(self, server_address, request_handler_class,
382
 
                 test_case_server):
383
 
        test_server.TestingThreadingTCPServer.__init__(self, server_address,
384
 
                                                       request_handler_class)
385
 
        TestingHTTPServerMixin.__init__(self, test_case_server)
386
 
 
387
 
 
388
 
class HttpServer(test_server.TestingTCPServerInAThread):
 
244
                head, word = os.path.split(word)
 
245
                if word in (os.curdir, os.pardir): continue
 
246
                path = os.path.join(path, word)
 
247
            return path
 
248
 
 
249
 
 
250
class TestingHTTPServer(BaseHTTPServer.HTTPServer):
 
251
    def __init__(self, server_address, RequestHandlerClass, test_case):
 
252
        BaseHTTPServer.HTTPServer.__init__(self, server_address,
 
253
                                                RequestHandlerClass)
 
254
        self.test_case = test_case
 
255
 
 
256
 
 
257
class HttpServer(Server):
389
258
    """A test server for http transports.
390
259
 
391
260
    Subclasses can provide a specific request handler.
392
261
    """
393
262
 
394
 
    # The real servers depending on the protocol
395
 
    http_server_class = {'HTTP/1.0': TestingHTTPServer,
396
 
                         'HTTP/1.1': TestingThreadingHTTPServer,
397
 
                         }
398
 
 
399
 
    # Whether or not we proxy the requests (see
400
 
    # TestingHTTPRequestHandler.translate_path).
401
 
    proxy_requests = False
402
 
 
403
263
    # used to form the url that connects to this server
404
264
    _url_protocol = 'http'
405
265
 
406
 
    def __init__(self, request_handler=TestingHTTPRequestHandler,
407
 
                 protocol_version=None):
408
 
        """Constructor.
409
 
 
410
 
        :param request_handler: a class that will be instantiated to handle an
411
 
            http connection (one or several requests).
412
 
 
413
 
        :param protocol_version: if specified, will override the protocol
414
 
            version of the request handler.
415
 
        """
416
 
        # Depending on the protocol version, we will create the approriate
417
 
        # server
418
 
        if protocol_version is None:
419
 
            # Use the request handler one
420
 
            proto_vers = request_handler.protocol_version
421
 
        else:
422
 
            # Use our own, it will be used to override the request handler
423
 
            # one too.
424
 
            proto_vers = protocol_version
425
 
        # Get the appropriate server class for the required protocol
426
 
        serv_cls = self.http_server_class.get(proto_vers, None)
427
 
        if serv_cls is None:
428
 
            raise httplib.UnknownProtocol(proto_vers)
429
 
        self.host = 'localhost'
430
 
        self.port = 0
431
 
        super(HttpServer, self).__init__((self.host, self.port),
432
 
                                         serv_cls,
433
 
                                         request_handler)
434
 
        self.protocol_version = proto_vers
435
 
        # Allows tests to verify number of GET requests issued
436
 
        self.GET_request_nb = 0
437
 
        self._http_base_url = None
438
 
        self.logs = []
439
 
 
440
 
    def create_server(self):
441
 
        return self.server_class(
442
 
            (self.host, self.port), self.request_handler_class, self)
 
266
    # Subclasses can provide a specific request handler
 
267
    def __init__(self, request_handler=TestingHTTPRequestHandler):
 
268
        Server.__init__(self)
 
269
        self.request_handler = request_handler
 
270
 
 
271
    def _get_httpd(self):
 
272
        return TestingHTTPServer(('localhost', 0),
 
273
                                  self.request_handler,
 
274
                                  self)
 
275
 
 
276
    def _http_start(self):
 
277
        httpd = None
 
278
        httpd = self._get_httpd()
 
279
        host, self.port = httpd.socket.getsockname()
 
280
        self._http_base_url = '%s://localhost:%s/' % (self._url_protocol,
 
281
                                                      self.port)
 
282
        self._http_starting.release()
 
283
        httpd.socket.settimeout(0.1)
 
284
 
 
285
        while self._http_running:
 
286
            try:
 
287
                httpd.handle_request()
 
288
            except socket.timeout:
 
289
                pass
443
290
 
444
291
    def _get_remote_url(self, path):
445
292
        path_parts = path.split(os.path.sep)
457
304
        """Capture Server log output."""
458
305
        self.logs.append(format % args)
459
306
 
460
 
    def start_server(self, backing_transport_server=None):
461
 
        """See bzrlib.transport.Server.start_server.
462
 
 
463
 
        :param backing_transport_server: The transport that requests over this
464
 
            protocol should be forwarded to. Note that this is currently not
465
 
            supported for HTTP.
466
 
        """
467
 
        # XXX: TODO: make the server back onto vfs_server rather than local
468
 
        # disk.
469
 
        if not (backing_transport_server is None
470
 
                or isinstance(backing_transport_server,
471
 
                              test_server.LocalURLServer)):
472
 
            raise AssertionError(
473
 
                "HTTPServer currently assumes local transport, got %s" %
474
 
                backing_transport_server)
 
307
    def setUp(self):
 
308
        """See bzrlib.transport.Server.setUp."""
475
309
        self._home_dir = os.getcwdu()
476
310
        self._local_path_parts = self._home_dir.split(os.path.sep)
 
311
        self._http_starting = threading.Lock()
 
312
        self._http_starting.acquire()
 
313
        self._http_running = True
 
314
        self._http_base_url = None
 
315
        self._http_thread = threading.Thread(target=self._http_start)
 
316
        self._http_thread.setDaemon(True)
 
317
        self._http_thread.start()
 
318
        # Wait for the server thread to start (i.e release the lock)
 
319
        self._http_starting.acquire()
 
320
        self._http_starting.release()
477
321
        self.logs = []
478
322
 
479
 
        super(HttpServer, self).start_server()
480
 
        self._http_base_url = '%s://%s:%s/' % (
481
 
            self._url_protocol, self.host, self.port)
 
323
    def tearDown(self):
 
324
        """See bzrlib.transport.Server.tearDown."""
 
325
        self._http_running = False
 
326
        self._http_thread.join()
482
327
 
483
328
    def get_url(self):
484
329
        """See bzrlib.transport.Server.get_url."""
488
333
        """See bzrlib.transport.Server.get_bogus_url."""
489
334
        # this is chosen to try to prevent trouble with proxies, weird dns,
490
335
        # etc
491
 
        return self._url_protocol + '://127.0.0.1:1/'
 
336
        return 'http://127.0.0.1:1/'
492
337
 
493
338
 
494
339
class HttpServer_urllib(HttpServer):