[cxf] branch 3.3.x-fixes updated: Incompatibility with recent Brave versions causes NullPointerException and hides root cause (#639)

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[cxf] branch 3.3.x-fixes updated: Incompatibility with recent Brave versions causes NullPointerException and hides root cause (#639)

reta
This is an automated email from the ASF dual-hosted git repository.

reta pushed a commit to branch 3.3.x-fixes
in repository https://gitbox.apache.org/repos/asf/cxf.git


The following commit(s) were added to refs/heads/3.3.x-fixes by this push:
     new 431bf66  Incompatibility with recent Brave versions causes NullPointerException and hides root cause (#639)
431bf66 is described below

commit 431bf666e8810c87ab5f47b0c137d329d381b1c6
Author: Tim Peeters <[hidden email]>
AuthorDate: Tue Feb 11 01:07:18 2020 +0100

    Incompatibility with recent Brave versions causes NullPointerException and hides root cause (#639)
   
    * Bump to latest Brave and Zipkin versions
   
    * Fix NullPointerException in case of SOAP faults
---
 distribution/src/main/release/samples/jax_rs/tracing_brave/pom.xml  | 2 +-
 .../org/apache/cxf/tracing/brave/AbstractBraveClientProvider.java   | 5 ++++-
 parent/pom.xml                                                      | 6 +++---
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/distribution/src/main/release/samples/jax_rs/tracing_brave/pom.xml b/distribution/src/main/release/samples/jax_rs/tracing_brave/pom.xml
index 6036993..5a81303 100644
--- a/distribution/src/main/release/samples/jax_rs/tracing_brave/pom.xml
+++ b/distribution/src/main/release/samples/jax_rs/tracing_brave/pom.xml
@@ -30,7 +30,7 @@
     </parent>
     <properties>
         <cxf.version>${project.version}</cxf.version>
-        <cxf.brave.version>5.4.2</cxf.brave.version>
+        <cxf.brave.version>5.9.4</cxf.brave.version>
     </properties>
     <profiles>
         <profile>
diff --git a/integration/tracing/tracing-brave/src/main/java/org/apache/cxf/tracing/brave/AbstractBraveClientProvider.java b/integration/tracing/tracing-brave/src/main/java/org/apache/cxf/tracing/brave/AbstractBraveClientProvider.java
index 2f4d7ed..4071d0c 100644
--- a/integration/tracing/tracing-brave/src/main/java/org/apache/cxf/tracing/brave/AbstractBraveClientProvider.java
+++ b/integration/tracing/tracing-brave/src/main/java/org/apache/cxf/tracing/brave/AbstractBraveClientProvider.java
@@ -28,6 +28,8 @@ import brave.Span;
 import brave.Tracer.SpanInScope;
 import brave.http.HttpClientAdapter;
 import brave.http.HttpClientHandler;
+import brave.http.HttpClientRequest;
+import brave.http.HttpClientResponse;
 import brave.http.HttpTracing;
 import brave.propagation.Propagation.Setter;
 import org.apache.cxf.common.logging.LogUtils;
@@ -123,7 +125,8 @@ public abstract class AbstractBraveClientProvider extends AbstractTracingProvide
                     brave.tracing().tracer().joinSpan(scope.getSpan().context());
                 }
     
-                final HttpClientHandler<?, Response> handler = HttpClientHandler.create(brave, null);
+                final HttpClientHandler<HttpClientRequest, HttpClientResponse> handler =
+                        HttpClientHandler.create(brave);
                 handler.handleReceive(null, ex, scope.getSpan());
             } finally {
                 scope.close();
diff --git a/parent/pom.xml b/parent/pom.xml
index bb239f9..5080a93 100644
--- a/parent/pom.xml
+++ b/parent/pom.xml
@@ -88,9 +88,9 @@
         <cxf.atmosphere.version.range>[2.4,3.0)</cxf.atmosphere.version.range>
         <cxf.atmosphere.version>2.5.2</cxf.atmosphere.version>
         <cxf.bcprov.version>1.64</cxf.bcprov.version>
-        <cxf.brave.reporter.version>2.10.0</cxf.brave.reporter.version>
-        <cxf.brave.version>5.6.9</cxf.brave.version>
-        <cxf.brave.zipkin.version>2.14.2</cxf.brave.zipkin.version>
+        <cxf.brave.reporter.version>2.12.1</cxf.brave.reporter.version>
+        <cxf.brave.version>5.9.4</cxf.brave.version>
+        <cxf.brave.zipkin.version>2.19.3</cxf.brave.zipkin.version>
         <cxf.cda.api.osgi.range>[1.1,2)</cxf.cda.api.osgi.range>
         <cxf.cdi.api.version>1.2</cxf.cdi.api.version>
         <cxf.classgraph.version>4.6.32</cxf.classgraph.version>