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

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

    Details

    • Similar Issues:

      Description

      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.

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link 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

            Show
            scm_issue_link 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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: