ondemand icon indicating copy to clipboard operation
ondemand copied to clipboard

batch_connect_sessions and authentication timeout

Open mattmix opened this issue 8 months ago • 1 comments

When authentication times out for a user, requests are redirected to the auth server so they can login again. However, if they are sitting on the /pun/sys/dashboard/batch_connect/sessions page, the background polling for session updates will hit the redirect and fail in a way that gives no indication to the user that they aren't getting updates without pulling up the browser's console. This also causes noise on the authentication server since the polling function will follow the redirect on each poll and, at least on ours, fail also due to a CORS error.

I think it would be better if a failure to retrieve session information caused the whole page to be reloaded. That way, if the user's session has timed out they will be properly redirected to the login page.

The code that does the session polling was changed since 3.1's release, but I think the result is still the same regarding this issue: https://github.com/OSC/ondemand/commit/fdf9920d0ab733ff054b764ccd8c2ef66ed5a439

mattmix avatar Jun 12 '24 02:06 mattmix