Skip to content
This repository has been archived by the owner on Sep 1, 2020. It is now read-only.

SameSite=none on session cookie #226

Open
travisspencer opened this issue May 11, 2020 · 6 comments
Open

SameSite=none on session cookie #226

travisspencer opened this issue May 11, 2020 · 6 comments

Comments

@travisspencer
Copy link

The test tools session cookie does not have a SameSite attribute. We foresee this causing problems in some of our tests where we vary the user agent, version, and domains. We haven't seen any issues yet, but wanted to raise it given all the issues we've been seeing lately with this new cookie flag. Our suggestion is to add SameSite=None on its session cookie.

@rohe
Copy link

rohe commented May 12, 2020

Hmm, I'm not sure we can do this. Depends on whether CherryPy which is the framework we use for the test tool supports it. I'll try to find out.

@rohe
Copy link

rohe commented May 12, 2020

Actually, we've locked the test framework to an older CherryPy version which don't have any support for SameSite. Since the Python framework is soon to be decommissioned I can't see us spending the time necessary to move to the latest version (Still not sure the latest version supports SameSite).

@travisspencer
Copy link
Author

In the end, it's just an HTTP response header, right? Loop over and add SameSite=None if the header name is Set-Cookie and the value starts with sessionid. Seems easy. Point me to where, and I'll send a PR.

@rohe
Copy link

rohe commented May 12, 2020

Right now I/we leave it to the CherryPy software to issue session cookies. Which means it all happens in the background. Out of sight for me.

@travisspencer
Copy link
Author

Ah, but in CherryPy, you can add a post-request filter, right? There this kinda header post-processing could occur. I worked with CherryPy years ago, and then only doing pre-request handling for authentication. It's a very flexible framework though, so I'd be surprised if there wasn't some kinda post-processing hook as well. Do you know if there is? Could that be used do you think?

@rohe
Copy link

rohe commented May 12, 2020

I'ver never worked with CherryPy on that level.
Well, like you I've done pre-request handling.
Don't know about post-processing but it's worth looking into.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants