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

Display URL Plugin does not consult with DisplayURLProvider implementations when injecting environment variables

    • display-url-api 2.3.4

      As a user, I want to override URLs injected by the plugin into environment variables

      There is a DisplayURLProvider#getDefault() API which can be used to retrieve the provider extension defined by the user. There is also a DisplayURLProvider#get() which always returns the default provider instance.

      The second method is used in https://github.com/jenkinsci/display-url-api-plugin/blob/master/src/main/java/org/jenkinsci/plugins/displayurlapi/EnvironmentContributorImpl.java which contributes environment variables. It means that the provider implementations won't be taken into account.

       

      This is somewaht related to JENKINS-41919, but it is not reported as defect

          [JENKINS-64119] Display URL Plugin does not consult with DisplayURLProvider implementations when injecting environment variables

          Oleg Nenashev created issue -
          Oleg Nenashev made changes -
          Link New: This issue relates to JENKINS-41919 [ JENKINS-41919 ]
          Oleg Nenashev made changes -
          Link New: This issue relates to JENKINS-56406 [ JENKINS-56406 ]
          Olivier Lamy made changes -
          Assignee Original: James Dumay [ jamesdumay ] New: Olivier Lamy [ olamy ]
          Olivier Lamy made changes -
          Comment [ {{[~oleg_nenashev] user defined means class implementation defined via sysproperty `jenkins.displayurl.provider` or }}PreferredProviderUserProperty#getConfiguredProvider {{?}}

          or both? :)  ]
          Kalle Niemitalo made changes -
          Link New: This issue causes JENKINS-69006 [ JENKINS-69006 ]
          Devin Nusbaum made changes -
          Released As New: display-url-api 2.3.4
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Devin Nusbaum made changes -
          Link New: This issue is duplicated by JENKINS-41919 [ JENKINS-41919 ]
          Devin Nusbaum made changes -
          Link Original: This issue relates to JENKINS-41919 [ JENKINS-41919 ]

            olamy Olivier Lamy
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: