• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • Jenkins 2.346
    • 2.357, 2.346.2

      The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)

      Functions.tryGetIconPath() appends the context path to the URL https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/Functions.java#L2380
      but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice. https://github.com/jenkinsci/jenkins/blob/20e34cb942a85fe1b55bb193ef3378c2eeb82098/core/src/main/java/hudson/model/StockStatusIcon.java#L31-L36

          [JENKINS-68639] Malformed URLs for status icons images

          Alina Strohaya created issue -
          Alina Strohaya made changes -
          Assignee New: Alina Strohaya [ astrohaya ]
          Alina Strohaya made changes -
          Description Original: The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)
          !image-2022-05-30-18-35-27-817.png!

          Functions.tryGetIconPath() appends the context path to the URL, but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice.
          New: The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)
          !image-2022-05-30-18-35-27-817.png!

          Functions.tryGetIconPath() appends the context path to the URL, but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice. (hudson.model.StockStatusIcon#22)
          Alina Strohaya made changes -
          Description Original: The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)
          !image-2022-05-30-18-35-27-817.png!

          Functions.tryGetIconPath() appends the context path to the URL, but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice. (hudson.model.StockStatusIcon#22)
          New: The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)
          !image-2022-05-30-18-35-27-817.png!

          Functions.tryGetIconPath() appends the context path to the URL (hudson.Functions#2380), but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice. (hudson.model.StockStatusIcon#22)
          Alina Strohaya made changes -
          Description Original: The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)
          !image-2022-05-30-18-35-27-817.png!

          Functions.tryGetIconPath() appends the context path to the URL (hudson.Functions#2380), but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice. (hudson.model.StockStatusIcon#22)
          New: The raster image status icons of jobs on the dashboard are expected to display correctly. The actual behavior is icons being missing because the image URLs are malformed, containing the context path twice (/jenkins/jenkins/ on the following screenshot)
          !image-2022-05-30-18-35-27-817.png!

          Functions.tryGetIconPath() appends the context path to the URL https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/Functions.java#L2380
          but before that, StatusIcon.getImageOf() will have already appended it for StockStatusIcon and every other extension of StatusIcon that implements that method similarly, resulting in the malformed URL where the context path repeats twice. [https://github.com/jenkinsci/jenkins/blob/20e34cb942a85fe1b55bb193ef3378c2eeb82098/core/src/main/java/hudson/model/StockStatusIcon.java#L31-L36]
          Vincent Latombe made changes -
          Labels New: lts-candidate
          Vincent Latombe made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Vincent Latombe made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Vincent Latombe made changes -
          Remote Link New: This issue links to "PR #6616 (Web Link)" [ 27821 ]
          Basil Crow made changes -
          Released As New: 2.357
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Closed [ 6 ]

            astrohaya Alina Strohaya
            astrohaya Alina Strohaya
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: