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

Link created by naginator plugin contains comma in job numbers over 999

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: naginator-plugin
    • Labels:
      None
    • Environment:
      Jenkins 2.89.4
      Naginator 1.17.2
    • Similar Issues:
    • Released As:
      naginator-1.18

      Description

      The Naginator plugin creates a link on the project's build page like the following:

      Started by Naginator after the failure of build 2018-04-11-143326

      Where "2018-04-11-143326" is the name of a build that failed and is hyperlinked to the build output.  The hyperlink is:

      https://my.jenkins.unisys.com/view/Stealth(aware)/job/NFoS-Sysgen-Branch/1,070/

      As you can see, when the build number exceeds 999 a comma is injected into the URL, making it invalid.

       

        Attachments

          Issue Links

            Activity

            Show
            ikedam ikedam added a comment - https://github.com/jenkinsci/naginator-plugin/pull/45
            Hide
            ryantse_sap Ryan Tse added a comment -

            Can this change be reviewed, merged, and released?

            Show
            ryantse_sap Ryan Tse added a comment - Can this change be reviewed, merged, and released?
            Hide
            ikedam ikedam added a comment -

            Ryan Tse
            I plan to make a new release in February with other fixes.

            Show
            ikedam ikedam added a comment - Ryan Tse I plan to make a new release in February with other fixes.
            Hide
            ikedam ikedam added a comment -

            Robert Baker Ryan Tse
            This is fixed in naginator-1.18.
            It will be available in the update center in a day.
            Please try that.

            Show
            ikedam ikedam added a comment - Robert Baker Ryan Tse This is fixed in naginator-1.18. It will be available in the update center in a day. Please try that.

              People

              • Assignee:
                rsbaker56 Robert Baker
                Reporter:
                rsbaker56 Robert Baker
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: