search

Automattic / knox

S3 Lib
MIT License
1.74k stars 285 forks source link

result.req.url for each specific simbole like space, (, ), etc... make double url encoding #288

Open tapin13 opened 8 years ago

tapin13 commented 8 years ago

If I upload file with file name "Tux (copy).svg.72e06f3cdf16d9481d9bc1c2eb92f1d5.png", I got URL: https://binary-options.s3.amazonaws.com/pf/Tux%2520%2528copy%2529.svg.72e06f3cdf16d9481d9bc1c2eb92f1d5.png

%2520 - is incorrect, have to be %20

jamesearl commented 8 years ago

Haven't run it through the test suite yet, but that fork ^^ has a fix that works for me.