← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/server.py

  • Committer: Tarmac
  • Author(s): Vincent Ladeuil
  • Date: 2017-01-30 14:42:05 UTC
  • mfrom: (6620.1.1 trunk)
  • Revision ID: tarmac-20170130144205-r8fh2xpmiuxyozpv
Merge  2.7 into trunk including fix for bug #1657238 [r=vila]

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/smart/server.py
1
 
# Copyright (C) 2006, 2007, 2008 Canonical Ltd
 
1
# Copyright (C) 2006-2011 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
16
16
 
17
17
"""Server for smart-server protocol."""
18
18
 
 
19
from __future__ import absolute_import
 
20
 
19
21
import errno
 
22
import os.path
20
23
import socket
21
24
import sys
 
25
import time
22
26
import threading
23
27
 
24
 
from bzrlib.hooks import HookPoint, Hooks
 
28
from bzrlib.hooks import Hooks
25
29
from bzrlib import (
26
30
    errors,
27
31
    trace,
28
 
    transport,
 
32
    transport as _mod_transport,
29
33
)
 
34
from bzrlib.i18n import gettext
30
35
from bzrlib.lazy_import import lazy_import
31
36
lazy_import(globals(), """
32
 
from bzrlib.smart import medium
 
37
from bzrlib.smart import (
 
38
    medium,
 
39
    signals,
 
40
    )
 
41
from bzrlib.transport import (
 
42
    chroot,
 
43
    pathfilter,
 
44
    )
 
45
from bzrlib import (
 
46
    config,
 
47
    urlutils,
 
48
    )
33
49
""")
34
50
 
35
51
 
42
58
    hooks: An instance of SmartServerHooks.
43
59
    """
44
60
 
45
 
    def __init__(self, backing_transport, host='127.0.0.1', port=0,
46
 
                 root_client_path='/'):
 
61
    # This is the timeout on the socket we use .accept() on. It is exposed here
 
62
    # so the test suite can set it faster. (It thread.interrupt_main() will not
 
63
    # fire a KeyboardInterrupt during socket.accept)
 
64
    _ACCEPT_TIMEOUT = 1.0
 
65
    _SHUTDOWN_POLL_TIMEOUT = 1.0
 
66
    _LOG_WAITING_TIMEOUT = 10.0
 
67
 
 
68
    _timer = time.time
 
69
 
 
70
    def __init__(self, backing_transport, root_client_path='/',
 
71
                 client_timeout=None):
47
72
        """Construct a new server.
48
73
 
49
74
        To actually start it running, call either start_background_thread or
50
75
        serve.
51
76
 
52
77
        :param backing_transport: The transport to serve.
 
78
        :param root_client_path: The client path that will correspond to root
 
79
            of backing_transport.
 
80
        :param client_timeout: See SmartServerSocketStreamMedium's timeout
 
81
            parameter.
 
82
        """
 
83
        self.backing_transport = backing_transport
 
84
        self.root_client_path = root_client_path
 
85
        self._client_timeout = client_timeout
 
86
        self._active_connections = []
 
87
        # This is set to indicate we want to wait for clients to finish before
 
88
        # we disconnect.
 
89
        self._gracefully_stopping = False
 
90
 
 
91
    def start_server(self, host, port):
 
92
        """Create the server listening socket.
 
93
 
53
94
        :param host: Name of the interface to listen on.
54
95
        :param port: TCP port to listen on, or 0 to allocate a transient port.
55
 
        :param root_client_path: The client path that will correspond to root
56
 
            of backing_transport.
57
96
        """
58
97
        # let connections timeout so that we get a chance to terminate
59
98
        # Keep a reference to the exceptions we want to catch because the socket
79
118
        self._sockname = self._server_socket.getsockname()
80
119
        self.port = self._sockname[1]
81
120
        self._server_socket.listen(1)
82
 
        self._server_socket.settimeout(1)
83
 
        self.backing_transport = backing_transport
 
121
        self._server_socket.settimeout(self._ACCEPT_TIMEOUT)
 
122
        # Once we start accept()ing connections, we set started.
84
123
        self._started = threading.Event()
 
124
        # Once we stop accept()ing connections (and are closing the socket) we
 
125
        # set _stopped
85
126
        self._stopped = threading.Event()
86
 
        self.root_client_path = root_client_path
 
127
        # Once we have finished waiting for all clients, etc. We set
 
128
        # _fully_stopped
 
129
        self._fully_stopped = threading.Event()
87
130
 
88
 
    def serve(self, thread_name_suffix=''):
89
 
        self._should_terminate = False
90
 
        # for hooks we are letting code know that a server has started (and
91
 
        # later stopped).
 
131
    def _backing_urls(self):
92
132
        # There are three interesting urls:
93
133
        # The URL the server can be contacted on. (e.g. bzr://host/)
94
134
        # The URL that a commit done on the same machine as the server will
95
135
        # have within the servers space. (e.g. file:///home/user/source)
96
136
        # The URL that will be given to other hooks in the same process -
97
 
        # the URL of the backing transport itself. (e.g. chroot+:///)
 
137
        # the URL of the backing transport itself. (e.g. filtered-36195:///)
98
138
        # We need all three because:
99
139
        #  * other machines see the first
100
140
        #  * local commits on this machine should be able to be mapped to
104
144
        # The latter two urls are different aliases to the servers url,
105
145
        # so we group those in a list - as there might be more aliases
106
146
        # in the future.
107
 
        backing_urls = [self.backing_transport.base]
 
147
        urls = [self.backing_transport.base]
108
148
        try:
109
 
            backing_urls.append(self.backing_transport.external_url())
 
149
            urls.append(self.backing_transport.external_url())
110
150
        except errors.InProcessTransport:
111
151
            pass
 
152
        return urls
 
153
 
 
154
    def run_server_started_hooks(self, backing_urls=None):
 
155
        if backing_urls is None:
 
156
            backing_urls = self._backing_urls()
112
157
        for hook in SmartTCPServer.hooks['server_started']:
113
158
            hook(backing_urls, self.get_url())
114
159
        for hook in SmartTCPServer.hooks['server_started_ex']:
115
160
            hook(backing_urls, self)
 
161
 
 
162
    def run_server_stopped_hooks(self, backing_urls=None):
 
163
        if backing_urls is None:
 
164
            backing_urls = self._backing_urls()
 
165
        for hook in SmartTCPServer.hooks['server_stopped']:
 
166
            hook(backing_urls, self.get_url())
 
167
 
 
168
    def _stop_gracefully(self):
 
169
        trace.note(gettext('Requested to stop gracefully'))
 
170
        self._should_terminate = True
 
171
        self._gracefully_stopping = True
 
172
        for handler, _ in self._active_connections:
 
173
            handler._stop_gracefully()
 
174
 
 
175
    def _wait_for_clients_to_disconnect(self):
 
176
        self._poll_active_connections()
 
177
        if not self._active_connections:
 
178
            return
 
179
        trace.note(gettext('Waiting for %d client(s) to finish')
 
180
                   % (len(self._active_connections),))
 
181
        t_next_log = self._timer() + self._LOG_WAITING_TIMEOUT
 
182
        while self._active_connections:
 
183
            now = self._timer()
 
184
            if now >= t_next_log:
 
185
                trace.note(gettext('Still waiting for %d client(s) to finish')
 
186
                           % (len(self._active_connections),))
 
187
                t_next_log = now + self._LOG_WAITING_TIMEOUT
 
188
            self._poll_active_connections(self._SHUTDOWN_POLL_TIMEOUT)
 
189
 
 
190
    def serve(self, thread_name_suffix=''):
 
191
        # Note: There is a temptation to do
 
192
        #       signals.register_on_hangup(id(self), self._stop_gracefully)
 
193
        #       However, that creates a temporary object which is a bound
 
194
        #       method. signals._on_sighup is a WeakKeyDictionary so it
 
195
        #       immediately gets garbage collected, because nothing else
 
196
        #       references it. Instead, we need to keep a real reference to the
 
197
        #       bound method for the lifetime of the serve() function.
 
198
        stop_gracefully = self._stop_gracefully
 
199
        signals.register_on_hangup(id(self), stop_gracefully)
 
200
        self._should_terminate = False
 
201
        # for hooks we are letting code know that a server has started (and
 
202
        # later stopped).
 
203
        self.run_server_started_hooks()
116
204
        self._started.set()
117
205
        try:
118
206
            try:
124
212
                        pass
125
213
                    except self._socket_error, e:
126
214
                        # if the socket is closed by stop_background_thread
127
 
                        # we might get a EBADF here, any other socket errors
128
 
                        # should get logged.
129
 
                        if e.args[0] != errno.EBADF:
130
 
                            trace.warning("listening socket error: %s", e)
 
215
                        # we might get a EBADF here, or if we get a signal we
 
216
                        # can get EINTR, any other socket errors should get
 
217
                        # logged.
 
218
                        if e.args[0] not in (errno.EBADF, errno.EINTR):
 
219
                            trace.warning(gettext("listening socket error: %s")
 
220
                                          % (e,))
131
221
                    else:
 
222
                        if self._should_terminate:
 
223
                            conn.close()
 
224
                            break
132
225
                        self.serve_conn(conn, thread_name_suffix)
 
226
                    # Cleanout any threads that have finished processing.
 
227
                    self._poll_active_connections()
133
228
            except KeyboardInterrupt:
134
229
                # dont log when CTRL-C'd.
135
230
                raise
136
231
            except Exception, e:
137
 
                trace.error("Unhandled smart server error.")
138
 
                trace.log_exception_quietly()
 
232
                trace.report_exception(sys.exc_info(), sys.stderr)
139
233
                raise
140
234
        finally:
141
 
            self._stopped.set()
142
235
            try:
143
236
                # ensure the server socket is closed.
144
237
                self._server_socket.close()
145
238
            except self._socket_error:
146
239
                # ignore errors on close
147
240
                pass
148
 
            for hook in SmartTCPServer.hooks['server_stopped']:
149
 
                hook(backing_urls, self.get_url())
 
241
            self._stopped.set()
 
242
            signals.unregister_on_hangup(id(self))
 
243
            self.run_server_stopped_hooks()
 
244
        if self._gracefully_stopping:
 
245
            self._wait_for_clients_to_disconnect()
 
246
        self._fully_stopped.set()
150
247
 
151
248
    def get_url(self):
152
249
        """Return the url of the server"""
153
 
        return "bzr://%s:%d/" % self._sockname
 
250
        return "bzr://%s:%s/" % (self._sockname[0], self._sockname[1])
 
251
 
 
252
    def _make_handler(self, conn):
 
253
        return medium.SmartServerSocketStreamMedium(
 
254
            conn, self.backing_transport, self.root_client_path,
 
255
            timeout=self._client_timeout)
 
256
 
 
257
    def _poll_active_connections(self, timeout=0.0):
 
258
        """Check to see if any active connections have finished.
 
259
 
 
260
        This will iterate through self._active_connections, and update any
 
261
        connections that are finished.
 
262
 
 
263
        :param timeout: The timeout to pass to thread.join(). By default, we
 
264
            set it to 0, so that we don't hang if threads are not done yet.
 
265
        :return: None
 
266
        """
 
267
        still_active = []
 
268
        for handler, thread in self._active_connections:
 
269
            thread.join(timeout)
 
270
            if thread.isAlive():
 
271
                still_active.append((handler, thread))
 
272
        self._active_connections = still_active
154
273
 
155
274
    def serve_conn(self, conn, thread_name_suffix):
156
275
        # For WIN32, where the timeout value from the listening socket
157
276
        # propagates to the newly accepted socket.
158
277
        conn.setblocking(True)
159
278
        conn.setsockopt(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)
160
 
        handler = medium.SmartServerSocketStreamMedium(
161
 
            conn, self.backing_transport, self.root_client_path)
162
279
        thread_name = 'smart-server-child' + thread_name_suffix
 
280
        handler = self._make_handler(conn)
163
281
        connection_thread = threading.Thread(
164
282
            None, handler.serve, name=thread_name)
 
283
        self._active_connections.append((handler, connection_thread))
165
284
        connection_thread.setDaemon(True)
166
285
        connection_thread.start()
 
286
        return connection_thread
167
287
 
168
288
    def start_background_thread(self, thread_name_suffix=''):
169
289
        self._started.clear()
209
329
        These are all empty initially, because by default nothing should get
210
330
        notified.
211
331
        """
212
 
        Hooks.__init__(self)
213
 
        self.create_hook(HookPoint('server_started',
 
332
        Hooks.__init__(self, "bzrlib.smart.server", "SmartTCPServer.hooks")
 
333
        self.add_hook('server_started',
214
334
            "Called by the bzr server when it starts serving a directory. "
215
335
            "server_started is called with (backing urls, public url), "
216
336
            "where backing_url is a list of URLs giving the "
217
337
            "server-specific directory locations, and public_url is the "
218
 
            "public URL for the directory being served.", (0, 16), None))
219
 
        self.create_hook(HookPoint('server_started_ex',
 
338
            "public URL for the directory being served.", (0, 16))
 
339
        self.add_hook('server_started_ex',
220
340
            "Called by the bzr server when it starts serving a directory. "
221
341
            "server_started is called with (backing_urls, server_obj).",
222
 
            (1, 17), None))
223
 
        self.create_hook(HookPoint('server_stopped',
 
342
            (1, 17))
 
343
        self.add_hook('server_stopped',
224
344
            "Called by the bzr server when it stops serving a directory. "
225
345
            "server_stopped is called with the same parameters as the "
226
 
            "server_started hook: (backing_urls, public_url).", (0, 16), None))
 
346
            "server_started hook: (backing_urls, public_url).", (0, 16))
 
347
        self.add_hook('server_exception',
 
348
            "Called by the bzr server when an exception occurs. "
 
349
            "server_exception is called with the sys.exc_info() tuple "
 
350
            "return true for the hook if the exception has been handled, "
 
351
            "in which case the server will exit normally.", (2, 4))
227
352
 
228
353
SmartTCPServer.hooks = SmartServerHooks()
229
354
 
230
355
 
231
 
class SmartTCPServer_for_testing(SmartTCPServer):
232
 
    """Server suitable for use by transport tests.
233
 
 
234
 
    This server is backed by the process's cwd.
235
 
    """
236
 
 
237
 
    def __init__(self, thread_name_suffix=''):
238
 
        SmartTCPServer.__init__(self, None)
239
 
        self.client_path_extra = None
240
 
        self.thread_name_suffix = thread_name_suffix
241
 
 
242
 
    def get_backing_transport(self, backing_transport_server):
243
 
        """Get a backing transport from a server we are decorating."""
244
 
        return transport.get_transport(backing_transport_server.get_url())
245
 
 
246
 
    def setUp(self, backing_transport_server=None,
247
 
              client_path_extra='/extra/'):
248
 
        """Set up server for testing.
249
 
 
250
 
        :param backing_transport_server: backing server to use.  If not
251
 
            specified, a LocalURLServer at the current working directory will
252
 
            be used.
253
 
        :param client_path_extra: a path segment starting with '/' to append to
254
 
            the root URL for this server.  For instance, a value of '/foo/bar/'
255
 
            will mean the root of the backing transport will be published at a
256
 
            URL like `bzr://127.0.0.1:nnnn/foo/bar/`, rather than
257
 
            `bzr://127.0.0.1:nnnn/`.  Default value is `extra`, so that tests
258
 
            by default will fail unless they do the necessary path translation.
259
 
        """
260
 
        if not client_path_extra.startswith('/'):
261
 
            raise ValueError(client_path_extra)
262
 
        from bzrlib.transport.chroot import ChrootServer
263
 
        if backing_transport_server is None:
264
 
            from bzrlib.transport.local import LocalURLServer
265
 
            backing_transport_server = LocalURLServer()
266
 
        self.chroot_server = ChrootServer(
267
 
            self.get_backing_transport(backing_transport_server))
268
 
        self.chroot_server.setUp()
269
 
        self.backing_transport = transport.get_transport(
270
 
            self.chroot_server.get_url())
271
 
        self.root_client_path = self.client_path_extra = client_path_extra
272
 
        self.start_background_thread(self.thread_name_suffix)
273
 
 
274
 
    def tearDown(self):
275
 
        self.stop_background_thread()
276
 
        self.chroot_server.tearDown()
277
 
 
278
 
    def get_url(self):
279
 
        url = super(SmartTCPServer_for_testing, self).get_url()
280
 
        return url[:-1] + self.client_path_extra
281
 
 
282
 
    def get_bogus_url(self):
283
 
        """Return a URL which will fail to connect"""
284
 
        return 'bzr://127.0.0.1:1/'
285
 
 
286
 
 
287
 
class ReadonlySmartTCPServer_for_testing(SmartTCPServer_for_testing):
288
 
    """Get a readonly server for testing."""
289
 
 
290
 
    def get_backing_transport(self, backing_transport_server):
291
 
        """Get a backing transport from a server we are decorating."""
292
 
        url = 'readonly+' + backing_transport_server.get_url()
293
 
        return transport.get_transport(url)
294
 
 
295
 
 
296
 
class SmartTCPServer_for_testing_v2_only(SmartTCPServer_for_testing):
297
 
    """A variation of SmartTCPServer_for_testing that limits the client to
298
 
    using RPCs in protocol v2 (i.e. bzr <= 1.5).
299
 
    """
300
 
 
301
 
    def get_url(self):
302
 
        url = super(SmartTCPServer_for_testing_v2_only, self).get_url()
303
 
        url = 'bzr-v2://' + url[len('bzr://'):]
304
 
        return url
305
 
 
306
 
 
307
 
class ReadonlySmartTCPServer_for_testing_v2_only(SmartTCPServer_for_testing_v2_only):
308
 
    """Get a readonly server for testing."""
309
 
 
310
 
    def get_backing_transport(self, backing_transport_server):
311
 
        """Get a backing transport from a server we are decorating."""
312
 
        url = 'readonly+' + backing_transport_server.get_url()
313
 
        return transport.get_transport(url)
314
 
 
315
 
 
316
 
def serve_bzr(transport, host=None, port=None, inet=False):
317
 
    from bzrlib import lockdir, ui
318
 
    from bzrlib.transport import get_transport
319
 
    from bzrlib.transport.chroot import ChrootServer
320
 
    chroot_server = ChrootServer(transport)
321
 
    chroot_server.setUp()
322
 
    transport = get_transport(chroot_server.get_url())
323
 
    if inet:
324
 
        smart_server = medium.SmartServerPipeStreamMedium(
325
 
            sys.stdin, sys.stdout, transport)
 
356
def _local_path_for_transport(transport):
 
357
    """Return a local path for transport, if reasonably possible.
 
358
    
 
359
    This function works even if transport's url has a "readonly+" prefix,
 
360
    unlike local_path_from_url.
 
361
    
 
362
    This essentially recovers the --directory argument the user passed to "bzr
 
363
    serve" from the transport passed to serve_bzr.
 
364
    """
 
365
    try:
 
366
        base_url = transport.external_url()
 
367
    except (errors.InProcessTransport, NotImplementedError):
 
368
        return None
326
369
    else:
327
 
        if host is None:
328
 
            host = medium.BZR_DEFAULT_INTERFACE
329
 
        if port is None:
330
 
            port = medium.BZR_DEFAULT_PORT
331
 
        smart_server = SmartTCPServer(transport, host=host, port=port)
332
 
        trace.note('listening on port: %s' % smart_server.port)
333
 
    # For the duration of this server, no UI output is permitted. note
334
 
    # that this may cause problems with blackbox tests. This should be
335
 
    # changed with care though, as we dont want to use bandwidth sending
336
 
    # progress over stderr to smart server clients!
337
 
    old_factory = ui.ui_factory
338
 
    old_lockdir_timeout = lockdir._DEFAULT_TIMEOUT_SECONDS
339
 
    try:
 
370
        # Strip readonly prefix
 
371
        if base_url.startswith('readonly+'):
 
372
            base_url = base_url[len('readonly+'):]
 
373
        try:
 
374
            return urlutils.local_path_from_url(base_url)
 
375
        except errors.InvalidURL:
 
376
            return None
 
377
 
 
378
 
 
379
class BzrServerFactory(object):
 
380
    """Helper class for serve_bzr."""
 
381
 
 
382
    def __init__(self, userdir_expander=None, get_base_path=None):
 
383
        self.cleanups = []
 
384
        self.base_path = None
 
385
        self.backing_transport = None
 
386
        if userdir_expander is None:
 
387
            userdir_expander = os.path.expanduser
 
388
        self.userdir_expander = userdir_expander
 
389
        if get_base_path is None:
 
390
            get_base_path = _local_path_for_transport
 
391
        self.get_base_path = get_base_path
 
392
 
 
393
    def _expand_userdirs(self, path):
 
394
        """Translate /~/ or /~user/ to e.g. /home/foo, using
 
395
        self.userdir_expander (os.path.expanduser by default).
 
396
 
 
397
        If the translated path would fall outside base_path, or the path does
 
398
        not start with ~, then no translation is applied.
 
399
 
 
400
        If the path is inside, it is adjusted to be relative to the base path.
 
401
 
 
402
        e.g. if base_path is /home, and the expanded path is /home/joe, then
 
403
        the translated path is joe.
 
404
        """
 
405
        result = path
 
406
        if path.startswith('~'):
 
407
            expanded = self.userdir_expander(path)
 
408
            if not expanded.endswith('/'):
 
409
                expanded += '/'
 
410
            if expanded.startswith(self.base_path):
 
411
                result = expanded[len(self.base_path):]
 
412
        return result
 
413
 
 
414
    def _make_expand_userdirs_filter(self, transport):
 
415
        return pathfilter.PathFilteringServer(transport, self._expand_userdirs)
 
416
 
 
417
    def _make_backing_transport(self, transport):
 
418
        """Chroot transport, and decorate with userdir expander."""
 
419
        self.base_path = self.get_base_path(transport)
 
420
        chroot_server = chroot.ChrootServer(transport)
 
421
        chroot_server.start_server()
 
422
        self.cleanups.append(chroot_server.stop_server)
 
423
        transport = _mod_transport.get_transport_from_url(chroot_server.get_url())
 
424
        if self.base_path is not None:
 
425
            # Decorate the server's backing transport with a filter that can
 
426
            # expand homedirs.
 
427
            expand_userdirs = self._make_expand_userdirs_filter(transport)
 
428
            expand_userdirs.start_server()
 
429
            self.cleanups.append(expand_userdirs.stop_server)
 
430
            transport = _mod_transport.get_transport_from_url(expand_userdirs.get_url())
 
431
        self.transport = transport
 
432
 
 
433
    def _get_stdin_stdout(self):
 
434
        return sys.stdin, sys.stdout
 
435
 
 
436
    def _make_smart_server(self, host, port, inet, timeout):
 
437
        if timeout is None:
 
438
            c = config.GlobalStack()
 
439
            timeout = c.get('serve.client_timeout')
 
440
        if inet:
 
441
            stdin, stdout = self._get_stdin_stdout()
 
442
            smart_server = medium.SmartServerPipeStreamMedium(
 
443
                stdin, stdout, self.transport, timeout=timeout)
 
444
        else:
 
445
            if host is None:
 
446
                host = medium.BZR_DEFAULT_INTERFACE
 
447
            if port is None:
 
448
                port = medium.BZR_DEFAULT_PORT
 
449
            smart_server = SmartTCPServer(self.transport,
 
450
                                          client_timeout=timeout)
 
451
            smart_server.start_server(host, port)
 
452
            trace.note(gettext('listening on port: %s') % smart_server.port)
 
453
        self.smart_server = smart_server
 
454
 
 
455
    def _change_globals(self):
 
456
        from bzrlib import lockdir, ui
 
457
        # For the duration of this server, no UI output is permitted. note
 
458
        # that this may cause problems with blackbox tests. This should be
 
459
        # changed with care though, as we dont want to use bandwidth sending
 
460
        # progress over stderr to smart server clients!
 
461
        old_factory = ui.ui_factory
 
462
        old_lockdir_timeout = lockdir._DEFAULT_TIMEOUT_SECONDS
 
463
        def restore_default_ui_factory_and_lockdir_timeout():
 
464
            ui.ui_factory = old_factory
 
465
            lockdir._DEFAULT_TIMEOUT_SECONDS = old_lockdir_timeout
 
466
        self.cleanups.append(restore_default_ui_factory_and_lockdir_timeout)
340
467
        ui.ui_factory = ui.SilentUIFactory()
341
468
        lockdir._DEFAULT_TIMEOUT_SECONDS = 0
342
 
        smart_server.serve()
 
469
        orig = signals.install_sighup_handler()
 
470
        def restore_signals():
 
471
            signals.restore_sighup_handler(orig)
 
472
        self.cleanups.append(restore_signals)
 
473
 
 
474
    def set_up(self, transport, host, port, inet, timeout):
 
475
        self._make_backing_transport(transport)
 
476
        self._make_smart_server(host, port, inet, timeout)
 
477
        self._change_globals()
 
478
 
 
479
    def tear_down(self):
 
480
        for cleanup in reversed(self.cleanups):
 
481
            cleanup()
 
482
 
 
483
 
 
484
def serve_bzr(transport, host=None, port=None, inet=False, timeout=None):
 
485
    """This is the default implementation of 'bzr serve'.
 
486
 
 
487
    It creates a TCP or pipe smart server on 'transport, and runs it.  The
 
488
    transport will be decorated with a chroot and pathfilter (using
 
489
    os.path.expanduser).
 
490
    """
 
491
    bzr_server = BzrServerFactory()
 
492
    try:
 
493
        bzr_server.set_up(transport, host, port, inet, timeout)
 
494
        bzr_server.smart_server.serve()
 
495
    except:
 
496
        hook_caught_exception = False
 
497
        for hook in SmartTCPServer.hooks['server_exception']:
 
498
            hook_caught_exception = hook(sys.exc_info())
 
499
        if not hook_caught_exception:
 
500
            raise
343
501
    finally:
344
 
        ui.ui_factory = old_factory
345
 
        lockdir._DEFAULT_TIMEOUT_SECONDS = old_lockdir_timeout
346
 
 
 
502
        bzr_server.tear_down()