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

pipeline build failed mail - missing line break

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • mailer-plugin
    • jenkins 2.19.3
      Mailer plugin 1.18
    • 1.26

      when catching a build failure and sending an email via:

      catch (err) {
    node ('Client-Slave'){
        echo "Failed: ${err}"
        currentBuild.result = 'FAILURE'
        step([$class: 'Mailer', recipients: 'example@example.com, recipient2@example.com'])
    }
}

      the mail is send as:

      [...]
X-Jenkins-Job: example_pipeline
X-Jenkins-Result: FAILURE

https://jenkins.example.com/blue/organizations/jenkins/example_pipeline/detail/example_pipeline/21/------------------------------------------
[...truncated 25 lines...]
[...]

      it would be better to send it with a line break after the url, as the ----- are treated as part of the url by some mail clients:

      [...]
X-Jenkins-Job: example_pipeline
X-Jenkins-Result: FAILURE

https://jenkins.example.com/blue/organizations/jenkins/example_pipeline/detail/example_pipeline/21/
------------------------------------------
[...truncated 25 lines...]
[...]

          [JENKINS-39976] pipeline build failed mail - missing line break

          Thomas Leufkes added a comment -

          please fix this.

          Thomas Leufkes added a comment - please fix this.

          Niko Madalinski added a comment -

          Same bug here. The generated url in mail ist broken

          Niko Madalinski added a comment - Same bug here. The generated url in mail ist broken

          Jesse Glick added a comment -

          Is this actually Pipeline-specific?

          Jesse Glick added a comment - Is this actually Pipeline-specific?

          Jesse Glick added a comment -

          PR 33?

          Jesse Glick added a comment - PR 33?

          Roman Pickl added a comment -

          I think it is fixed.

          Roman Pickl added a comment - I think it is fixed.

          Adrien Lecharpentier added a comment -

          romanp are you sure it's fixed? I was about to integrate the PR in mailer plugin but if you say it's not a problem anymore.. I'm perplexed now.

          Adrien Lecharpentier added a comment - romanp are you sure it's fixed? I was about to integrate the PR in mailer plugin but if you say it's not a problem anymore.. I'm perplexed now.

          Oleg Nenashev added a comment -

          I do not see changes which would have fixed that. The PR is trivial, so probably we should integrate it and move on

          Oleg Nenashev added a comment - I do not see changes which would have fixed that. The PR is trivial, so probably we should integrate it and move on

            Unassigned Unassigned
            romanp Roman Pickl
            Votes:
            3 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved: