New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ENG-2002] redirect user who is already registered #9450
[ENG-2002] redirect user who is already registered #9450
Conversation
91af7d4
to
5c15c6e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks for this improvement!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question on an assertion comment.
tests/test_views.py
Outdated
assert_equal(self.user.email_verifications, {}) | ||
|
||
res = self.app.get(url, auth=self.auth) | ||
assert_equal(res.status_code, 302, 'redirects to cas logout') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a correct comment? It looks like it's redirecting to the logged in dashboard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it's wrong, it's a good old copy and paste error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃帀 馃惂 馃帀
Purpose
Orcid sign on causing "bad token" sentry errors, I believe this is because the
email_verifications
is cleared when the user registers, so users attempting to register twice (knowingly do to button mashings or unknowingly due to email spam checking) have no token inemail_verifications
hence the error.Changes
QA Notes
No Migration, low risk
To reproduce replicate steps 1, 3 and 4 from this doc: https://docs.google.com/document/d/1wCk_nHQKCVozGk0j0UrU2VFWiF6Qk4qsIYaDzNezjvU/edit
Documentation
馃悶 fix, no docs.
Side Effects
None that I know of.
Ticket
https://openscience.atlassian.net/browse/ENG-2002