Changed encode-path to use URLEncoder and replace + with %20, fix for issue #21 in neocons

michaelklishin / urly

A tiny Clojure library that parses and attempts to unify URIs, URLs and relative values found in real world HTML anchors

116 stars 10 forks source link

Changed encode-path to use URLEncoder and replace + with %20, fix for issue #21 in neocons #2

Closed tutysara closed 11 years ago

tutysara commented 11 years ago

Changed encode-path to use URLEncoder and replace + with %20, fix for issue #21 in neocons. Change passed all test cases in neocons and urly. Haven't removed the old implementation, can be removed if it doesn't affect other projects in clojurewerks that uses urly.

michaelklishin commented 11 years ago

This approach doesn't seem to be correct. URLEncoder should not be used for URI path encoding.

michaelklishin commented 11 years ago

Oh well, the current one uses URLEncoder, too. Ok, then we should merge this as a temporary band-aid.