Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

403 executing a mutation #612

Closed
kinow opened this issue Mar 15, 2021 · 3 comments
Closed

403 executing a mutation #612

kinow opened this issue Mar 15, 2021 · 3 comments
Labels
bug Something isn't working
Milestone

Comments

@kinow
Copy link
Member

kinow commented Mar 15, 2021

Describe the bug

A strange bug. I noticed this morning that after leaving the workflow running for a very long time in the browser, it gave me an error executing a mutation.

So went for lunch and left a workflow running. Came back and waited a bit more. It (five) was probably left unattended for 1-2 hours. I left the browser window in another Linux desktop running, with UIS and Flow. No errors in the the log. Then clicked to pause it, but got an error 403?

Screenshot from 2021-03-15 14-04-45

Release version(s) and/or repository branch(es) affected?

master flow/uis/and UI

Steps to reproduce the bug

Leave JupyterHub + UIS running for a long time, making sure the UIS has been spawned (just open the browser, for instance).

Try clicking on the pause/hold mutation icon.

Expected behavior

The mutation is executed.

Screenshots

See description above.

Additional context

I think it could be due to inactivity in the Hub. There could be a setting somewhere that we can add to keep the connection alive, or send some ping to the backend telling that the session is still active.

I suspect some users of long-running workflows could want to trigger mutations after the UI has been running for a long time, even if they haven't interacted with the browser.

Note that the session is refreshed automatically when you reload the browser, as the cookie is still valid, but I think the token is not.

2021-03-15 14:04:24,656 tornado.access WARNING  403 POST /user/kinow/graphql (127.0.0.1) 25.51ms
2021-03-15 14:04:49,463 tornado.application WARNING  Token stored in cookie may have expired

Pull requests welcome!
This is an Open Source project - please consider contributing a bug fix
yourself (please read CONTRIBUTING.md before starting any work though).

@kinow kinow added the bug Something isn't working label Mar 15, 2021
@kinow kinow added this to the 0.3 milestone Mar 15, 2021
@kinow kinow added good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed labels Mar 15, 2021
@oliver-sanders oliver-sanders modified the milestones: 0.3, 0.4 Mar 25, 2021
@kinow
Copy link
Member Author

kinow commented Mar 28, 2021

JupyterHub settings that I think could be related:

@oliver-sanders oliver-sanders modified the milestones: 0.4.0, 0.5.0 Apr 16, 2021
@oliver-sanders oliver-sanders modified the milestones: 0.5.0, 0.6.0 Jul 28, 2021
@kinow kinow modified the milestones: 0.6.0, 1.0 Aug 16, 2021
@kinow
Copy link
Member Author

kinow commented Aug 16, 2021

Note: confirm it's still an issue after #711

@kinow kinow modified the milestones: 1.0, 2.0 Sep 10, 2021
@oliver-sanders oliver-sanders modified the milestones: 2.0.0, Pending Jun 8, 2022
@MetRonnie MetRonnie removed help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest labels Nov 11, 2022
@MetRonnie
Copy link
Member

This has not been seen for a long time; closing.

@oliver-sanders oliver-sanders modified the milestones: Pending, 1.4.0 Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants