You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2021/01/29 17:08:03 UTC

[camel] branch master updated: CAMEL-16110: don't leak internal QUERY_STRING header (#4956)

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

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/master by this push:
     new 812d364  CAMEL-16110: don't leak internal QUERY_STRING header (#4956)
812d364 is described below

commit 812d3648964be56db7043022683732d276d77ac9
Author: fizzet <je...@gmx.net>
AuthorDate: Fri Jan 29 18:07:41 2021 +0100

    CAMEL-16110: don't leak internal QUERY_STRING header (#4956)
---
 .../org/apache/camel/component/cxf/common/header/CxfHeaderHelper.java  | 3 ++-
 .../camel/component/cxf/jaxrs/CxfRsProducerHttpMethodHeaderTest.java   | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/components/camel-cxf-transport/src/main/java/org/apache/camel/component/cxf/common/header/CxfHeaderHelper.java b/components/camel-cxf-transport/src/main/java/org/apache/camel/component/cxf/common/header/CxfHeaderHelper.java
index eb05b4f..179b2f1 100644
--- a/components/camel-cxf-transport/src/main/java/org/apache/camel/component/cxf/common/header/CxfHeaderHelper.java
+++ b/components/camel-cxf-transport/src/main/java/org/apache/camel/component/cxf/common/header/CxfHeaderHelper.java
@@ -91,7 +91,8 @@ public final class CxfHeaderHelper {
 
             // drop this header as we do not want to propagate the http method/path into the CXF request message
             if (Exchange.HTTP_METHOD.equalsIgnoreCase(entry.getKey())
-                    || Exchange.HTTP_PATH.equalsIgnoreCase(entry.getKey())) {
+                    || Exchange.HTTP_PATH.equalsIgnoreCase(entry.getKey())
+                    || Exchange.HTTP_QUERY.equalsIgnoreCase(entry.getKey())) {
                 return;
             }
 
diff --git a/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducerHttpMethodHeaderTest.java b/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducerHttpMethodHeaderTest.java
index b95279c..adb3286 100644
--- a/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducerHttpMethodHeaderTest.java
+++ b/components/camel-cxf/src/test/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducerHttpMethodHeaderTest.java
@@ -48,6 +48,7 @@ public class CxfRsProducerHttpMethodHeaderTest extends CamelTestSupport {
                         Message inMessage = exchange.getIn();
                         inMessage.setHeader(Exchange.HTTP_METHOD, "GET");
                         inMessage.setHeader(Exchange.HTTP_PATH, "/CxfRsProducerHttpMethodHeaderTest/");
+                        inMessage.setHeader(Exchange.HTTP_QUERY, "q=1");
                         inMessage.setHeader(Exchange.CONTENT_TYPE, "application/text");
                         inMessage.setBody("Hello World");
                     }