Skip Menu |
 

This queue is for tickets about the URI CPAN distribution.

Report information
The Basics
Id: 85910
Status: new
Priority: 0/
Queue: URI

People
Owner: Nobody in particular
Requestors: caine [...] duckduckgo.com
Cc:
AdminCc:

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



Subject: URI::query_form has potential to double encode
Date: Wed, 5 Jun 2013 18:58:10 -0400
To: bug-URI [...] rt.cpan.org, Dylan Lloyd <dylan [...] duckduckgo.com>
From: Caine Tighe <caine [...] duckduckgo.com>
Download (untitled) / with headers
text/plain 556b
Does URI::query_form use + instead of space on purpose? I think that most people are using URI::Encode::uri_unescape for unescaping which i don't think decodes + to space. If a package (in this case URI::Amazon::APA) calls uri_unescape after query_form run on spaces and then needs to run query_form again, it will result in %2Bs instead of +s. The easiest thing to do might be to just replace the +s in URI::_query.pm with %20, but I haven't read the RFC and don't know if there is a github -- so wanted to inquire. respectfully, caine. opensesame.st


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.