Skip to content
Snippets Groups Projects
Commit 759a472f authored by ml bonhomme's avatar ml bonhomme :bee:
Browse files

Warn the user when the password reset link is expired

parent 42d1adf4
No related branches found
No related tags found
1 merge request!2278Warn the user when the password reset link is expired
This commit is part of merge request !2278. Comments created here will be created in the context of that merge request.
......@@ -140,8 +140,10 @@ class PasswordResetConfirmSerializer(serializers.Serializer):
def save(self):
user = self.validated_data["user"]
if not user or not self.validated_data["valid_token"]:
if not user:
return
if not self.validated_data["valid_token"]:
raise serializers.ValidationError({"token": "This password reset link has expired. Please generate a new one using the 'Forgot your password?' link on the Login page."})
user.set_password(self.validated_data["password"])
user.save()
......
......@@ -75,7 +75,7 @@ class TestPasswordReset(FixtureAPITestCase):
},
format="json",
)
self.assertEqual(resp.status_code, status.HTTP_201_CREATED)
self.assertEqual(resp.status_code, status.HTTP_400_BAD_REQUEST)
self.user.refresh_from_db()
self.assertFalse(self.user.check_password("S3cr37Pa$$w0rd"))
self.assertEqual(token_gen_mock.check_token.call_count, 1)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment