Plus sign is not escaped correctly about sttp HOT 9 CLOSED

No, you are right, somehow the + is not escaped correctly. I'll take a look tomorrow.

from sttp.

Hmm I suppose we'll need context-sensitive decoding, not only encoding, wich makes sense :) The + in the query string should be converted to a (space), but not in the path

from sttp.

By query string, do you mean the named URI parameters? Actually my use case is a URI as follows:

http://localhost:8081/user?phone=+123456

The context is looking up by phone that is. Come to think about it, "+" is really used as a space in this context. However, does it make sense to still have it as a space in sttp? After all, you can perfectly use spaces in query string and they will be encoded properly.

from sttp.

If you want to include + in the query string, it should be %-encoded, as + is the escaped form of space (in that context).

The interpolator assumes that when you write down a literal URI (as a String, without embedding any expressions), it's in encoded form - hence it decodes any special characters. So:

uri"http://localhost:8081/user?phone=+123456" will be parsed as an URI with a single query parameter phone with the value 123456

uri"http://localhost:8081/user?phone=${"+123456"}" will be parsed as an URI with a single query parameter phone with the value +123456

from sttp.

Hm, then this does not seem to be working with get requests (maybe also with others). When I do:

sttp.get(uri"http://localhost:8888/appointment?phone=${"+1234"}").send()

And on the server side (http4s):

  val root = Root / "appointment"
/*...*/
    case req @ GET -> `root` :? PhoneParam(phoneRaw) =>
      println(s"Raw request parameter: $phoneRaw")

This prints:

Raw request parameter:  1234

from sttp.

Ah, come to think about it this may be http4s issue.

from sttp.

I think it's an easy fix (only the + is problematic), see the commit. I'm publishing a snapshot (for some reason it gets published as 0.0.20-SNAPSHOT), but the release will definitely have to wait until tomorrow ;)

from sttp.

Awesome, thank you for the quick fix @adamw! I will check it out tomorrow.

from sttp.

Released 1.0.6

from sttp.