This is an automated email from the ASF dual-hosted git repository. davsclaus pushed a commit to branch camel-4.0.x in repository https://gitbox.apache.org/repos/asf/camel.git
The following commit(s) were added to refs/heads/camel-4.0.x by this push: new fddd6f58a1a [CAMEL-20037] camel-http builds StringEntity with wrong contentEncoding (#11828) fddd6f58a1a is described below commit fddd6f58a1a16a4cfdda9f50e31071db64dfc6d1 Author: Simo Kivimäki <85223075+solita-sim...@users.noreply.github.com> AuthorDate: Wed Oct 25 12:02:33 2023 +0300 [CAMEL-20037] camel-http builds StringEntity with wrong contentEncoding (#11828) * CAMEL-20037: camel-http must not create StringEntity having charset as content encoding * camel-http autoformatted code --- .../src/main/java/org/apache/camel/component/http/HttpProducer.java | 2 +- .../camel/component/http/HttpProducerContentTypeWithCharsetTest.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java b/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java index da9c7e0d034..b4974b1c5e7 100644 --- a/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java +++ b/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java @@ -778,7 +778,7 @@ public class HttpProducer extends DefaultProducer { contentType = ContentType.parse(contentType + ";charset=" + charset); } - answer = new StringEntity(content, contentType, charset, false); + answer = new StringEntity(content, contentType, false); } // fallback as input stream diff --git a/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerContentTypeWithCharsetTest.java b/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerContentTypeWithCharsetTest.java index eef598225ef..7e2f05a2745 100644 --- a/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerContentTypeWithCharsetTest.java +++ b/components/camel-http/src/test/java/org/apache/camel/component/http/HttpProducerContentTypeWithCharsetTest.java @@ -51,6 +51,7 @@ public class HttpProducerContentTypeWithCharsetTest extends BaseHttpTest { String contentType = request.getFirstHeader(Exchange.CONTENT_TYPE).getValue(); assertEquals(CONTENT_TYPE_WITH_CHARSET.replace(";", "; "), contentType); + assertFalse(request.containsHeader(Exchange.CONTENT_ENCODING)); response.setEntity(new StringEntity(contentType, StandardCharsets.US_ASCII)); response.setCode(HttpStatus.SC_OK);