Multuple cookies do not supported #7

Open
sparfenyuk opened this Issue Oct 2, 2010 · 2 comments

Comments

Projects
None yet
2 participants

If you want to set multiple cookies you will see that this is impossible. The problem is described in this post http://caolanmcmahon.com/multiple_set_cookie_headers_in_node_js.html

jwatte commented Nov 15, 2010

I see the same problem.

It seems as if it appears to set many cookies in a single Set-Cookie header, but the browser somehow doesn't like it. (I'm using Firefox 3.6 on Ubuntu)

jwatte commented Nov 15, 2010

OK, so this seems simple to fix. The cookie spec says that multiple cookies, as well as parameters, are separated by semicolons only. The cookie-node library separates different cookies with ', '. That should be a single space: " ".
Line 92 in index.js, "cookies.join()" should not take a comma.

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