Bug #55073 for AnyEvent-HTTPD: a tiny fix for response

Sun Feb 28 21:33:05 2010 hideki.yamamura [...] gmail.com - Ticket created

Subject:	a tiny fix for response_done in HTTPConnection.pm
Date:	Mon, 1 Mar 2010 11:32:00 +0900
To:	bug-AnyEvent-HTTPD [...] rt.cpan.org
From:	Hideki Yamamura <hideki.yamamura [...] gmail.com>

Dear Robin Redeker, I'm really grateful to you for your cpan modules. I got an error msg like this: EV: error in callback (ignoring): Can't call method "on_drain" on an undefined value at /usr/lib/perl5/site_perl/5.8.8/AnyEvent/HTTPD/HTTPConnection.pm line 69. This error can reproduce with aehttpd-test.pl. run the httpd server, request "/test" with IE8, and stop immediate. I think $self->{hdl} is already deleted by on_error callback. so I made a patch to do nothing when $self->{hdl} doesn't exist. Sincerely yours, -- Hideki YAMAMURA <hideki.yamamura@gmail.com>

Message body is not shown because sender requested not to inline it.

Mon Mar 01 02:26:10 2010 elmex [...] ta-sa.org - Correspondence added

On Sun Feb 28 21:33:05 2010, hideki.yamamura@gmail.com wrote: Show quoted text

> Dear Robin Redeker, > > I'm really grateful to you for your cpan modules. > > I got an error msg like this: > EV: error in callback (ignoring): Can't call method "on_drain" on an > undefined value at > /usr/lib/perl5/site_perl/5.8.8/AnyEvent/HTTPD/HTTPConnection.pm line > 69. > > This error can reproduce with aehttpd-test.pl. > run the httpd server, request "/test" with IE8, and stop immediate. > > I think $self->{hdl} is already deleted by on_error callback. > so I made a patch to do nothing when $self->{hdl} doesn't exist. > > Sincerely yours,

Hi! Thanks for the small patch, it's indeed a small but useful fix. Applied it to my git repository, it will be fixed in the next release. Thanks & Greetings, Robin

Mon Mar 01 02:26:12 2010 The RT System itself - Status changed from 'new' to 'open'

Mon Mar 01 02:26:12 2010 elmex [...] ta-sa.org - Status changed from 'open' to 'patched'

Tue Mar 01 05:53:18 2011 elmex [...] ta-sa.org - Correspondence added

New version has been released. Thanks and Greetings, Robin

Tue Mar 01 05:53:19 2011 The RT System itself - Status changed from 'patched' to 'open'

Tue Mar 01 05:53:19 2011 elmex [...] ta-sa.org - Status changed from 'open' to 'resolved'

Bug #55073 for AnyEvent-HTTPD: a tiny fix for response_done in HTTPConnection.pm