Results 41 comments of Alex Edwards

You're right, it does. Good spot 👍 I'm not sure what the best course of action here is. Memstore needs to remain the default store to avoid breaking backward compatibility....

@dpotapov That's a great suggestion, thank you 👍 I've made `ContextKey` an exported field in https://github.com/alexedwards/scs/commit/fb8d6f122cfb2228bb787059a8908bb27ecb4cda, which makes it possible to work around this issue by creating your own `SessionManager`...

@jackielii Sorry, I think this PR has the right general idea behind it, but I'm going to close it because it now has some merge commits that I'd prefer not...

I've been thinking about this more just now, and it's a bit of a tricky problem. At the moment, SCS commits the session changes to the DB and sets the...

The expiry time issue described in point 5 has been fixed in 481af0a28857c8d0fbb2709d9623b03e7f67848b. The uncommitted modifications check and panic described in point 4 has been implemented in 2060319a03b0788d7b4562a865c57b42061c10bc. The documentation...

@tommyo Sorry for the slow reply on this. I'm just trying to work through the backlog now. I'm at a point where I don't want to merge more store implementations...

Right now I don't think there is any further action to take, so I'm going to close down this PR. If you do create a repo, please let me know...

@guybrush Thanks for the PR. Can you describe a real-world scenario where this is required? As far as I know, apart from `Partitioned`, all the cookie settings can be configured...

@AmurgCodru I'm not sure what the problem here is. What you're describing sounds to me exactly like the correct and expected behavior for `SameSite=Strict` and `SameSite=Lax`, and not a problem...

I think it should be possible to implement by adding a special `__idleTimeout` value to sessions, in a similar way to the [`RememberMe()`](https://pkg.go.dev/github.com/alexedwards/scs/v2#SessionManager.RememberMe) method. If anyone is interested in working...