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

Mailer links of failed jobs are broken with BlueOcean plugin, not redirecting properly

    Details

    • Epic Link:
    • Sprint:
      tethys, pannonian
    • Similar Issues:

      Description

      After installing BlueOcean beta, all links in the emails were changed to use BlueOcean URLs (which I find inconvenient, this should not be automatic in my opinion and I have been able to find a config option for this − happy to open a separate bug for this).

      For the emails indicating a failure, the links are broken. Here are some examples:

      After uninstalling the BlueOcean plugins, the links are functional again:

        Attachments

          Issue Links

            Activity

            Hide
            jamesdumay James Dumay added a comment -

            Good news is that I tracked this down to a regression when we introduced the hook to allow Blue Ocean URLs to override the classic ones. It was a simple fix and it is currently in review. You will require an upgrade to the mailer plugin to receive this fix.

            Show
            jamesdumay James Dumay added a comment - Good news is that I tracked this down to a regression when we introduced the hook to allow Blue Ocean URLs to override the classic ones. It was a simple fix and it is currently in review . You will require an upgrade to the mailer plugin to receive this fix.
            Hide
            jamesdumay James Dumay added a comment -

            Jean-Frédéric you should be able to set a preference in your user profile so that you receive classic Jenkins links instead of Blue Ocean and there is a fix for Matrix projects in beta 21 this week.

            Show
            jamesdumay James Dumay added a comment - Jean-Frédéric you should be able to set a preference in your user profile so that you receive classic Jenkins links instead of Blue Ocean and there is a fix for Matrix projects in beta 21 this week.
            Hide
            jamesdumay James Dumay added a comment -

            This fix should appear in mailer 1.19 shortly.

            Thanks Jesse Glick for merging and releasing this one.

            Show
            jamesdumay James Dumay added a comment - This fix should appear in mailer 1.19 shortly. Thanks Jesse Glick for merging and releasing this one.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Evaristo Gutiérrez
            Path:
            src/main/java/hudson/tasks/MailSender.java
            src/test/java/hudson/tasks/MailerTest.java
            http://jenkins-ci.org/commit/mailer-plugin/11b6c3a5119d29e27b8dfd1b5a0080b5b9497da7
            Log:
            JENKINS-41881 Include automated test to cover fix for JENKINS-40224

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Evaristo Gutiérrez Path: src/main/java/hudson/tasks/MailSender.java src/test/java/hudson/tasks/MailerTest.java http://jenkins-ci.org/commit/mailer-plugin/11b6c3a5119d29e27b8dfd1b5a0080b5b9497da7 Log: JENKINS-41881 Include automated test to cover fix for JENKINS-40224
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/hudson/tasks/MailSender.java
            src/test/java/hudson/tasks/MailerTest.java
            http://jenkins-ci.org/commit/mailer-plugin/d54f8d8ed32227c66a9a24a1a31986a19838ce26
            Log:
            Merge pull request #35 from varyvol/JENKINS-41881

            JENKINS-41881 Automated test to cover fix for JENKINS-40224

            Compare: https://github.com/jenkinsci/mailer-plugin/compare/596255227d97...d54f8d8ed322

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/hudson/tasks/MailSender.java src/test/java/hudson/tasks/MailerTest.java http://jenkins-ci.org/commit/mailer-plugin/d54f8d8ed32227c66a9a24a1a31986a19838ce26 Log: Merge pull request #35 from varyvol/ JENKINS-41881 JENKINS-41881 Automated test to cover fix for JENKINS-40224 Compare: https://github.com/jenkinsci/mailer-plugin/compare/596255227d97...d54f8d8ed322

              People

              • Assignee:
                jamesdumay James Dumay
                Reporter:
                jeanfred Jean-Frédéric
              • Votes:
                1 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: