Skip to content

Commit

Permalink
Merge pull request #787 from Natay/master
Browse files Browse the repository at this point in the history
better logging
  • Loading branch information
ialbert committed Mar 31, 2021
2 parents 79cc8ce + 349c8d0 commit a1cd528
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions biostar/accounts/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,10 +175,12 @@ def user_signup(request):

form = forms.SignUpWithCaptcha(request.POST)
if form.is_valid():

user = form.save()
login(request, user, backend="django.contrib.auth.backends.ModelBackend")
Profile.objects.filter(user=user).update(last_login=now())
tasks.verification_email.spool(user_id=user.pk)

return redirect("/")
else:
messages.error(request, "Invalid form submission")
Expand Down
2 changes: 1 addition & 1 deletion biostar/utils/decorators.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def inner(request, **kwargs):
# Expected parameter names.
diff = incoming - allowed
if diff:
logger.error(f"invalid get request parameters {diff}")
logger.warning(f"invalid get request parameters {diff}")
raise Http404("Parameter does not exist.")

return func(request, **kwargs)
Expand Down

0 comments on commit a1cd528

Please sign in to comment.