Skip to content
This repository was archived by the owner on May 26, 2020. It is now read-only.

Commit 23f5a51

Browse files
author
OrPo
committed
Using the general built-in 'pk' property of Django's User model instead of 'id'.
1 parent c37c9ff commit 23f5a51

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

rest_framework_jwt/tests/test_utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ def test_jwt_payload_handler(self):
1616
payload = utils.jwt_payload_handler(self.user)
1717

1818
self.assertTrue(isinstance(payload, dict))
19-
self.assertEqual(payload['user_id'], self.user.id)
19+
self.assertEqual(payload['user_id'], self.user.pk)
2020
self.assertEqual(payload['email'], self.email)
2121
self.assertEqual(payload['username'], self.username)
2222
self.assertTrue('exp' in payload)

rest_framework_jwt/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
def jwt_payload_handler(user):
88
return {
9-
'user_id': user.id,
9+
'user_id': user.pk,
1010
'email': user.email,
1111
'username': user.get_username(),
1212
'exp': datetime.datetime.utcnow() + api_settings.JWT_EXPIRATION_DELTA

0 commit comments

Comments
 (0)