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

Is this an expected behavior: Logged in on hitting refresh #152

Open
sitapriyamoorthi opened this issue Oct 21, 2024 · 6 comments
Open

Is this an expected behavior: Logged in on hitting refresh #152

sitapriyamoorthi opened this issue Oct 21, 2024 · 6 comments
Labels
investigating Still assessing priority, ignore for now

Comments

@sitapriyamoorthi
Copy link
Collaborator

Hi,
Been doing a lot of PROOFing... just wanted to make sure that this is an expected behavior:
When I refresh my PROOF page I am still logged in...
I am just thinking from a security perspective thats all!
Please ignore if this is an expected behavior.

@sckott
Copy link
Collaborator

sckott commented Oct 22, 2024

yes, that's expected behavior. we changed to this behavior maybe a few months ago to solve a number of problems

  • sometimes we'd have to reload the app to make sure it knows the server is up after starting it - and before you'd get logged out if you did that
  • you can close the tab with the app and reopen and not have to log back in

from a security perspective you can't get to the app unless you're on campus or on a vpn - but i guess you could leave your computer in a public place someone could get it

any thoughts sita?

@sitapriyamoorthi
Copy link
Collaborator Author

Totally makes sense why those changes were made. But it would perhaps be worthwhile to check with Jenny and Ty to see if this actually works from a Info-Sec perspective!

@sckott
Copy link
Collaborator

sckott commented Oct 22, 2024

yeah I can ask them

@sckott
Copy link
Collaborator

sckott commented Oct 22, 2024

ok, pinged them in slack

@tefirman tefirman added the investigating Still assessing priority, ignore for now label Oct 28, 2024
@sitapriyamoorthi
Copy link
Collaborator Author

just adding a comment here that the log-in persists even now...

@sckott
Copy link
Collaborator

sckott commented Nov 4, 2024

Jennifer and Ty got back to me about this - they asked InfoSec and they suggested 12 hrs. That seems too short, but maybe 24 hrs would be good - that's what the Cisco VPN does I'm pretty sure, which seems reasonable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating Still assessing priority, ignore for now
Projects
None yet
Development

No branches or pull requests

3 participants