Camel
  1. Camel
  2. CAMEL-4126

S3Producer does not use CamelAwsS3ContentLength or CamelAwsS3ContentType

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8.0
    • Fix Version/s: 2.8.0
    • Component/s: camel-aws
    • Labels:
      None

      Description

      With InputStreams, Amazon's Java libraries aren't able to generate the content length or content type, and we get warnings like this:

      WARN AmazonS3Client - No content length specified for stream data. Stream contents will be buffered in memory and could result in out of memory errors.

      Not including the content type means S3 will be serving the uploaded file as generic binary.

      Since there are already Camel headers for this, I'm proposing that they be used when sending messages to S3. I can write a patch for this if it would help.

        Activity

        Brendan Long created issue -
        Brendan Long made changes -
        Field Original Value New Value
        Component/s camel-aws [ 12314194 ]
        Component/s camel-atom [ 12313954 ]
        Hide
        Richard Kettelerij added a comment -

        Patches are always welcome.

        Show
        Richard Kettelerij added a comment - Patches are always welcome.
        Hide
        Brendan Long added a comment -

        Patch to use these headers

        Show
        Brendan Long added a comment - Patch to use these headers
        Brendan Long made changes -
        Attachment s3-content-length-and-type-patch.diff [ 12483203 ]
        Christian Müller made changes -
        Assignee Christian Müller [ muellerc ]
        Hide
        Christian Müller added a comment -

        Committed r1139939
        Thanks Brendan for the patch.
        Working on the wiki page...

        Show
        Christian Müller added a comment - Committed r1139939 Thanks Brendan for the patch. Working on the wiki page...
        Christian Müller made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 2.8.0 [ 12316226 ]
        Resolution Fixed [ 1 ]
        Christian Müller made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Christian Müller
            Reporter:
            Brendan Long
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development