clowder icon indicating copy to clipboard operation
clowder copied to clipboard
verified publisher icon RedHatInsights

fix: Add support for multiple APIPaths in caddygateway

Open ezr-ondrej opened this issue 1 year ago • 5 comments

ezr-ondrej avatar Jan 27 '25 22:01 ezr-ondrej

Good fix - is it possible to get a test in here too?

psav avatar Jan 28 '25 13:01 psav

I've refactored the method a bit to separate the logic for preparation of the paths. That allowed me to cover it by unit test. It feels good enough to me, WDYT @psav ?

ezr-ondrej avatar Mar 19 '25 12:03 ezr-ondrej

@wcmitchell a good one to review ;)

psav avatar Mar 21 '25 17:03 psav

/retest

gburges avatar May 16 '25 16:05 gburges

@ezr-ondrej I'm seeing a Kuttl test failure for test-multiple-app-endpoints. Looks like this needs to be looked at again? Happy to review once this has passed.

maknop avatar Aug 12 '25 17:08 maknop