apreq does the right thing now by both throwing an exception (indicating a malformed cookie) and parsing the header correctly. This is a very old subject: Thomas should be using eval like so:
$apreq = APR::Request::Apache2->handle($r); my $jar = eval { $apreq->jar }; $jar = $@->jar if $@; ... IOW not a bug, please don't fix. ----- Original Message ----- > From: Issac Goldstand <mar...@beamartyr.net> > To: j...@apache.org > Cc: Thomas Busch <tbu...@cpan.org>; apreq-dev@httpd.apache.org > Sent: Thursday, June 21, 2012 2:19 PM > Subject: Empty cookies [Was Re: libapreq2 co-maintainer] > > On 20/06/2012 14:19, Thomas Busch wrote: >> On 20/06/2012 13:35, Thomas Busch wrote: >>> The reason I'm asking is that the following bug >>>> >>>> https://rt.cpan.org/Public/Bug/Display.html?id=69866 >>>> >>>> hasn't been solved and is still causing Internal Server > Error's on >>>> a lot of mod_perl installation. >>>> >>> I see you've got a suggested patch there, so there's a good > chance that >>> someone will pick that up - if noone does it by the weekend, I'll > try to >>> pick it up myself. >>> > > Joe, you seemingly addressed this in r748782 and then intentionally put > it back in r748790 - you touched the exact test case which would deal > with the a cookie that looked like "Cookie: foo=bar;foo2=test;blabla". > > Any recollections of why you put it back? > > Issac >