Commit 16e5fdd9 authored by Matt Robenolt's avatar Matt Robenolt Committed by Aymeric Augustin
Browse files

[1.5.x] Propagate get_user_model exception from get_user 

Fixes #21439

Backport of 3560ef04 from master.

Conflicts:
	django/contrib/auth/tests/auth_backends.py
parent 8d1f3396

django/contrib/auth/backends.py

+1 −1
Original line number Diff line number Diff line
@@ -62,8 +62,8 @@ class ModelBackend(object): 
        return False



    def get_user(self, user_id):

        try:

        UserModel = get_user_model()

        try:

            return UserModel._default_manager.get(pk=user_id)

        except UserModel.DoesNotExist:

            return None

django/contrib/auth/tests/auth_backends.py

+24 −0
Original line number Diff line number Diff line
@@ -367,3 +367,27 @@ class InActiveUserBackendTest(TestCase): 
    def test_has_module_perms(self):

        self.assertEqual(self.user1.has_module_perms("app1"), False)

        self.assertEqual(self.user1.has_module_perms("app2"), False)





@skipIfCustomUser

class ImproperlyConfiguredUserModelTest(TestCase):

    """

    Tests that an exception from within get_user_model is propagated and doesn't

    raise an UnboundLocalError.



    Regression test for ticket #21439

    """

    def setUp(self):

        self.user1 = User.objects.create_user('test', 'test@example.com', 'test')

        self.client.login(

            username='test',

            password='test'

        )



    @override_settings(AUTH_USER_MODEL='thismodel.doesntexist')

    def test_does_not_shadow_exception(self):

        # Prepare a request object

        request = HttpRequest()

        request.session = self.client.session



        self.assertRaises(ImproperlyConfigured, get_user, request)