~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/HttpServer.py

  • Committer: Ian Clatworthy
  • Date: 2007-08-13 14:33:10 UTC
  • mto: (2733.1.1 ianc-integration)
  • mto: This revision was merged to the branch mainline in revision 2734.
  • Revision ID: ian.clatworthy@internode.on.net-20070813143310-twhj4la0qnupvze8
Added Quick Start Summary

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
25
28
import urllib
26
29
import urlparse
27
30
 
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
 
78
 
        We catch all socket errors occurring when the client close the
79
 
        connection early to avoid polluting the test results.
 
63
        You normally don't need to override this method; see the class
 
64
        __doc__ string for information on how to handle specific HTTP
 
65
        commands such as GET and POST.
 
66
 
80
67
        """
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.
 
68
        for i in xrange(1,11): # Don't try more than 10 times
 
69
            try:
 
70
                self.raw_requestline = self.rfile.readline()
 
71
            except socket.error, e:
 
72
                if e.args[0] in (errno.EAGAIN, errno.EWOULDBLOCK):
 
73
                    # omitted for now because some tests look at the log of
 
74
                    # the server and expect to see no errors.  see recent
 
75
                    # email thread. -- mbp 20051021. 
 
76
                    ## self.log_message('EAGAIN (%d) while reading from raw_requestline' % i)
 
77
                    time.sleep(0.01)
 
78
                    continue
 
79
                raise
 
80
            else:
 
81
                break
 
82
        if not self.raw_requestline:
87
83
            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+)?$')
 
84
            return
 
85
        if not self.parse_request(): # An error code has been sent, just exit
 
86
            return
 
87
        mname = 'do_' + self.command
 
88
        if getattr(self, mname, None) is None:
 
89
            self.send_error(501, "Unsupported method (%r)" % self.command)
 
90
            return
 
91
        method = getattr(self, mname)
 
92
        method()
 
93
 
 
94
    _range_regexp = re.compile(r'^(?P<start>\d+)-(?P<end>\d+)$')
132
95
    _tail_regexp = re.compile(r'^-(?P<tail>\d+)$')
133
96
 
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.
 
97
    def parse_ranges(self, ranges_header):
 
98
        """Parse the range header value and returns ranges and tail.
 
99
 
 
100
        RFC2616 14.35 says that syntactically invalid range
 
101
        specifiers MUST be ignored. In that case, we return 0 for
 
102
        tail and [] for ranges.
146
103
        """
 
104
        tail = 0
 
105
        ranges = []
147
106
        if not ranges_header.startswith('bytes='):
148
107
            # Syntactically invalid header
149
 
            return None
 
108
            return 0, []
150
109
 
151
 
        tail = None
152
 
        ranges = []
153
110
        ranges_header = ranges_header[len('bytes='):]
154
111
        for range_str in ranges_header.split(','):
 
112
            # FIXME: RFC2616 says end is optional and default to file_size
155
113
            range_match = self._range_regexp.match(range_str)
156
114
            if range_match is not None:
157
115
                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)
 
116
                end = int(range_match.group('end'))
164
117
                if start > end:
165
118
                    # Syntactically invalid range
166
 
                    return None
 
119
                    return 0, []
167
120
                ranges.append((start, end))
168
121
            else:
169
122
                tail_match = self._tail_regexp.match(range_str)
171
124
                    tail = int(tail_match.group('tail'))
172
125
                else:
173
126
                    # 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)
 
127
                    return 0, []
 
128
        return tail, ranges
209
129
 
210
130
    def send_range_content(self, file, start, length):
211
131
        file.seek(start)
227
147
    def get_multiple_ranges(self, file, file_size, ranges):
228
148
        self.send_response(206)
229
149
        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)
 
150
        boundary = "%d" % random.randint(0,0x7FFFFFFF)
 
151
        self.send_header("Content-Type",
 
152
                         "multipart/byteranges; boundary=%s" % boundary)
246
153
        self.end_headers()
247
 
 
248
 
        # Send the multipart body
249
154
        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))
 
155
            self.wfile.write("--%s\r\n" % boundary)
 
156
            self.send_header("Content-type", 'application/octet-stream')
 
157
            self.send_header("Content-Range", "bytes %d-%d/%d" % (start,
 
158
                                                                  end,
 
159
                                                                  file_size))
254
160
            self.end_headers()
255
161
            self.send_range_content(file, start, end - start + 1)
256
 
        # Final boundary
257
 
        self.wfile.write(boundary_line)
 
162
            self.wfile.write("--%s\r\n" % boundary)
258
163
 
259
164
    def do_GET(self):
260
165
        """Serve a GET request.
261
166
 
262
167
        Handles the Range header.
263
168
        """
264
 
        # Update statistics
265
 
        self.server.test_case_server.GET_request_nb += 1
266
169
 
267
170
        path = self.translate_path(self.path)
268
171
        ranges_header_value = self.headers.get('Range')
269
172
        if ranges_header_value is None or os.path.isdir(path):
270
173
            # Let the mother class handle most cases
271
 
            return SimpleHTTPServer.SimpleHTTPRequestHandler.do_GET(self)
 
174
            return SimpleHTTPRequestHandler.do_GET(self)
272
175
 
273
176
        try:
274
177
            # Always read in binary mode. Opening files in text
275
178
            # mode may cause newline translations, making the
276
179
            # actual size of the content transmitted *less* than
277
180
            # the content-length!
278
 
            f = open(path, 'rb')
 
181
            file = open(path, 'rb')
279
182
        except IOError:
280
183
            self.send_error(404, "File not found")
281
184
            return
282
185
 
283
 
        file_size = os.fstat(f.fileno())[6]
284
 
        ranges = self._parse_ranges(ranges_header_value, file_size)
285
 
        if not ranges:
 
186
        file_size = os.fstat(file.fileno())[6]
 
187
        tail, ranges = self.parse_ranges(ranges_header_value)
 
188
        # Normalize tail into ranges
 
189
        if tail != 0:
 
190
            ranges.append((file_size - tail, file_size))
 
191
 
 
192
        self._satisfiable_ranges = True
 
193
        if len(ranges) == 0:
 
194
            self._satisfiable_ranges = False
 
195
        else:
 
196
            def check_range(range_specifier):
 
197
                start, end = range_specifier
 
198
                # RFC2616 14.35, ranges are invalid if start >= file_size
 
199
                if start >= file_size:
 
200
                    self._satisfiable_ranges = False # Side-effect !
 
201
                    return 0, 0
 
202
                # RFC2616 14.35, end values should be truncated
 
203
                # to file_size -1 if they exceed it
 
204
                end = min(end, file_size - 1)
 
205
                return start, end
 
206
 
 
207
            ranges = map(check_range, ranges)
 
208
 
 
209
        if not self._satisfiable_ranges:
286
210
            # RFC2616 14.16 and 14.35 says that when a server
287
211
            # encounters unsatisfiable range specifiers, it
288
212
            # SHOULD return a 416.
289
 
            f.close()
 
213
            file.close()
290
214
            # FIXME: We SHOULD send a Content-Range header too,
291
215
            # but the implementation of send_error does not
292
216
            # allows that. So far.
295
219
 
296
220
        if len(ranges) == 1:
297
221
            (start, end) = ranges[0]
298
 
            self.get_single_range(f, file_size, start, end)
 
222
            self.get_single_range(file, file_size, start, end)
299
223
        else:
300
 
            self.get_multiple_ranges(f, file_size, ranges)
301
 
        f.close()
 
224
            self.get_multiple_ranges(file, file_size, ranges)
 
225
        file.close()
302
226
 
303
227
    def translate_path(self, path):
304
228
        """Translate a /-separated PATH to the local filename syntax.
324
248
        return self._translate_path(path)
325
249
 
326
250
    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(urllib.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:
 
251
        return SimpleHTTPRequestHandler.translate_path(self, path)
 
252
 
 
253
    if sys.platform == 'win32':
 
254
        # On win32 you cannot access non-ascii filenames without
 
255
        # decoding them into unicode first.
 
256
        # However, under Linux, you can access bytestream paths
 
257
        # without any problems. If this function was always active
 
258
        # it would probably break tests when LANG=C was set
 
259
        def _translate_path(self, path):
 
260
            """Translate a /-separated PATH to the local filename syntax.
 
261
 
 
262
            For bzr, all url paths are considered to be utf8 paths.
 
263
            On Linux, you can access these paths directly over the bytestream
 
264
            request, but on win32, you must decode them, and access them
 
265
            as Unicode files.
 
266
            """
 
267
            # abandon query parameters
 
268
            path = urlparse.urlparse(path)[2]
 
269
            path = posixpath.normpath(urllib.unquote(path))
 
270
            path = path.decode('utf-8')
 
271
            words = path.split('/')
 
272
            words = filter(None, words)
 
273
            path = os.getcwdu()
 
274
            for word in words:
346
275
                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):
 
276
                head, word = os.path.split(word)
 
277
                if word in (os.curdir, os.pardir): continue
 
278
                path = os.path.join(path, word)
 
279
            return path
 
280
 
 
281
 
 
282
class TestingHTTPServer(BaseHTTPServer.HTTPServer):
 
283
 
 
284
    def __init__(self, server_address, RequestHandlerClass,
 
285
                 test_case_server):
 
286
        BaseHTTPServer.HTTPServer.__init__(self, server_address,
 
287
                                           RequestHandlerClass)
356
288
        # test_case_server can be used to communicate between the
357
289
        # tests and the server (or the request handler and the
358
290
        # server), allowing dynamic behaviors to be defined from
359
291
        # the tests cases.
360
292
        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):
 
293
 
 
294
 
 
295
class HttpServer(Server):
389
296
    """A test server for http transports.
390
297
 
391
298
    Subclasses can provide a specific request handler.
392
299
    """
393
300
 
394
 
    # The real servers depending on the protocol
395
 
    http_server_class = {'HTTP/1.0': TestingHTTPServer,
396
 
                         'HTTP/1.1': TestingThreadingHTTPServer,
397
 
                         }
398
 
 
399
301
    # Whether or not we proxy the requests (see
400
302
    # TestingHTTPRequestHandler.translate_path).
401
303
    proxy_requests = False
403
305
    # used to form the url that connects to this server
404
306
    _url_protocol = 'http'
405
307
 
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)
 
308
    # Subclasses can provide a specific request handler
 
309
    def __init__(self, request_handler=TestingHTTPRequestHandler):
 
310
        Server.__init__(self)
 
311
        self.request_handler = request_handler
429
312
        self.host = 'localhost'
430
313
        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)
 
314
        self._httpd = None
 
315
 
 
316
    def _get_httpd(self):
 
317
        if self._httpd is None:
 
318
            self._httpd = TestingHTTPServer((self.host, self.port),
 
319
                                            self.request_handler,
 
320
                                            self)
 
321
            host, self.port = self._httpd.socket.getsockname()
 
322
        return self._httpd
 
323
 
 
324
    def _http_start(self):
 
325
        httpd = self._get_httpd()
 
326
        self._http_base_url = '%s://%s:%s/' % (self._url_protocol,
 
327
                                               self.host,
 
328
                                               self.port)
 
329
        self._http_starting.release()
 
330
        httpd.socket.settimeout(0.1)
 
331
 
 
332
        while self._http_running:
 
333
            try:
 
334
                httpd.handle_request()
 
335
            except socket.timeout:
 
336
                pass
443
337
 
444
338
    def _get_remote_url(self, path):
445
339
        path_parts = path.split(os.path.sep)
457
351
        """Capture Server log output."""
458
352
        self.logs.append(format % args)
459
353
 
460
 
    def start_server(self, backing_transport_server=None):
461
 
        """See bzrlib.transport.Server.start_server.
462
 
 
 
354
    def setUp(self, backing_transport_server=None):
 
355
        """See bzrlib.transport.Server.setUp.
 
356
        
463
357
        :param backing_transport_server: The transport that requests over this
464
358
            protocol should be forwarded to. Note that this is currently not
465
359
            supported for HTTP.
466
360
        """
467
361
        # XXX: TODO: make the server back onto vfs_server rather than local
468
362
        # 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)
 
363
        assert backing_transport_server is None or \
 
364
            isinstance(backing_transport_server, LocalURLServer), \
 
365
            "HTTPServer currently assumes local transport, got %s" % \
 
366
            backing_transport_server
475
367
        self._home_dir = os.getcwdu()
476
368
        self._local_path_parts = self._home_dir.split(os.path.sep)
 
369
        self._http_starting = threading.Lock()
 
370
        self._http_starting.acquire()
 
371
        self._http_running = True
 
372
        self._http_base_url = None
 
373
        self._http_thread = threading.Thread(target=self._http_start)
 
374
        self._http_thread.setDaemon(True)
 
375
        self._http_thread.start()
 
376
        # Wait for the server thread to start (i.e release the lock)
 
377
        self._http_starting.acquire()
 
378
        self._http_starting.release()
477
379
        self.logs = []
478
380
 
479
 
        super(HttpServer, self).start_server()
480
 
        self._http_base_url = '%s://%s:%s/' % (
481
 
            self._url_protocol, self.host, self.port)
 
381
    def tearDown(self):
 
382
        """See bzrlib.transport.Server.tearDown."""
 
383
        self._http_running = False
 
384
        self._http_thread.join()
482
385
 
483
386
    def get_url(self):
484
387
        """See bzrlib.transport.Server.get_url."""
488
391
        """See bzrlib.transport.Server.get_bogus_url."""
489
392
        # this is chosen to try to prevent trouble with proxies, weird dns,
490
393
        # etc
491
 
        return self._url_protocol + '://127.0.0.1:1/'
 
394
        return 'http://127.0.0.1:1/'
492
395
 
493
396
 
494
397
class HttpServer_urllib(HttpServer):