-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add warning about SESSIONSTORE #264
Comments
I agree that it may be dangerous and I really wonder why it defaults to not storing any information, because this probably really breaks user expectations. I see it as appropriate for #50 and otherwise it should probably not be changed. On the other hand I do not understand why it should break logins. It may break login pages after a sessions restore (missing CSRF fields or similar things), but I do not see why it should be a problem in normal use. And do you have a suggestion how to phrase a warning? |
This may have a different default in a "paranoid" profile, see #50.
Maybe in conjunction with other odd settings.
Currently, it states:
We could add:
|
In general I'd like to explain in easy terms what a setting does and why it can break things, so people can understand the trade-off. Currently we have this report that it broke the login, but not details why. So the safe option for the user would be to turn it off as there is no information what may be broken. Can you go into detail with "does not remember user"? Is the username missing from the suggestions in the login box? Does it break logged in sessions? Can you login afterward correctly? |
Site: [https://gitlab.gnome.org/], on a fresh Profile,
I am not an expert, but if you think it could be a FF bug, I will open an issue on their bug tracker! |
Just to be sure: Did you check "Remember me" on the login page? |
If you did not check Remember me, I suppose that the sessionstore restores your session-cookie and without session-store (or when ending the session) the session cookie is deleted. "Remember me" should probably create a cookie with expiry date in the future that is independent from the session. |
Yes, I flag it |
I don't think gitlab uses more than cookies to store the login (session-id). So the store seems to interfere with keeping cookies in some way. If it deletes non-session cookies that are not expired yet I would consider it to be a Firefox bug as they should be stored independent from the session store. |
Setting recommended:
browser.sessionstore.privacy_level = 2
It has the drawback of break some site login
i.e. gitlab, that's [https://gitlab.gnome.org/GNOME/gnome-control-center/] does not remember user when close FF
So, we must warn the users of this chance!
The text was updated successfully, but these errors were encountered: