Skip Menu |
 

This queue is for tickets about the AnyEvent-HTTPD CPAN distribution.

Report information
The Basics
Id: 68096
Status: resolved
Priority: 0/
Queue: AnyEvent-HTTPD

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

Bug Information
Severity: Normal
Broken in: 0.92
Fixed in: (no value)



Subject: URI object returned by AnyEvent::HTTPD::Request has query set to undef
Download (untitled) / with headers
text/plain 521b
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!
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>
Download (untitled) / with headers
text/plain 1.4k
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/ |
Download (untitled) / with headers
text/plain 672b
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!
Fixed in version 0.93, which is on it's way to CPAN. Thanks & Greetings, Robin


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.