Skip Menu |

This queue is for tickets about the HTTP-DAV CPAN distribution.

Report information
The Basics
Id: 52665
Status: resolved
Worked: 1 hour (60 min)
Priority: 0/
Queue: HTTP-DAV

People
Owner: OPERA [...] cpan.org
Requestors: CEBJYRE [...] cpan.org
Cc:
AdminCc:

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



Subject: Propfind doesn't work properly with some escaped uris
Propfind is case sensitive to the escaping of characters, an example being: In dave doing an ls of 'file_%5B.txt' will not properly get the properties if the web server reports the href of the file as 'file_%5b.txt'. On the one hand URI::Escape outputs escaped chars in capitals, and on the other apache/mod_dav is outputting them lowercased. I should have a patch covering this on github soon.
I've forked and added a fix and test (http://github.com/cebjyre/perl5-http-dav)
Excellent. Thanks! I'll look at this over the weekend. -- Cosimo
I have applied your patch. Passed all tests for me. I just added one more test and changed a couple of things here and there. I was uncertain of the minimum perl version that could run \p {IsXDigit} so I replaced it with a more noob [0-9a-fA-F]. Do you know that? Anyway, thanks for the patch and especially the test case. Good stuff. v0.39 is on its way to CPAN. I'm closing this ticket as fixed but let me know how it works for you.
On Sat Dec 12 17:22:37 2009, OPERA wrote: Show quoted text
> I have applied your patch. Passed all tests for me. > I just added one more test and changed a couple of things here and > there. I was uncertain of the minimum perl version that could run \p > {IsXDigit} so I replaced it with a more noob [0-9a-fA-F]. Do you know > that? > > Anyway, thanks for the patch and especially the test case. Good stuff. > v0.39 is on its way to CPAN. > I'm closing this ticket as fixed but let me know how it works for you.
Yeah the IsXDigit approach was probably too fancy for such a simple case. I've upgraded to the current version, and it's all good.
Sorry, didn't mean to reopen it.