Extra path separator when using `RUSTUS_URL=/`
Kaelten opened this issue · comments
Bryan McLemore 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.
Pavel Kirilin 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.
Bryan McLemore commented
Looks great here. Thanks again! :)