Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

You must be a member of this account.

Set-Cookie header incompatible with current Rack spec

#73

Using Thin + Rack::Adapter::Rails + Lint won't work if you have two cookies being set in the same response.

The problem is Rack::Adapter::Rails is joining the cookies with a newline, which is not accepted in the current Lint implementation.

Here's the original discussion in rack-devel

And here's a small patch to keep the Set-Cookie header as one array

Reported by Pedro Belo · June 30th, 2008 @ 04:36 PM

State: resolved
Milestone: 1.0
Assigned to: macournoyer macournoyer

Activity

  1. macournoyer
    macournoyer
    • State changed from new to open
    • Milestone changed from Future to 1.0

    July 1st, 2008 @ 09:47 AM

  2. macournoyer
    macournoyer

    thx for the patch Pedro, I'll apply and test when I get a minute

    July 1st, 2008 @ 09:47 AM

  3. macournoyer
    macournoyer

    hm not sure about it. Like christian said: 

    "Set-Cookie" => [cookie1.to_s, cookie2.to_s]  works in current Rack. 
In future Racks (for 1.9 support, not yet fully specified), the 
newline is the correct way.

    The newline is the correct way in 1.9, not sure if this is gonna end up the correct way for both version.

    July 1st, 2008 @ 10:51 PM

  4. macournoyer
    macournoyer
    • State changed from open to resolved

    (from [34da9ea800d4667a8a15700610ffc9fff71b9806]) Make Set-Cookie header, in Rails adapter, compatible with current Rack spec [Pedro Belo]

    [#73, state:resolved]

    http://github.com/macournoyer/th...

    July 14th, 2008 @ 12:03 AM

  5. John
    John

    Faktisk er ikke alle online casinoer fantastiske. Jeg vil varmt anbefale dig at besøge mr bet online casino for at få den bedste spiloplevelse. Du tror mig måske ikke, så i dette tilfælde kan du læse rigtige https://mrbet-casino.dk/ anmeldelser.

    May 13th, 2024 @ 10:57 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.