Bug #68096 for AnyEvent-HTTPD: URI object returned by AnyEvent::HTTPD::Request has query set to undef

Tue May 10 16:40:29 2011 ASACKETT [...] cpan.org - Ticket created

Subject:

URI object returned by AnyEvent::HTTPD::Request has query set to undef

At line 36 of AnyEvent::HTTPD::Request (v0.92) URI's query() is set to undef, but this is unexpected because it's not documented anywhere (that I've found). Whether it's best to document this or change it I don't know. It'd work best for me if it were changed, but I'm working around it for the time being by reaching into $req->{url} to get the URL with query string so it's not going to ruin my day if it's just documented and left as-is. Thanks for putting AnyEvent::HTTPD into CPAN. It's making my life a lot easier!

Wed May 11 02:41:06 2011 elmex [...] ta-sa.org - Correspondence added

Subject:	Re: [rt.cpan.org #68096] URI object returned by AnyEvent::HTTPD::Request has query set to undef
Date:	Wed, 11 May 2011 08:40:49 +0200
To:	Art Sackett via RT <bug-AnyEvent-HTTPD [...] rt.cpan.org>
From:	Robin Redeker <elmex [...] ta-sa.org>

Hi! On Tue, May 10, 2011 at 04:40:30PM -0400, Art Sackett via RT wrote: Show quoted text

> Tue May 10 16:40:29 2011: Request 68096 was acted upon. > Transaction: Ticket created by ASACKETT > Queue: AnyEvent-HTTPD > Subject: URI object returned by AnyEvent::HTTPD::Request has query set to > undef > Broken in: 0.92 > Severity: Normal > Owner: Nobody > Requestors: ASACKETT@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=68096 > > > > At line 36 of AnyEvent::HTTPD::Request (v0.92) URI's query() is set to > undef, but this is unexpected because it's not documented anywhere (that > I've found). Whether it's best to document this or change it I don't > know. It'd work best for me if it were changed, but I'm working around > it for the time being by reaching into $req->{url} to get the URL with > query string so it's not going to ruin my day if it's just documented > and left as-is.

I removed the ->query (undef). As my test suite still runs through I assume it's an old artifact from an early (less general) Version of the Module. Will be fixed in the next release :-) Show quoted text

> Thanks for putting AnyEvent::HTTPD into CPAN. It's making my life a lot > easier!

Thanks a lot for the feedback! Greetings, Robin -- Robin Redeker | Deliantra, the free code+content MORPG elmex@ta-sa.org / r.redeker@gmail.com | http://www.deliantra.net http://www.ta-sa.org/ |

Wed May 11 02:41:07 2011 The RT System itself - Status changed from 'new' to 'open'

Wed May 11 03:40:40 2011 ASACKETT [...] cpan.org - Correspondence added

On Wed May 11 02:41:06 2011, ELMEX wrote: Show quoted text

> I removed the ->query (undef). As my test suite still runs through I > assume it's an old artifact from an early (less general) Version of > the Module.

I'll do the same in my local copy, and if I run into any "anomalous operation" I'll let you know. Hopefully with a patch to bring things back to an as-expected state. If there's any work-needed on this module (suite) please let me know. I don't have unlimited time, but once my current project is deployed I intend to spend some time knocking around on things that I find interesting and this is one one of those things. I really like this module. Thanks again. Be well!

Thu Aug 04 04:48:13 2011 elmex [...] ta-sa.org - Correspondence added

Fixed in version 0.93, which is on it's way to CPAN. Thanks & Greetings, Robin

Thu Aug 04 04:48:13 2011 elmex [...] ta-sa.org - Status changed from 'open' to 'resolved'