~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/HttpServer.py

  • Committer: Robert Collins
  • Date: 2007-11-18 19:56:39 UTC
  • mfrom: (3006 +trunk)
  • mto: This revision was merged to the branch mainline in revision 3038.
  • Revision ID: robertc@robertcollins.net-20071118195639-m6zf3d5ljjw88kkn
Merge bzr.dev.

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
from bzrlib.transport.local import LocalURLServer
 
33
 
 
34
 
 
35
class WebserverNotAvailable(Exception):
 
36
    pass
29
37
 
30
38
 
31
39
class BadWebserverPath(ValueError):
33
41
        return 'path %s is not in %s' % self.args
34
42
 
35
43
 
36
 
class TestingHTTPRequestHandler(SimpleHTTPServer.SimpleHTTPRequestHandler):
 
44
class TestingHTTPRequestHandler(SimpleHTTPRequestHandler):
37
45
    """Handles one request.
38
46
 
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.
 
47
    A TestingHTTPRequestHandler is instantiated for every request
 
48
    received by the associated server.
43
49
    """
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
57
50
 
58
51
    def log_message(self, format, *args):
59
52
        tcs = self.server.test_case_server
64
57
                self.headers.get('referer', '-'),
65
58
                self.headers.get('user-agent', '-'))
66
59
 
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()
74
 
 
75
60
    def handle_one_request(self):
76
61
        """Handle a single HTTP request.
77
62
 
79
64
        connection early to avoid polluting the test results.
80
65
        """
81
66
        try:
82
 
            self._handle_one_request()
 
67
            SimpleHTTPRequestHandler.handle_one_request(self)
83
68
        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.
87
 
            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)):
 
69
            if (len(e.args) > 0
 
70
                and e.args[0] in (errno.EPIPE, errno.ECONNRESET,
 
71
                                  errno.ECONNABORTED,)):
 
72
                self.close_connection = 1
 
73
                pass
 
74
            else:
91
75
                raise
92
76
 
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
    _range_regexp = re.compile(r'^(?P<start>\d+)-(?P<end>\d+)$')
132
78
    _tail_regexp = re.compile(r'^-(?P<tail>\d+)$')
133
79
 
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.
 
80
    def parse_ranges(self, ranges_header):
 
81
        """Parse the range header value and returns ranges and tail.
 
82
 
 
83
        RFC2616 14.35 says that syntactically invalid range
 
84
        specifiers MUST be ignored. In that case, we return 0 for
 
85
        tail and [] for ranges.
146
86
        """
 
87
        tail = 0
 
88
        ranges = []
147
89
        if not ranges_header.startswith('bytes='):
148
90
            # Syntactically invalid header
149
 
            return None
 
91
            return 0, []
150
92
 
151
 
        tail = None
152
 
        ranges = []
153
93
        ranges_header = ranges_header[len('bytes='):]
154
94
        for range_str in ranges_header.split(','):
 
95
            # FIXME: RFC2616 says end is optional and default to file_size
155
96
            range_match = self._range_regexp.match(range_str)
156
97
            if range_match is not None:
157
98
                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)
 
99
                end = int(range_match.group('end'))
164
100
                if start > end:
165
101
                    # Syntactically invalid range
166
 
                    return None
 
102
                    return 0, []
167
103
                ranges.append((start, end))
168
104
            else:
169
105
                tail_match = self._tail_regexp.match(range_str)
171
107
                    tail = int(tail_match.group('tail'))
172
108
                else:
173
109
                    # 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)
 
110
                    return 0, []
 
111
        return tail, ranges
209
112
 
210
113
    def send_range_content(self, file, start, length):
211
114
        file.seek(start)
227
130
    def get_multiple_ranges(self, file, file_size, ranges):
228
131
        self.send_response(206)
229
132
        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)
 
133
        boundary = "%d" % random.randint(0,0x7FFFFFFF)
 
134
        self.send_header("Content-Type",
 
135
                         "multipart/byteranges; boundary=%s" % boundary)
246
136
        self.end_headers()
247
 
 
248
 
        # Send the multipart body
249
137
        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))
 
138
            self.wfile.write("--%s\r\n" % boundary)
 
139
            self.send_header("Content-type", 'application/octet-stream')
 
140
            self.send_header("Content-Range", "bytes %d-%d/%d" % (start,
 
141
                                                                  end,
 
142
                                                                  file_size))
254
143
            self.end_headers()
255
144
            self.send_range_content(file, start, end - start + 1)
256
 
        # Final boundary
257
 
        self.wfile.write(boundary_line)
 
145
            self.wfile.write("--%s\r\n" % boundary)
258
146
 
259
147
    def do_GET(self):
260
148
        """Serve a GET request.
261
149
 
262
150
        Handles the Range header.
263
151
        """
264
 
        # Update statistics
265
 
        self.server.test_case_server.GET_request_nb += 1
266
152
 
267
153
        path = self.translate_path(self.path)
268
154
        ranges_header_value = self.headers.get('Range')
269
155
        if ranges_header_value is None or os.path.isdir(path):
270
156
            # Let the mother class handle most cases
271
 
            return SimpleHTTPServer.SimpleHTTPRequestHandler.do_GET(self)
 
157
            return SimpleHTTPRequestHandler.do_GET(self)
272
158
 
273
159
        try:
274
160
            # Always read in binary mode. Opening files in text
275
161
            # mode may cause newline translations, making the
276
162
            # actual size of the content transmitted *less* than
277
163
            # the content-length!
278
 
            f = open(path, 'rb')
 
164
            file = open(path, 'rb')
279
165
        except IOError:
280
166
            self.send_error(404, "File not found")
281
167
            return
282
168
 
283
 
        file_size = os.fstat(f.fileno())[6]
284
 
        ranges = self._parse_ranges(ranges_header_value, file_size)
285
 
        if not ranges:
 
169
        file_size = os.fstat(file.fileno())[6]
 
170
        tail, ranges = self.parse_ranges(ranges_header_value)
 
171
        # Normalize tail into ranges
 
172
        if tail != 0:
 
173
            ranges.append((file_size - tail, file_size))
 
174
 
 
175
        self._satisfiable_ranges = True
 
176
        if len(ranges) == 0:
 
177
            self._satisfiable_ranges = False
 
178
        else:
 
179
            def check_range(range_specifier):
 
180
                start, end = range_specifier
 
181
                # RFC2616 14.35, ranges are invalid if start >= file_size
 
182
                if start >= file_size:
 
183
                    self._satisfiable_ranges = False # Side-effect !
 
184
                    return 0, 0
 
185
                # RFC2616 14.35, end values should be truncated
 
186
                # to file_size -1 if they exceed it
 
187
                end = min(end, file_size - 1)
 
188
                return start, end
 
189
 
 
190
            ranges = map(check_range, ranges)
 
191
 
 
192
        if not self._satisfiable_ranges:
286
193
            # RFC2616 14.16 and 14.35 says that when a server
287
194
            # encounters unsatisfiable range specifiers, it
288
195
            # SHOULD return a 416.
289
 
            f.close()
 
196
            file.close()
290
197
            # FIXME: We SHOULD send a Content-Range header too,
291
198
            # but the implementation of send_error does not
292
199
            # allows that. So far.
295
202
 
296
203
        if len(ranges) == 1:
297
204
            (start, end) = ranges[0]
298
 
            self.get_single_range(f, file_size, start, end)
 
205
            self.get_single_range(file, file_size, start, end)
299
206
        else:
300
 
            self.get_multiple_ranges(f, file_size, ranges)
301
 
        f.close()
 
207
            self.get_multiple_ranges(file, file_size, ranges)
 
208
        file.close()
302
209
 
303
210
    def translate_path(self, path):
304
211
        """Translate a /-separated PATH to the local filename syntax.
324
231
        return self._translate_path(path)
325
232
 
326
233
    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:
 
234
        return SimpleHTTPRequestHandler.translate_path(self, path)
 
235
 
 
236
    if sys.platform == 'win32':
 
237
        # On win32 you cannot access non-ascii filenames without
 
238
        # decoding them into unicode first.
 
239
        # However, under Linux, you can access bytestream paths
 
240
        # without any problems. If this function was always active
 
241
        # it would probably break tests when LANG=C was set
 
242
        def _translate_path(self, path):
 
243
            """Translate a /-separated PATH to the local filename syntax.
 
244
 
 
245
            For bzr, all url paths are considered to be utf8 paths.
 
246
            On Linux, you can access these paths directly over the bytestream
 
247
            request, but on win32, you must decode them, and access them
 
248
            as Unicode files.
 
249
            """
 
250
            # abandon query parameters
 
251
            path = urlparse.urlparse(path)[2]
 
252
            path = posixpath.normpath(urllib.unquote(path))
 
253
            path = path.decode('utf-8')
 
254
            words = path.split('/')
 
255
            words = filter(None, words)
 
256
            path = os.getcwdu()
 
257
            for word in words:
346
258
                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):
 
259
                head, word = os.path.split(word)
 
260
                if word in (os.curdir, os.pardir): continue
 
261
                path = os.path.join(path, word)
 
262
            return path
 
263
 
 
264
 
 
265
class TestingHTTPServer(BaseHTTPServer.HTTPServer):
 
266
 
 
267
    def __init__(self, server_address, RequestHandlerClass,
 
268
                 test_case_server):
 
269
        BaseHTTPServer.HTTPServer.__init__(self, server_address,
 
270
                                           RequestHandlerClass)
356
271
        # test_case_server can be used to communicate between the
357
272
        # tests and the server (or the request handler and the
358
273
        # server), allowing dynamic behaviors to be defined from
359
274
        # the tests cases.
360
275
        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):
 
276
 
 
277
    def server_close(self):
 
278
        """Called to clean-up the server.
 
279
 
 
280
        Since the server may be in a blocking read, we shutdown the socket
 
281
        before closing it.
 
282
        """
 
283
        self.socket.shutdown(socket.SHUT_RDWR)
 
284
        BaseHTTPServer.HTTPServer.server_close(self)
 
285
 
 
286
 
 
287
class HttpServer(Server):
389
288
    """A test server for http transports.
390
289
 
391
290
    Subclasses can provide a specific request handler.
392
291
    """
393
292
 
394
 
    # The real servers depending on the protocol
395
 
    http_server_class = {'HTTP/1.0': TestingHTTPServer,
396
 
                         'HTTP/1.1': TestingThreadingHTTPServer,
397
 
                         }
398
 
 
399
293
    # Whether or not we proxy the requests (see
400
294
    # TestingHTTPRequestHandler.translate_path).
401
295
    proxy_requests = False
403
297
    # used to form the url that connects to this server
404
298
    _url_protocol = 'http'
405
299
 
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)
 
300
    # Subclasses can provide a specific request handler
 
301
    def __init__(self, request_handler=TestingHTTPRequestHandler):
 
302
        Server.__init__(self)
 
303
        self.request_handler = request_handler
429
304
        self.host = 'localhost'
430
305
        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)
 
306
        self._httpd = None
 
307
 
 
308
    def _get_httpd(self):
 
309
        if self._httpd is None:
 
310
            self._httpd = TestingHTTPServer((self.host, self.port),
 
311
                                            self.request_handler,
 
312
                                            self)
 
313
            host, self.port = self._httpd.socket.getsockname()
 
314
        return self._httpd
 
315
 
 
316
    def _http_start(self):
 
317
        httpd = self._get_httpd()
 
318
        self._http_base_url = '%s://%s:%s/' % (self._url_protocol,
 
319
                                               self.host,
 
320
                                               self.port)
 
321
        self._http_starting.release()
 
322
 
 
323
        while self._http_running:
 
324
            try:
 
325
                httpd.handle_request()
 
326
            except socket.timeout:
 
327
                pass
443
328
 
444
329
    def _get_remote_url(self, path):
445
330
        path_parts = path.split(os.path.sep)
457
342
        """Capture Server log output."""
458
343
        self.logs.append(format % args)
459
344
 
460
 
    def start_server(self, backing_transport_server=None):
461
 
        """See bzrlib.transport.Server.start_server.
462
 
 
 
345
    def setUp(self, backing_transport_server=None):
 
346
        """See bzrlib.transport.Server.setUp.
 
347
        
463
348
        :param backing_transport_server: The transport that requests over this
464
349
            protocol should be forwarded to. Note that this is currently not
465
350
            supported for HTTP.
466
351
        """
467
352
        # XXX: TODO: make the server back onto vfs_server rather than local
468
353
        # 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)
 
354
        assert backing_transport_server is None or \
 
355
            isinstance(backing_transport_server, LocalURLServer), \
 
356
            "HTTPServer currently assumes local transport, got %s" % \
 
357
            backing_transport_server
475
358
        self._home_dir = os.getcwdu()
476
359
        self._local_path_parts = self._home_dir.split(os.path.sep)
 
360
        self._http_starting = threading.Lock()
 
361
        self._http_starting.acquire()
 
362
        self._http_running = True
 
363
        self._http_base_url = None
 
364
        self._http_thread = threading.Thread(target=self._http_start)
 
365
        self._http_thread.setDaemon(True)
 
366
        self._http_thread.start()
 
367
        # Wait for the server thread to start (i.e release the lock)
 
368
        self._http_starting.acquire()
 
369
        self._http_starting.release()
477
370
        self.logs = []
478
371
 
479
 
        super(HttpServer, self).start_server()
480
 
        self._http_base_url = '%s://%s:%s/' % (
481
 
            self._url_protocol, self.host, self.port)
 
372
    def tearDown(self):
 
373
        """See bzrlib.transport.Server.tearDown."""
 
374
        self._httpd.server_close()
 
375
        self._http_running = False
 
376
        self._http_thread.join()
482
377
 
483
378
    def get_url(self):
484
379
        """See bzrlib.transport.Server.get_url."""
488
383
        """See bzrlib.transport.Server.get_bogus_url."""
489
384
        # this is chosen to try to prevent trouble with proxies, weird dns,
490
385
        # etc
491
 
        return self._url_protocol + '://127.0.0.1:1/'
 
386
        return 'http://127.0.0.1:1/'
492
387
 
493
388
 
494
389
class HttpServer_urllib(HttpServer):