Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-49913

email-ext: check for attachment size does not consider compression

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • email-ext-plugin
    • None

      We are attaching build log to the mail in Jenkins pipeline, and are setting both "attachLog" and "compressLog" options to true. The build logs say "Skipping build log attachment -  too large for maximum attachments size".

      When downloading the logs manually, these are about 40 MB in size (uncompressed) and less than 3 MB compressed. The "Maximum Attachment Size" is set to 16 MB on global setting page, so the compressed log should pass in any case.

      It seem the plugin is comparing size of uncompressed logs with the given limit, which is not what I would expect.

          [JENKINS-49913] email-ext: check for attachment size does not consider compression

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: David van Laatum
          Path:
          src/main/java/hudson/plugins/emailext/AttachmentUtils.java
          src/main/java/hudson/plugins/emailext/SizedDataSource.java
          src/main/java/hudson/plugins/emailext/plugins/ZipDataSource.java
          src/test/java/hudson/plugins/emailext/AttachmentUtilsTest.java
          http://jenkins-ci.org/commit/email-ext-plugin/1cde3b10372c11a2b010f25c1d99092d248ed7ae
          Log:
          JENKINS-49913 check for attachment size does not consider compression

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: David van Laatum Path: src/main/java/hudson/plugins/emailext/AttachmentUtils.java src/main/java/hudson/plugins/emailext/SizedDataSource.java src/main/java/hudson/plugins/emailext/plugins/ZipDataSource.java src/test/java/hudson/plugins/emailext/AttachmentUtilsTest.java http://jenkins-ci.org/commit/email-ext-plugin/1cde3b10372c11a2b010f25c1d99092d248ed7ae Log: JENKINS-49913 check for attachment size does not consider compression

            davidvanlaatum David van Laatum
            chamshoff Christoph Amshoff
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: