You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by re...@apache.org on 2023/10/17 01:01:31 UTC

[cxf] branch 3.6.x-fixes updated (41b0b0793c -> 3b5ea3c9e2)

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

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


    from 41b0b0793c Recording .gitmergeinfo Changes
     new 59bbb479dd Replace deprecated OpenTelemetry semantic convention attributes
     new 575e24e5f6 Recording .gitmergeinfo Changes
     new 3b5ea3c9e2 Update Apache Tomcat to 9.0.81

The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .gitmergeinfo                                                     | 1 +
 distribution/src/main/release/samples/pom.xml                     | 2 +-
 .../opentelemetry/AbstractOpenTelemetryClientProvider.java        | 6 +++---
 .../cxf/tracing/opentelemetry/AbstractOpenTelemetryProvider.java  | 6 +++---
 parent/pom.xml                                                    | 2 +-
 .../jaxrs/tracing/opentelemetry/OpenTelemetryTracingTest.java     | 8 ++++----
 .../jaxws/tracing/opentelemetry/OpenTelemetryTracingTest.java     | 4 ++--
 7 files changed, 15 insertions(+), 14 deletions(-)


[cxf] 01/03: Replace deprecated OpenTelemetry semantic convention attributes

Posted by re...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 59bbb479dd3997fb715a057d4ff7f57df89fa57a
Author: Andriy Redko <dr...@gmail.com>
AuthorDate: Mon Oct 16 20:57:48 2023 -0400

    Replace deprecated OpenTelemetry semantic convention attributes
    
    (cherry picked from commit d17fe348a8153db12508c5efe274d262a7b4d83f)
---
 .../opentelemetry/AbstractOpenTelemetryClientProvider.java        | 6 +++---
 .../cxf/tracing/opentelemetry/AbstractOpenTelemetryProvider.java  | 6 +++---
 .../jaxrs/tracing/opentelemetry/OpenTelemetryTracingTest.java     | 8 ++++----
 .../jaxws/tracing/opentelemetry/OpenTelemetryTracingTest.java     | 4 ++--
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryClientProvider.java b/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryClientProvider.java
index d2e5717eb6..52cfd7e893 100644
--- a/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryClientProvider.java
+++ b/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryClientProvider.java
@@ -59,8 +59,8 @@ public abstract class AbstractOpenTelemetryClientProvider extends AbstractTracin
         Context parentContext = Context.current();
         Span activeSpan = tracer.spanBuilder(buildSpanDescription(uri.toString(), method))
             .setParent(parentContext).setSpanKind(SpanKind.CLIENT)
-            .setAttribute(SemanticAttributes.HTTP_METHOD, method)
-            .setAttribute(SemanticAttributes.HTTP_URL, uri.toString())
+            .setAttribute(SemanticAttributes.HTTP_REQUEST_METHOD, method)
+            .setAttribute(SemanticAttributes.URL_FULL, uri.toString())
             // TODO: Enhance with semantics from request
             .startSpan();
         Scope scope = activeSpan.makeCurrent();
@@ -100,7 +100,7 @@ public abstract class AbstractOpenTelemetryClientProvider extends AbstractTracin
                 scope = span.makeCurrent();
             }
 
-            span.setAttribute(SemanticAttributes.HTTP_STATUS_CODE.getKey(), responseStatus);
+            span.setAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE.getKey(), responseStatus);
 
             span.end();
 
diff --git a/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryProvider.java b/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryProvider.java
index d8ddda3e8d..b23d47537a 100644
--- a/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryProvider.java
+++ b/integration/tracing/tracing-opentelemetry/src/main/java/org/apache/cxf/tracing/opentelemetry/AbstractOpenTelemetryProvider.java
@@ -63,8 +63,8 @@ public abstract class AbstractOpenTelemetryProvider extends AbstractTracingProvi
 
         SpanBuilder spanBuilder = tracer.spanBuilder(buildSpanDescription(uri.getPath(), method))
             .setSpanKind(SpanKind.SERVER)
-            .setAttribute(SemanticAttributes.HTTP_METHOD, method)
-            .setAttribute(SemanticAttributes.HTTP_URL, uri.toString());
+            .setAttribute(SemanticAttributes.HTTP_REQUEST_METHOD, method)
+            .setAttribute(SemanticAttributes.URL_FULL, uri.toString());
         Span activeSpan = spanBuilder.startSpan();
         Scope scope = activeSpan.makeCurrent();
 
@@ -104,7 +104,7 @@ public abstract class AbstractOpenTelemetryProvider extends AbstractTracingProvi
                 scope = span.makeCurrent();
             }
 
-            span.setAttribute(SemanticAttributes.HTTP_STATUS_CODE, responseStatus);
+            span.setAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, responseStatus);
             span.end();
 
             scope.close();
diff --git a/systests/tracing/src/test/java/org/apache/cxf/systest/jaxrs/tracing/opentelemetry/OpenTelemetryTracingTest.java b/systests/tracing/src/test/java/org/apache/cxf/systest/jaxrs/tracing/opentelemetry/OpenTelemetryTracingTest.java
index 1a5df89044..d49604f3bf 100644
--- a/systests/tracing/src/test/java/org/apache/cxf/systest/jaxrs/tracing/opentelemetry/OpenTelemetryTracingTest.java
+++ b/systests/tracing/src/test/java/org/apache/cxf/systest/jaxrs/tracing/opentelemetry/OpenTelemetryTracingTest.java
@@ -126,7 +126,7 @@ public class OpenTelemetryTracingTest extends AbstractClientServerTestBase {
         assertThat(otelRule.getSpans().get(0).getName(), equalTo("Get Books"));
         assertThat(otelRule.getSpans().get(1).getName(), equalTo("GET /bookstore/books"));
         assertThat(otelRule.getSpans().get(1).getAttributes(),
-                   hasAttribute(SemanticAttributes.HTTP_STATUS_CODE, 200L));
+                   hasAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, 200L));
         assertThat(otelRule.getSpans().get(1).getInstrumentationScopeInfo().getName(),
             equalTo("jaxrs-server-test"));
     }
@@ -397,7 +397,7 @@ public class OpenTelemetryTracingTest extends AbstractClientServerTestBase {
         assertThat(otelRule.getSpans().toString(), otelRule.getSpans().size(), equalTo(1));
         assertThat(otelRule.getSpans().get(0).getName(), equalTo("GET /bookstore/books/exception"));
         assertThat(otelRule.getSpans().get(0).getAttributes(),
-                   hasAttribute(SemanticAttributes.HTTP_STATUS_CODE, 500L));
+                   hasAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, 500L));
     }
 
     @Test
@@ -408,7 +408,7 @@ public class OpenTelemetryTracingTest extends AbstractClientServerTestBase {
         assertThat(otelRule.getSpans().toString(), otelRule.getSpans().size(), equalTo(1));
         assertThat(otelRule.getSpans().get(0).getName(), equalTo("GET /bookstore/books/error"));
         assertThat(otelRule.getSpans().get(0).getAttributes(),
-                   hasAttribute(SemanticAttributes.HTTP_STATUS_CODE, 503L));
+                   hasAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, 503L));
     }
 
     @Test
@@ -419,7 +419,7 @@ public class OpenTelemetryTracingTest extends AbstractClientServerTestBase {
         assertThat(otelRule.getSpans().toString(), otelRule.getSpans().size(), equalTo(1));
         assertThat(otelRule.getSpans().get(0).getName(), equalTo("GET /bookstore/books/mapper"));
         assertThat(otelRule.getSpans().get(0).getAttributes(),
-                   hasAttribute(SemanticAttributes.HTTP_STATUS_CODE, 404L));
+                   hasAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, 404L));
     }
 
     private WebClient withTrace(final WebClient client) {
diff --git a/systests/tracing/src/test/java/org/apache/cxf/systest/jaxws/tracing/opentelemetry/OpenTelemetryTracingTest.java b/systests/tracing/src/test/java/org/apache/cxf/systest/jaxws/tracing/opentelemetry/OpenTelemetryTracingTest.java
index bab22a1ee1..f24d254bca 100644
--- a/systests/tracing/src/test/java/org/apache/cxf/systest/jaxws/tracing/opentelemetry/OpenTelemetryTracingTest.java
+++ b/systests/tracing/src/test/java/org/apache/cxf/systest/jaxws/tracing/opentelemetry/OpenTelemetryTracingTest.java
@@ -238,7 +238,7 @@ public class OpenTelemetryTracingTest extends AbstractClientServerTestBase {
         assertThat(otelRule.getSpans().size(), equalTo(2));
         assertThat(otelRule.getSpans().get(0).getName(), equalTo("POST /BookStore"));
         assertThat(otelRule.getSpans().get(0).getAttributes(),
-                   hasAttribute(SemanticAttributes.HTTP_STATUS_CODE, 500L));
+                   hasAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, 500L));
         assertThat(otelRule.getSpans().get(1).getName(),
                    equalTo("POST http://localhost:" + PORT + "/BookStore"));
     }
@@ -253,7 +253,7 @@ public class OpenTelemetryTracingTest extends AbstractClientServerTestBase {
         assertThat(otelRule.getSpans().size(), equalTo(1));
         assertThat(otelRule.getSpans().get(0).getName(), equalTo("POST /BookStore"));
         assertThat(otelRule.getSpans().get(0).getAttributes(),
-                   hasAttribute(SemanticAttributes.HTTP_STATUS_CODE, 202L));
+                   hasAttribute(SemanticAttributes.HTTP_RESPONSE_STATUS_CODE, 202L));
     }
 
     @Test


[cxf] 02/03: Recording .gitmergeinfo Changes

Posted by re...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 575e24e5f632f5541736b0f7f06459b4d673a3d4
Author: Andriy Redko <dr...@gmail.com>
AuthorDate: Mon Oct 16 21:00:07 2023 -0400

    Recording .gitmergeinfo Changes
---
 .gitmergeinfo | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitmergeinfo b/.gitmergeinfo
index 640d64dc46..c257fcae29 100644
--- a/.gitmergeinfo
+++ b/.gitmergeinfo
@@ -726,6 +726,7 @@ M ed102a26c22b5376ace2951d508485457f1f606b
 M edea785abd33bbc363bb3a9af926a1dbf7956cd1
 M ee165a313d3f629d3de863d758554eabbe26f8b7
 M eefbb64995c59fea459ee7e2fe324831ebcc10c3
+M ef7d98970cfd067d3d6790f2d81f56e9cda881b5
 M f00ca03f0852b8be195adbcee20c164a189e8b6e
 M f03c3b0f9dccf7ebec8e1016247091cb363344ea
 M f05bdcfeafebdbe93c6d189514e084ae1d40cac0


[cxf] 03/03: Update Apache Tomcat to 9.0.81

Posted by re...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 3b5ea3c9e256a7373912f00f52344a1cded19bd5
Author: Andriy Redko <dr...@gmail.com>
AuthorDate: Mon Oct 16 21:01:04 2023 -0400

    Update Apache Tomcat to 9.0.81
---
 distribution/src/main/release/samples/pom.xml | 2 +-
 parent/pom.xml                                | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/distribution/src/main/release/samples/pom.xml b/distribution/src/main/release/samples/pom.xml
index e01a2289fc..99cddcde5a 100644
--- a/distribution/src/main/release/samples/pom.xml
+++ b/distribution/src/main/release/samples/pom.xml
@@ -37,7 +37,7 @@
         <cxf.httpcomponents.client.version>4.5.14</cxf.httpcomponents.client.version>
         <cxf.swagger.ui.version>5.9.0</cxf.swagger.ui.version>
         <cxf.tika.version>2.7.0</cxf.tika.version>
-        <cxf.tomcat.version>9.0.80</cxf.tomcat.version>
+        <cxf.tomcat.version>9.0.81</cxf.tomcat.version>
         <cxf.lucene.version>9.7.0</cxf.lucene.version>
         <cxf.aspectj.version>1.9.20.1</cxf.aspectj.version>
         <cxf.brave.version>5.16.0</cxf.brave.version>
diff --git a/parent/pom.xml b/parent/pom.xml
index 8ec43198ba..4342e8ff3d 100644
--- a/parent/pom.xml
+++ b/parent/pom.xml
@@ -216,7 +216,7 @@
         <cxf.swagger2.guava.version>32.1.3-jre</cxf.swagger2.guava.version>
         <cxf.swagger2.guava.failureaccess.version>1.0.1</cxf.swagger2.guava.failureaccess.version>
         <cxf.tika.version>2.7.0</cxf.tika.version>
-        <cxf.tomcat.version>9.0.80</cxf.tomcat.version>
+        <cxf.tomcat.version>9.0.81</cxf.tomcat.version>
         <cxf.tomitribe.http.signature.version>1.8</cxf.tomitribe.http.signature.version>
         <cxf.undertow.osgi.version>[1.4,3.0)</cxf.undertow.osgi.version>
         <cxf.undertow.xnio.osgi.version>[3.3,4.0)</cxf.undertow.xnio.osgi.version>