[dancer-users] Erroneous session-cookie setting behaviour in Dancer2
David Golden
xdg at xdg.me
Mon Mar 4 00:00:48 GMT 2013
There's no point putting in an expired cookie.
Expired cookies should only be used to signal cookie deletion when a
session is destroyed or no longer valid.
The right answer is fixing the session keyword (and possibly the session
accessor in Context) so that checking a session value when a session
doesn't exist returns undef without creating a new session.
It's a trivial patch. Just no one has written it yet.
return unless context->has_session
How about less complaining and more contributing?
David
On Mar 3, 2013 6:01 PM, "Rik Brown" <rik at rikbrown.co.uk> wrote:
> That sounds like it's working correctly. You got a new empty session and
> a cookie for it. I don't think it's expected that you won't get a cookie
> if your session is empty.
>
>
>
> So Rik, what's the point of getting a new empty session and cookie for
> it, instead of just a "negative" cookie (a set-cookie header with
> negative time)
>
> Yeah you're right. It is probably better to put a "negative" cookie there
> if the session is non-existent/empty. I think I was going with a "that
> sounds like more code/more complexity than it's worth", but the privacy
> concerns (I've never cared about what cookies sites are throwing at me, for
> better or for worse) are fair enough I suppose.
>
> and no session at all to waste space on the db?
>
> A smart session engine should probably not bother put any entries in the
> DB if the session is empty, but I take your point.
>
> --
> Rik Brown
> http://www.rikbrown.co.uk
>
> On Sunday, 3 March 2013 at 22:47, Punter wrote:
>
>
>
> On 03/03/2013 05:55 PM, Rik Brown wrote:
>
> That sounds like it's working correctly. You got a new empty session and
> a cookie for it. I don't think it's expected that you won't get a cookie
> if your session is empty.
>
>
>
> So Rik, what's the point of getting a new empty session and cookie for
> it, instead of just a "negative" cookie (a set-cookie header with
> negative time) and no session at all to waste space on the db?
> _______________________________________________
> dancer-users mailing list
> dancer-users at dancer.pm
> http://lists.preshweb.co.uk/mailman/listinfo/dancer-users
>
>
>
> _______________________________________________
> dancer-users mailing list
> dancer-users at dancer.pm
> http://lists.preshweb.co.uk/mailman/listinfo/dancer-users
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.preshweb.co.uk/pipermail/dancer-users/attachments/20130303/f6c1d7f8/attachment.htm>
More information about the dancer-users
mailing list