was there something changed with (un)escaping of i...
# http4k
v
was there something changed with (un)escaping of input params in recent versions? consider
field=foo|bar
as input to
Path.string().of("field")
- browsers normally escape
foo|bar
to
foo%7Cbar
which in 249 always arrived as
foo|bar
to the controller. Now it arrives as
foo%7Cbar
. I can prepare a demo if it helps.
d
v
yeah that looks like it
I was not sure if this is a bug or feature 🙂
d
If we call it a feature, does that mean we don't need to fix it? 😂
v
yeah usually that is the case 🙂
e
I have come across issues with http4k path encoding recently and I have been trying to fix it so that it reflects the spec more accurately. I have submitted a PR for https://github.com/http4k/http4k/issues/602 https://github.com/http4k/http4k/pull/690
🙌 1