Open lucasoares opened 3 weeks ago
Thanks @lucasoares. From the perspective of SC OpenFeign, this is not a bug, however, this is something that we should implement for this release. I'd make it configurable with a property, and with the next major, we could make it default.
Thanks @lucasoares. From the perspective of SC OpenFeign, this is not a bug, however, this is something that we should implement for this release. I'd make it configurable with a property, and with the next major, we could make it default.
From the perspective of the SPRING Cloud OpenFeign it started being a bug in the same moment the spring boot GA release changed its default behavior on how to deal with trailing slash...
But yes, it should be a configuration available since Spring Boot itself has its own way to configure if the trailing slash will be accepted.
Thanks.
Describe the bug
Spring is deprecating the trailing slash automatic support: https://github.com/spring-projects/spring-framework/issues/28552
In my tests I notice all my servers using feign clients started to throw a not found error after the update.
Will spring cloud feign follow spring definitions and stop adding trailing slash? Or at least make it configurable? I'm reporting it as a bug since it is a breaking change in the spring versions.
Sample
When I use the
#getTest
method andmy.api.test
points to, for example,http://localhost:8080
, the generated URL will behttp://localhost:8080/foo/
instead ofhttp://localhost:8080/foo
, which will throw an error if requesting a server with the latest spring version.I'm using spring boot 3.3.3 and 4.1.3 starter-openfeign.