ktor icon indicating copy to clipboard operation
ktor copied to clipboard

KTOR-7048: Support parameters in Resources url inside the path

Open hfhbd opened this issue 1 year ago • 6 comments

Subsystem Client, Server, shared resources

Motivation KTOR-7048: Resources url builder does not replaces parameters inside the path

Solution Use a regex to replace the placeholder url with the actual values

hfhbd avatar May 30 '24 15:05 hfhbd

It looks like the resource tests are failing. Could you check?

e5l avatar May 31 '24 17:05 e5l

Can you rerun the CI?

hfhbd avatar Jun 06 '24 10:06 hfhbd

Sure, sorry for the delay

e5l avatar Jun 21 '24 07:06 e5l

Seems there's a test failure in ResourcesTest.testEncodesParameters() 🔴

bjhham avatar Sep 11 '24 12:09 bjhham

@hfhbd, would you like to rebase this PR and fix tests?

osipxd avatar Nov 20 '24 15:11 osipxd

@osipxd Yes, will do it, but I also noticed the failing test requires a little rework because I didn't handle the test case.

hfhbd avatar Nov 25 '24 13:11 hfhbd