Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

[1.5.x] Propagate get_user_model exception from get_user

Fixes #21439

Backport of 3560ef0 from master.

Conflicts:
	django/contrib/auth/tests/auth_backends.py
  • Loading branch information...
commit 16e5fdd9cb8cca4d203b9255a9771b3c370e37d6 1 parent 8d1f339
@mattrobenolt mattrobenolt authored aaugustin committed
View
2  django/contrib/auth/backends.py
@@ -62,8 +62,8 @@ def has_module_perms(self, user_obj, app_label):
return False
def get_user(self, user_id):
+ UserModel = get_user_model()
try:
- UserModel = get_user_model()
return UserModel._default_manager.get(pk=user_id)
except UserModel.DoesNotExist:
return None
View
24 django/contrib/auth/tests/auth_backends.py
@@ -367,3 +367,27 @@ def test_has_perm(self):
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)
Please sign in to comment.
Something went wrong with that request. Please try again.