From 0a0d36053d7fd77247c2da9af12d81e5a465b9ba Mon Sep 17 00:00:00 2001 From: Marcin Grzejszczak Date: Fri, 27 May 2016 08:52:47 +0200 Subject: [PATCH] Cloning the Retryer @dpalita's suggested that since Default Retryer implemenation is not thread-safe we have to clone the delegate. And obviously he's right! --- .../sleuth/instrument/web/client/feign/TraceFeignRetryer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/web/client/feign/TraceFeignRetryer.java b/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/web/client/feign/TraceFeignRetryer.java index 47a4f31ac..b0f5b07a2 100644 --- a/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/web/client/feign/TraceFeignRetryer.java +++ b/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/web/client/feign/TraceFeignRetryer.java @@ -62,6 +62,6 @@ final class TraceFeignRetryer implements Retryer { @Override public Retryer clone() { - return new TraceFeignRetryer(this.tracer, this.delegate); + return new TraceFeignRetryer(this.tracer, this.delegate.clone()); } } \ No newline at end of file