mozhttpd seems to have a problem with %252F -> %2f

RESOLVED FIXED

Status

Testing
Mozbase
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jmaher, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mozbase][good first bug][mentor=jhammel])

Attachments

(1 attachment, 1 obsolete attachment)

in tp5, there is a link to:
http://localhost/page_load_test/tp5/linkedin.com/www.linkedin.com/in/christopherblizzard@goback=.nppvan_%252Flemuelf.html

in literal translation with mozhttpd, we translate this to:
/page_load_test/tp5/linkedin.com/www.linkedin.com/in/christopherblizzard@goback=.nppvan_%252Flemuelf.html

on the file system, we have this file:
page_load_test/tp5/linkedin.com/www.linkedin.com/in/christopherblizzard@goback=.nppvan_%2Flemuelf.html


So when using apache we translate %252F -> %2F on the file system and %2f doesn't work.  Mozhttpd should support this mode.

Comment 1

7 years ago
In theory should just be able to url decode it.  Its becoming clearer that SimpleHTTPServer is just that
OS: Linux → All
Hardware: x86 → All
Whiteboard: [mozbase][good first bug][mentor=jhammel]
Created attachment 574922 [details] [diff] [review]
Fix for urldecoding and path splitting
Attachment #574922 - Flags: review?(jhammel)
Created attachment 574925 [details] [diff] [review]
Fix for urldecoding and path splitting

I forgot to remove the "print" statement used for debugging. Attaching the fixed patch, sorry.
Attachment #574922 - Attachment is obsolete: true
Attachment #574922 - Flags: review?(jhammel)
Attachment #574925 - Flags: review?(jhammel)

Comment 4

7 years ago
Comment on attachment 574925 [details] [diff] [review]
Fix for urldecoding and path splitting

Looks good!
Attachment #574925 - Flags: review?(jhammel) → review+

Comment 5

7 years ago
pushed: http://hg.mozilla.org/build/talos/rev/b76d25c978df

Thanks for the patch!
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.