[CAMEL-20099] Camel-http is creating invalid Content-Encoding header based on charset from Content-Type header

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.2, 4.1.0
• Fix Version/s: 4.0.3, 4.2.0
• Component/s: camel-http
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Damir Palinic
• Created: 10/Nov/23 10:49
• Updated: 23/Nov/23 11:10
• Resolved: 14/Nov/23 10:28

Description

In case Content-Type header has ";charset=UTF-8" suffix, camel-http component creates Content-Encoding: UTF-8 header which is invalid by RFC specification.

If you try to send request with Content-Encoding: UTF-8 to SAP systems for example, they will reject it due to invalid header. 

YAML DSL to reproduce the issue:

 

- route:
    nodePrefixId: route-7e2
    id: route-dafc
    from:
      uri: scheduler
      id: from-64dc
      parameters:
        name: scheduler
      steps:
        - setHeader:
            name: Content-Type
            expression:
              simple:
                expression: application/json;charset=UTF-8
                id: simple-dc79
            id: setHeader-2f26
        - setBody:
            expression:
              simple:
                expression: |-
                  {
                    "some": "value"
                  }
                id: simple-bfca
            id: setBody-243c
        - to:
            uri: http
            id: to-2cfa
            parameters:
              httpUri: >-
                01het4h4zq3gvtecszbv5rpx3y00-4f92b8ba4ea1459ece69.requestinspector.com
              httpMethod: POST
 

Additional notes:

• If you remove ";charset=UTF-8" part from the Content-Type header, and call one more http request after the first one, it gets appended automatically for some reason, even if you remove Content-Type header and set it again manually (possible another bug) between two calls
• Same behavior is for the https traffic

 

 

Issue Links

• duplicates: CAMEL-20037 camel-http builds StringEntity with wrong contentEncoding
• links to: GitHub Pull Request #11985