dj-rest-auth icon indicating copy to clipboard operation
dj-rest-auth copied to clipboard

Incorrect Response in ResendEmailVerificationView

Open uzza1hossain opened this issue 1 year ago • 0 comments

When email is not found or already verified, shouldn't we return HTTP_404_NOT_FOUND? Current version: https://github.com/iMerica/dj-rest-auth/blob/master/dj_rest_auth/registration/views.py#LL122-L130

Proposed version:

def create(self, request, *args, **kwargs):
        serializer = self.get_serializer(data=request.data)
        serializer.is_valid(raise_exception=True)

        email = EmailAddress.objects.filter(**serializer.validated_data).first()
        if email and not email.verified:
            email.send_confirmation(request)
            return Response({'detail': _('ok')}, status=status.HTTP_200_OK)
        return Response({'detail': _('Not found.')}, status=status.HTTP_404_NOT_FOUND)

uzza1hossain avatar May 27 '23 15:05 uzza1hossain