Extra path separator when using `RUSTUS_URL=/`
Kaelten opened this issue · 2 comments
Kaelten commented
When I use '/' for the path instead of /files things work, but the path that is generated for the files contain an extra /. Here's a log snippet that shows the issue.
rustus | [2022-06-22][01:03:13+00:00][INFO] "POST / HTTP/1.1" "-" "201" "172.19.0.1" "66.578333"
rustus | [2022-06-22][01:03:14+00:00][INFO] "PATCH //10ac0e0e-d67d-4e2c-880b-1346793008b5 HTTP/1.1" "-" "204" "172.19.0.1" "263.382625"
In these situations I've found that using a url join library often is the cleanest way to eliminate duplications and ensure urls are built properly. Failing that doing a regex replace so that /+ is replaced with / works fairly well as well.
s3rius commented
Hi, and thanks for pointing this out. I fixed URLs normalization. It was a problem caused by the logic I used for dynamic route creation.
Kaelten commented
Looks great here. Thanks again! :)