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

rssAll feed no longer contains Jenkins server URL, only incorrect relative path

    • 2.194

      Since you fixed JENKINS-58595, the rssAll feed no longer has the server name. Before this fix it was only showing the server HTTP address and no path in the href elements, now it's other way round. The HTTP-address is missing and only an (incorrect) relative link is shown. Not sure why it no longer shows full href.

      This still breaks our monitoring tools.

          [JENKINS-59167] rssAll feed no longer contains Jenkins server URL, only incorrect relative path

          Uwe Schindler added a comment -

          Uwe Schindler added a comment - Here is an example: https://jenkins.thetaphi.de/view/Lucene-Solr/rssAll

          Uwe Schindler added a comment -

          example: <link rel="alternate" type="text/html" href="view/Lucene-Solr/job/Lucene-Solr-master-Linux/24643/"/>

          Should look like this: <link rel="alternate" type="text/html" href="https://jenkins.thetaphi.de/view/Lucene-Solr/job/Lucene-Solr-master-Linux/24643/"/>

          Uwe Schindler added a comment - example: <link rel="alternate" type="text/html" href="view/Lucene-Solr/job/Lucene-Solr-master-Linux/24643/"/> Should look like this: <link rel="alternate" type="text/html" href="https://jenkins.thetaphi.de/view/Lucene-Solr/job/Lucene-Solr-master-Linux/24643/"/>

          Daniel Beck added a comment -

          thetaphi Could you clarify whether you configured the "Jenkins Location" in the global configuration?

          Daniel Beck added a comment - thetaphi Could you clarify whether you configured the "Jenkins Location" in the global configuration?

          I can confirm that the rootUrl being set or not does not have any incidence on the problem. You can find as attachment example before SEC-534 fix, one after SEC-534 and finally one after the correction of JENKINS-58595.

          Wadeck Follonier added a comment - I can confirm that the rootUrl being set or not does not have any incidence on the problem. You can find as attachment example before SEC-534 fix, one after SEC-534 and finally one after the correction of JENKINS-58595 .

          PR proposed, added the result after the correction as attachment.

          Wadeck Follonier added a comment - PR proposed, added the result after the correction as attachment.

          Uwe Schindler added a comment -

          Thanks. The chAnge looks fine, but not tested.

          Just for completeness: the Jenkins URL was of course defined, otherwise other stuff would have been broken, too.

          Uwe Schindler added a comment - Thanks. The chAnge looks fine, but not tested. Just for completeness: the Jenkins URL was of course defined, otherwise other stuff would have been broken, too.

          Uwe Schindler added a comment -

          Uwe Schindler added a comment - Just FYI, looks fine again: https://jenkins.thetaphi.de/view/Lucene-Solr/rssAll

          thetaphi Thank you for the quick feedback! really appreciated

          Wadeck Follonier added a comment - thetaphi Thank you for the quick feedback! really appreciated

          Kunal Kannav added a comment -

          Hi thetaphi, We are still facing the issue.

          '/' is missing in the href on the RSS XML

           

          Thanks,

          Kunal

           

          Kunal Kannav added a comment - Hi thetaphi , We are still facing the issue. '/' is missing in the href on the RSS XML   Thanks, Kunal  

          Hi kkannav,

          Which version did you test with? Only the 2.194+ has the correction yet and on Sept 25th, the 2.190.1 (new LTS) will also have it. You could already test the LTS RC version here: http://mirrors.jenkins-ci.org/war-stable-rc/2.190.1/jenkins.war.

          In case you tested with a "valid" version, could you please provide the RSS actual result and the expected result?

          Thank you in advance

          Wadeck Follonier added a comment - Hi kkannav , Which version did you test with? Only the 2.194+ has the correction yet and on Sept 25th, the 2.190.1 (new LTS) will also have it. You could already test the LTS RC version here: http://mirrors.jenkins-ci.org/war-stable-rc/2.190.1/jenkins.war . In case you tested with a "valid" version, could you please provide the RSS actual result and the expected result? Thank you in advance

            wfollonier Wadeck Follonier
            thetaphi Uwe Schindler
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: