Merge pull request #3711 from qnnn/fix-generateId

ryanjbaxter · ryanjbaxter · commit a4a4002a5cb7 · 2025-05-13T15:39:37.000-04:00
ensure generateId executes on the scheduler thread specified by publishOn
diff --git a/spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/route/CompositeRouteDefinitionLocator.java b/spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/route/CompositeRouteDefinitionLocator.java
@@ -65,7 +65,7 @@ public Flux<RouteDefinition> getRouteDefinitions() {
 	}
 
 	protected Mono<String> randomId() {
-		return Mono.fromSupplier(idGenerator::generateId).map(UUID::toString).publishOn(Schedulers.boundedElastic());
+		return Mono.fromSupplier(idGenerator::generateId).publishOn(Schedulers.boundedElastic()).map(UUID::toString);
 	}
 
 }

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public Flux<RouteDefinition> getRouteDefinitions() {`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`protected Mono<String> randomId() {`
`68`		`- return Mono.fromSupplier(idGenerator::generateId).map(UUID::toString).publishOn(Schedulers.boundedElastic());`
	`68`	`+ return Mono.fromSupplier(idGenerator::generateId).publishOn(Schedulers.boundedElastic()).map(UUID::toString);`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`}`