CXF-8812: Content-Type header created with collection containing 'nul…

…l' value
apache · Jan 15, 2024 · d5d3707 · d5d3707
1 parent 3b58579
commit d5d3707
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/JAXRSBinding.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/JAXRSBinding.java
@@ -52,6 +52,7 @@ public org.apache.cxf.message.Message createMessage(org.apache.cxf.message.Messa
             if (exchange != null) {
                 ct = (String)exchange.get(org.apache.cxf.message.Message.CONTENT_TYPE);
             }
+
             m.put(org.apache.cxf.message.Message.CONTENT_TYPE, ct);
         }
         return m;

diff --git a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
@@ -380,8 +380,8 @@ private boolean isWSAddressingReplyToSpecified(Exchange ex) {
         }
         String contentType = req.getContentType();
         inMessage.put(Message.CONTENT_TYPE, contentType);
-        setEncoding(inMessage, req, contentType);
 
+        setEncoding(inMessage, req, contentType);
         inMessage.put(Message.QUERY_STRING, req.getQueryString());
 
         inMessage.put(Message.ACCEPT_CONTENT_TYPE, req.getHeader("Accept"));

diff --git a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Headers.java b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/Headers.java
@@ -448,7 +448,12 @@ protected void copyFromRequest(HttpServletRequest req) {
             }
         }
         if (!headers.containsKey(Message.CONTENT_TYPE)) {
-            headers.put(Message.CONTENT_TYPE, Collections.singletonList(req.getContentType()));
+            final String contentType = req.getContentType();
+            if (contentType != null) {
+                headers.put(Message.CONTENT_TYPE, Collections.singletonList(contentType));
+            } else {
+                headers.put(Message.CONTENT_TYPE, null);
+            }
         }
         if (LOG.isLoggable(Level.FINE)) {
             Map<String, List<Object>> theHeaders = CastUtils.cast(headers);