Skip to content

Grizzly provider aggressively set cookie missing domain and path to / #631

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

Closed
slandelle opened this issue Jul 17, 2014 · 5 comments
Closed

Comments

@slandelle
Copy link
Contributor

slandelle pushed a commit that referenced this issue Jul 17, 2014
@oleksiys
Copy link
Contributor

Hey,
are you sure it's a Grizzly issue?
I might be wrong, but when I check cookie.getDomain() and cookie.getPath() they do have "/" values. On other hand I see Netty provider just ignoring these values. Can you pls. clarify?

Thank you!

@slandelle
Copy link
Contributor Author

A user agent should only send the cookie name and value to the server. Other informations such as domain and path only belongs to cookies that go from the server to the user agent, so it can figure out when to send them back.

@oleksiys
Copy link
Contributor

oh, ok so we're
http://tools.ietf.org/html/rfc6265
compatible only. Got it!
Thanks!

oleksiys pushed a commit that referenced this issue Jul 23, 2014
#631
"Grizzly provider aggressively set cookie missing domain and path to /"
oleksiys pushed a commit that referenced this issue Jul 23, 2014
#631
"Grizzly provider aggressively set cookie missing domain and path to /"
oleksiys pushed a commit that referenced this issue Jul 23, 2014
#631
"Grizzly provider aggressively set cookie missing domain and path to /"
@oleksiys
Copy link
Contributor

fixed

@slandelle
Copy link
Contributor Author

oh, ok so we're http://tools.ietf.org/html/rfc6265 compatible only

Yeah, that's how I should have stated it, it would have been clearer :)

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

No branches or pull requests

2 participants