search

laurelandwolf / sdk

JS SDK for Laurel & Wolf API
0 stars 1 forks source link

normalize paths to dedupe slashes #7

Closed scottcorgan closed 9 years ago

scottcorgan commented 9 years ago

http//asfasdfasdf.com//something//somethinelse

michaelgraham commented 9 years ago

What's the use case for this? // is a valid part of a path name, I believe. If it's guaranteed to never be a part of our JSON API paths, should it throw an error, rather than failing somewhat silently?

scottcorgan commented 9 years ago

Actually meant to remove the double slashes On Mon, Jul 6, 2015 at 5:23 PM Michael Graham notifications@github.com wrote:

What's the use case for this? // is a valid part of a path name, I believe. If it's guaranteed to never be a part of our JSON API paths, should it throw an error, rather than failing somewhat silently?

— Reply to this email directly or view it on GitHub https://github.com/laurelandwolf/sdk/issues/7#issuecomment-119033790.