Skip to content

Access to study with anonymous user returns 500 #1570

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
pcrespov opened this issue Jun 19, 2020 · 3 comments · Fixed by #1572
Closed

Access to study with anonymous user returns 500 #1570

pcrespov opened this issue Jun 19, 2020 · 3 comments · Fixed by #1572
Assignees
@pcrespov
Labels
bug buggy, it does not work as expected High Priority a totally crucial bug/feature to be fixed asap
Milestone
Huo Guo

Comments

@pcrespov
Copy link
Member

opening a study from portal returns sometimes server error (500)

Steps to reproduce

  • client clicks a link from portal
  • link request creating a study as anonymous user
  • server creates guest user, copies template and returns redirection with fragment and an encrypted cookie with user credentials
  • delete guest user (e.g. emulate garbage collection cleans up after user does not interact)
  • client click same study from portal (now request has cookie generated previously)
  • server responds with a 500 (Unable to start user session)
@pcrespov pcrespov added bug buggy, it does not work as expected High Priority a totally crucial bug/feature to be fixed asap labels Jun 19, 2020
@pcrespov pcrespov added this to the Huo Guo milestone Jun 19, 2020
@pcrespov pcrespov self-assigned this Jun 19, 2020
@pcrespov
Copy link
Member Author

@odeimaiz could you please check description?

@pcrespov
Copy link
Member Author

Some other error with a similar setup

Content-Type: text / plain; charset = utf-8
Content-Length: 147
Date: Fri, 19 Jun 2020 13:28:43 GMT
Server: Python / 3.6 aiohttp / 3.6.2
500: duplicate key value violates unique constraint "projects_uuid_key"
DETAIL: Key (uuid) = (5a625333-2693-59c2-9d5c-d1024156a56d) already exists.

@pcrespov pcrespov mentioned this issue Jun 20, 2020
Merged
4 tasks
pcrespov added a commit that referenced this issue Jun 22, 2020
@pcrespov
Is1570/study fails 500 (#1572)
1b7a47e 
Fixes #1570

* increases verbose in gc errors

* Turned gather errors into warnings

* Fixing issue when valid token with invalid user

* Authz cache cleared when any user is deleted

* Tests fix emulating a valid cookie invalid user

* Enhances error handling
pcrespov added a commit to pcrespov/osparc-simcore that referenced this issue Jun 22, 2020
@pcrespov
Hot fix for ITISFoundation#1570 in staging
6694641 
* increases verbose in gc errors
* Turned gather errors into warnings
* Fixing issue when valid token with invalid user
* Authz cache cleared when any user is deleted
* Tests fix emulating a valid cookie invalid user
* Enhances error handling
@pcrespov pcrespov mentioned this issue Jun 22, 2020
Merged
5 tasks
pcrespov added a commit that referenced this issue Jun 22, 2020
@pcrespov
Hot fix for #1570 in staging (#1577)
febb026 
* Hot fix for #1570 in staging

* increases verbose in gc errors
* Turned gather errors into warnings
* Fixing issue when valid token with invalid user
* Authz cache cleared when any user is deleted
* Tests fix emulating a valid cookie invalid user
* Enhances error handling

* Adapts testing to staging version

* Fixes tests
@pcrespov
Copy link
Member Author

in production

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcrespov pcrespov

Labels
bug buggy, it does not work as expected High Priority a totally crucial bug/feature to be fixed asap
Projects
None yet
Milestone
Huo Guo
Development

Successfully merging a pull request may close this issue.

Is1570/study fails 500 pcrespov/osparc-simcore
1 participant
@pcrespov