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

Question about SESSION_SECURITY_INSECURE setting #126

Closed
marsvolta80 opened this issue Jul 24, 2019 · 2 comments
Closed

Question about SESSION_SECURITY_INSECURE setting #126

marsvolta80 opened this issue Jul 24, 2019 · 2 comments

Comments

@marsvolta80
Copy link

Good morning,

I just wanted to get some clarification on the SESSION_SECURITY_INSECURE setting. Based on documentation it seems like if you set SESSION_EXPIRE_AT_BROWSE_CLOSE to True then you wouldn't need to set the SESSION_SECURITY_INSECURE setting? Also the documentation seems to suggest you wouldn't want to use this app without setting SESSION_EXPIRE_AT_BROWSER_CLOSE being set to True. Overall I was just a little confused on how this setting is used. Can you describe a scenario of how you would use this setting?

Thank you!

@claytondaley
Copy link
Contributor

claytondaley commented Jul 24, 2019

The package will throw an exception if you don't do one of two things:

  1. set SESSION_EXPIRE_AT_BROWSER_CLOSE=True
  2. set SESSION_SECURITY_INSECURE=True

You absolutely should do the first, but a bypass exists if (for some reason?!?!) you needed to prevent this from happening (e.g. for testing or for a specific type of user).

@claytondaley
Copy link
Contributor

If you have a suggestion on how to phrase it more clearly, you're welcome to submit a PR!

@jpic jpic closed this as completed Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants