https://kotlinlang.org logo
#http4k
Title
# http4k
v

Vojtěch Knyttl

12/07/2020, 7:25 PM
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

dave

12/07/2020, 7:41 PM
v

Vojtěch Knyttl

12/07/2020, 8:31 PM
yeah that looks like it
I was not sure if this is a bug or feature 🙂
d

dave

12/07/2020, 9:19 PM
If we call it a feature, does that mean we don't need to fix it? 😂
v

Vojtěch Knyttl

12/07/2020, 9:58 PM
yeah usually that is the case 🙂
e

Eduardo Narros

01/17/2022, 8:17 AM
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
5 Views