~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-12-18 23:41:30 UTC
  • mfrom: (3099.3.7 graph_optimization)
  • Revision ID: pqm@pqm.ubuntu.com-20071218234130-061grgxsaf1g7bao
(jam) Implement ParentProviders.get_parent_map() and deprecate
        get_parents()

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2006, 2007 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
145
        # Final boundary
257
 
        self.wfile.write(boundary_line)
 
146
        self.wfile.write("--%s\r\n" % boundary)
258
147
 
259
148
    def do_GET(self):
260
149
        """Serve a GET request.
268
157
        ranges_header_value = self.headers.get('Range')
269
158
        if ranges_header_value is None or os.path.isdir(path):
270
159
            # Let the mother class handle most cases
271
 
            return SimpleHTTPServer.SimpleHTTPRequestHandler.do_GET(self)
 
160
            return SimpleHTTPRequestHandler.do_GET(self)
272
161
 
273
162
        try:
274
163
            # Always read in binary mode. Opening files in text
275
164
            # mode may cause newline translations, making the
276
165
            # actual size of the content transmitted *less* than
277
166
            # the content-length!
278
 
            f = open(path, 'rb')
 
167
            file = open(path, 'rb')
279
168
        except IOError:
280
169
            self.send_error(404, "File not found")
281
170
            return
282
171
 
283
 
        file_size = os.fstat(f.fileno())[6]
284
 
        ranges = self._parse_ranges(ranges_header_value, file_size)
285
 
        if not ranges:
 
172
        file_size = os.fstat(file.fileno())[6]
 
173
        tail, ranges = self.parse_ranges(ranges_header_value)
 
174
        # Normalize tail into ranges
 
175
        if tail != 0:
 
176
            ranges.append((file_size - tail, file_size))
 
177
 
 
178
        self._satisfiable_ranges = True
 
179
        if len(ranges) == 0:
 
180
            self._satisfiable_ranges = False
 
181
        else:
 
182
            def check_range(range_specifier):
 
183
                start, end = range_specifier
 
184
                # RFC2616 14.35, ranges are invalid if start >= file_size
 
185
                if start >= file_size:
 
186
                    self._satisfiable_ranges = False # Side-effect !
 
187
                    return 0, 0
 
188
                # RFC2616 14.35, end values should be truncated
 
189
                # to file_size -1 if they exceed it
 
190
                end = min(end, file_size - 1)
 
191
                return start, end
 
192
 
 
193
            ranges = map(check_range, ranges)
 
194
 
 
195
        if not self._satisfiable_ranges:
286
196
            # RFC2616 14.16 and 14.35 says that when a server
287
197
            # encounters unsatisfiable range specifiers, it
288
198
            # SHOULD return a 416.
289
 
            f.close()
 
199
            file.close()
290
200
            # FIXME: We SHOULD send a Content-Range header too,
291
201
            # but the implementation of send_error does not
292
202
            # allows that. So far.
295
205
 
296
206
        if len(ranges) == 1:
297
207
            (start, end) = ranges[0]
298
 
            self.get_single_range(f, file_size, start, end)
 
208
            self.get_single_range(file, file_size, start, end)
299
209
        else:
300
 
            self.get_multiple_ranges(f, file_size, ranges)
301
 
        f.close()
 
210
            self.get_multiple_ranges(file, file_size, ranges)
 
211
        file.close()
302
212
 
303
213
    def translate_path(self, path):
304
214
        """Translate a /-separated PATH to the local filename syntax.
324
234
        return self._translate_path(path)
325
235
 
326
236
    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
        return SimpleHTTPRequestHandler.translate_path(self, path)
 
238
 
 
239
    if sys.platform == 'win32':
 
240
        # On win32 you cannot access non-ascii filenames without
 
241
        # decoding them into unicode first.
 
242
        # However, under Linux, you can access bytestream paths
 
243
        # without any problems. If this function was always active
 
244
        # it would probably break tests when LANG=C was set
 
245
        def _translate_path(self, path):
 
246
            """Translate a /-separated PATH to the local filename syntax.
 
247
 
 
248
            For bzr, all url paths are considered to be utf8 paths.
 
249
            On Linux, you can access these paths directly over the bytestream
 
250
            request, but on win32, you must decode them, and access them
 
251
            as Unicode files.
 
252
            """
 
253
            # abandon query parameters
 
254
            path = urlparse.urlparse(path)[2]
 
255
            path = posixpath.normpath(urllib.unquote(path))
 
256
            path = path.decode('utf-8')
 
257
            words = path.split('/')
 
258
            words = filter(None, words)
 
259
            path = os.getcwdu()
 
260
            for word in words:
346
261
                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):
 
262
                head, word = os.path.split(word)
 
263
                if word in (os.curdir, os.pardir): continue
 
264
                path = os.path.join(path, word)
 
265
            return path
 
266
 
 
267
 
 
268
class TestingHTTPServer(BaseHTTPServer.HTTPServer):
 
269
 
 
270
    def __init__(self, server_address, RequestHandlerClass,
 
271
                 test_case_server):
 
272
        BaseHTTPServer.HTTPServer.__init__(self, server_address,
 
273
                                           RequestHandlerClass)
356
274
        # test_case_server can be used to communicate between the
357
275
        # tests and the server (or the request handler and the
358
276
        # server), allowing dynamic behaviors to be defined from
359
277
        # the tests cases.
360
278
        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):
 
279
 
 
280
    def server_close(self):
 
281
        """Called to clean-up the server.
 
282
 
 
283
        Since the server may be in a blocking read, we shutdown the socket
 
284
        before closing it.
 
285
        """
 
286
        self.socket.shutdown(socket.SHUT_RDWR)
 
287
        BaseHTTPServer.HTTPServer.server_close(self)
 
288
 
 
289
 
 
290
class HttpServer(Server):
389
291
    """A test server for http transports.
390
292
 
391
293
    Subclasses can provide a specific request handler.
392
294
    """
393
295
 
394
 
    # The real servers depending on the protocol
395
 
    http_server_class = {'HTTP/1.0': TestingHTTPServer,
396
 
                         'HTTP/1.1': TestingThreadingHTTPServer,
397
 
                         }
398
 
 
399
296
    # Whether or not we proxy the requests (see
400
297
    # TestingHTTPRequestHandler.translate_path).
401
298
    proxy_requests = False
403
300
    # used to form the url that connects to this server
404
301
    _url_protocol = 'http'
405
302
 
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)
 
303
    # Subclasses can provide a specific request handler
 
304
    def __init__(self, request_handler=TestingHTTPRequestHandler):
 
305
        Server.__init__(self)
 
306
        self.request_handler = request_handler
429
307
        self.host = 'localhost'
430
308
        self.port = 0
431
 
        super(HttpServer, self).__init__((self.host, self.port),
432
 
                                         serv_cls,
433
 
                                         request_handler)
434
 
        self.protocol_version = proto_vers
 
309
        self._httpd = None
435
310
        # Allows tests to verify number of GET requests issued
436
311
        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)
 
312
 
 
313
    def _get_httpd(self):
 
314
        if self._httpd is None:
 
315
            self._httpd = TestingHTTPServer((self.host, self.port),
 
316
                                            self.request_handler,
 
317
                                            self)
 
318
            host, self.port = self._httpd.socket.getsockname()
 
319
        return self._httpd
 
320
 
 
321
    def _http_start(self):
 
322
        httpd = self._get_httpd()
 
323
        self._http_base_url = '%s://%s:%s/' % (self._url_protocol,
 
324
                                               self.host,
 
325
                                               self.port)
 
326
        self._http_starting.release()
 
327
 
 
328
        while self._http_running:
 
329
            try:
 
330
                httpd.handle_request()
 
331
            except socket.timeout:
 
332
                pass
443
333
 
444
334
    def _get_remote_url(self, path):
445
335
        path_parts = path.split(os.path.sep)
457
347
        """Capture Server log output."""
458
348
        self.logs.append(format % args)
459
349
 
460
 
    def start_server(self, backing_transport_server=None):
461
 
        """See bzrlib.transport.Server.start_server.
462
 
 
 
350
    def setUp(self, backing_transport_server=None):
 
351
        """See bzrlib.transport.Server.setUp.
 
352
        
463
353
        :param backing_transport_server: The transport that requests over this
464
354
            protocol should be forwarded to. Note that this is currently not
465
355
            supported for HTTP.
466
356
        """
467
357
        # XXX: TODO: make the server back onto vfs_server rather than local
468
358
        # 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)
 
359
        assert backing_transport_server is None or \
 
360
            isinstance(backing_transport_server, LocalURLServer), \
 
361
            "HTTPServer currently assumes local transport, got %s" % \
 
362
            backing_transport_server
475
363
        self._home_dir = os.getcwdu()
476
364
        self._local_path_parts = self._home_dir.split(os.path.sep)
 
365
        self._http_starting = threading.Lock()
 
366
        self._http_starting.acquire()
 
367
        self._http_running = True
 
368
        self._http_base_url = None
 
369
        self._http_thread = threading.Thread(target=self._http_start)
 
370
        self._http_thread.setDaemon(True)
 
371
        self._http_thread.start()
 
372
        # Wait for the server thread to start (i.e release the lock)
 
373
        self._http_starting.acquire()
 
374
        self._http_starting.release()
477
375
        self.logs = []
478
376
 
479
 
        super(HttpServer, self).start_server()
480
 
        self._http_base_url = '%s://%s:%s/' % (
481
 
            self._url_protocol, self.host, self.port)
 
377
    def tearDown(self):
 
378
        """See bzrlib.transport.Server.tearDown."""
 
379
        self._httpd.server_close()
 
380
        self._http_running = False
 
381
        self._http_thread.join()
482
382
 
483
383
    def get_url(self):
484
384
        """See bzrlib.transport.Server.get_url."""
488
388
        """See bzrlib.transport.Server.get_bogus_url."""
489
389
        # this is chosen to try to prevent trouble with proxies, weird dns,
490
390
        # etc
491
 
        return self._url_protocol + '://127.0.0.1:1/'
 
391
        return 'http://127.0.0.1:1/'
492
392
 
493
393
 
494
394
class HttpServer_urllib(HttpServer):