Skip Menu |

This queue is for tickets about the Catalyst-Plugin-Session-State-Cookie CPAN distribution.

Report information
The Basics
Id: 70229
Status: rejected
Priority: 0/
Queue: Catalyst-Plugin-Session-State-Cookie

People
Owner: Nobody in particular
Requestors: MAROS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.17
Fixed in: (no value)



Subject: cookie_path option does not work
Attached patch solves the problem for me and adds plack support Cheers Maros
Subject: cookie_path.diff
54c54,62 < return 1 if index '/'.$c->request->path, $cookie->{path}; --- > my ($request,$basepath); > $request = $c->request; > $basepath = $request->{_psgi_env}{SCRIPT_NAME} > if exists $request->{_psgi_env}; > my $path = join ('/',grep { defined $_ } ($basepath,$request->path)); > > return 0 if index '/'.$path, $cookie->{path}; > > return 1;
By 'adds plack support', you mean "breaks things unless you're using the very soon to be deprecated PSGI engine", right? Can we start with a failing test case for the problem we can then fix in a generic way instead?