djblets.extensions.SignalHook sandboxing

Review Request #6554 — Created Nov. 4, 2014 and submitted — Latest diff uploaded

Information

Djblets
master
24060c0...

Reviewers

Extensions that use a SignalHook to connect to a signal and run a callback function can throw exceptions inside Djblets. To fix this a new parameter, sandbox_errors which by default is set to True, has been added to SignalHook.

Now when sandbox_errors is set to True it will wrap the callback function in another function to log the exceptions that are thrown. If sandbox_errors is set to False, exceptions that are thrown are not caught by Djblets.

Two unit tests have been written to make sure the new parameter sandbox_errors logs exceptions when set to True, and does not log exceptions when set to False.

The test fails without the sandboxing, and succeeds with it.

    Loading...