Sandbox reviewboard.accounts.AuthBackend

Review Request #6399 — Created Oct. 4, 2014 and submitted — Latest diff uploaded

Information

Review Board
master
bc8a283...

Reviewers

Extensions that create an AuthBackend subclass (using the AuthBackendHook) can throw exceptions inside Reviewboard. The parts of Reviewboard that call those methods have been sandboxed.

Now when an AuthBackend subclass from an extension throws an exception; Reviewboard logs the errors with enough information to find which method in the AuthBackend subclass threw the exception.

Unit tests have been written to make sure that functions from an AuthBackend subclass have been called, and when an exception is thrown it gets logged.

The tests fail without the sanboxing, and succeed with it.

Diff Revision 1

This is not the most recent revision of the diff. The latest diff is revision 29. See what's changed.

orig
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
reviewboard/extensions/tests.py
Loading...