search

sintaxi / dbox

NodeJS SDK for Dropbox API (THIS LIBRARY IS OBSOLETE!!)
513 stars 91 forks source link

querystring.escape not sufficient #9

Closed dfelder closed 12 years ago

dfelder commented 12 years ago

querystring.escape doesn't escape !'()*. Dropbox allows at least a few of them in filenames and requires them to be escaped.

tomgco commented 12 years ago

Yes although querystring.escape doesn't escape !'()* it can be overridden, however I seem to be able to create files using !'()* with no problems. What scenario does this fail for you?

dfelder commented 12 years ago

My bad. The oauth.js lib takes perfectly care of it.

sintaxi commented 12 years ago

thanks for following up on this fellas :)