rewrite url + forward to new host does not apparently work

[geirih]

I need to match a rule on pattern, rewrite the uri, and forward the resulting uri to a new host.

The following route results in a request to "/api/test/1234" to be forwarded to "http://localhost:8090/", not
"http://localhost:8090/newpath/1234" as expected.
return builder.routes() .route("test_route", r -> r.path("/api/test/**") .add(rewritePath("/api/test/","/newpath/")) .uri("http://localhost:8090") )
Is this a bug, or is the rule incorrect ? Please advice

[spencergibb]

Rewrite path uses regular expressions. It won't work how you have defined it.

[geirih]

What do you mean ? at the core in RewritePathGatewayFilterFactory the code line

String newPath = path.replaceAll(regex, replacement);

performs the correct replacement, and the String newpath is set to "/newpath/1234", but the resulting newpath is ignored further down the chain, and the result consists only on the parameter to .uri.

[spencergibb]

Ok. Can you provide a sample that recreates the problem? There are integration tests for this that pass.

[geirih]

It might be a user problem... There is a subtle difference with / at the end of the .uri parameter. With a /, the forward is broken, without - it works.

@Configuration
@SpringBootApplication
public class Application {

  @Bean
  public RouteLocator customRouteLocator(RouteLocatorBuilder builder) {
    return builder.routes()
        .route("test_route",
            r -> r.path("/api/test/**")
                .add(rewritePath("/api/test/","/newpath/"))
//                .uri("http://httpbin.org:80/")) // this line makes the result /
                .uri("http://httpbin.org:80")) // This line makes the result /newpath/xxxx
        .build();
  }

  public static void main(String[] args) {
    SpringApplication.run(Application.class, args);
  }
}

The code is tested with the M5 build

[spencergibb]

Please learn how to properly format code and logs.

[geirih]

ok. thanks.

[spencergibb]

/ isn't a subtle difference, it in and of itself is a path.

[spencergibb]

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.