Version 1.1.2 breaks URL encoding

[brstgt]

Hi Nick!

Version 1.1.2 does not encode curly braces any more. IMHO this is not a good idea.
NSURL does not accept these URLs and returns nil, which in my case breaks my app.

Thanks for your help and of course for your library :)

[nicklockwood]

Oh, sorry about that. The docs on how to use the new encoding methods are not very clear. Please put up a PR to fix it and I'll merge it.

[brstgt]

I am very sorry but I am very busy right now and do not have the time to do
that right now.
I rollbacked to 1.1.1 - everything works fine again.
All I can tell you from my experience:
Do not only escape the "unsafe chars" listed in your code. A query
parameter should escape ANYTHING but alphanumeric chars.
See for example here http://php.net/manual/en/function.rawurlencode.php or
here http://www.w3schools.com/jsref/jsref_encodeuricomponent.asp

2016-02-11 16:47 GMT+01:00 Nick Lockwood notifications@github.com:

Oh, sorry about that. The docs on how to use the new encoding methods are
not very clear. Please put up a PR to fix it and I'll merge it.

—
Reply to this email directly or view it on GitHub
#23 (comment)
.