• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Trivial Trivial
    • core
    • None
    • All

      REST api link doesn't work if you access it from the security settings page for example.

      Steps to reproduce:

      1. Open Jenkins Frontend
      2. Manage Jenkins Link
      3. Configure Global Security
      4. Click "REST API" link at the bottom of the page

      Link points to:
      http://localhost:8080/configureSecurity/api/

      and should instead point to:
      http://localhost:8080/api/

          [JENKINS-29014] REST API Footer Link Doesn't Work from Subpages

          That seems not only for that page.. As for now a lot of page without it's own API seems to be broken with that link.

          Stanislav Eremin added a comment - That seems not only for that page.. As for now a lot of page without it's own API seems to be broken with that link.

          Christopher Simons added a comment - - edited

          Fixed in PR #1850.

          Christopher Simons added a comment - - edited Fixed in PR #1850 .

          Code changed in jenkins
          User: tfennelly
          Path:
          .gitignore
          changelog.html
          cli/pom.xml
          core/pom.xml
          core/src/main/java/hudson/model/AbstractProject.java
          core/src/main/java/hudson/model/Run.java
          core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly
          core/src/main/resources/lib/form/confirm.js
          core/src/main/resources/lib/form/select/select.js
          core/src/main/resources/lib/layout/layout.jelly
          plugins/pom.xml
          pom.xml
          test/pom.xml
          test/src/test/groovy/hudson/model/AbstractProjectTest.groovy
          war/pom.xml
          http://jenkins-ci.org/commit/jenkins/95ca3da67d217c90d31819ec92e521e2072acd5a
          Log:
          Merge branch 'master' into plugin-manager-dependants

          • master:
            Update the changelog by new merges:
            [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error
            [FIXED JENKINS-29014] render API link conditional on getApi() presence
            JENKINS-21720 JS alert preventig to leave a configuration page even without formulary changes
            add ctags file 'tags' to .gitignore
            [maven-release-plugin] prepare release jenkins-1.632
            [FIXED JENKINS-29888] Handling all exceptions returned by logRotator
            JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: tfennelly Path: .gitignore changelog.html cli/pom.xml core/pom.xml core/src/main/java/hudson/model/AbstractProject.java core/src/main/java/hudson/model/Run.java core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly core/src/main/resources/lib/form/confirm.js core/src/main/resources/lib/form/select/select.js core/src/main/resources/lib/layout/layout.jelly plugins/pom.xml pom.xml test/pom.xml test/src/test/groovy/hudson/model/AbstractProjectTest.groovy war/pom.xml http://jenkins-ci.org/commit/jenkins/95ca3da67d217c90d31819ec92e521e2072acd5a Log: Merge branch 'master' into plugin-manager-dependants master: Update the changelog by new merges: [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error [FIXED JENKINS-29014] render API link conditional on getApi() presence JENKINS-21720 JS alert preventig to leave a configuration page even without formulary changes add ctags file 'tags' to .gitignore [maven-release-plugin] prepare release jenkins-1.632 [FIXED JENKINS-29888] Handling all exceptions returned by logRotator JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()

          Code changed in jenkins
          User: Christopher Simons
          Path:
          core/src/main/resources/lib/layout/layout.jelly
          http://jenkins-ci.org/commit/jenkins/76067e0af93d3681756bf290aeb1f3a1e88b1d39
          Log:
          [FIXED JENKINS-29014] render API link conditional on getApi() presence

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Christopher Simons Path: core/src/main/resources/lib/layout/layout.jelly http://jenkins-ci.org/commit/jenkins/76067e0af93d3681756bf290aeb1f3a1e88b1d39 Log: [FIXED JENKINS-29014] render API link conditional on getApi() presence

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/resources/lib/layout/layout.jelly
          http://jenkins-ci.org/commit/jenkins/d7fe23ca32c5794070ae924de9a2b0ebdfb42a11
          Log:
          Merge pull request #1850 from csimons/fix-rest-api-link

          [FIXED JENKINS-29014] fix broken 'REST API' link

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/resources/lib/layout/layout.jelly http://jenkins-ci.org/commit/jenkins/d7fe23ca32c5794070ae924de9a2b0ebdfb42a11 Log: Merge pull request #1850 from csimons/fix-rest-api-link [FIXED JENKINS-29014] fix broken 'REST API' link

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: changelog.html http://jenkins-ci.org/commit/jenkins/a79e2f144a932fe0a97b524f79133d1ea5d0fef0 Log: Update the changelog by new merges: JENKINS-29888 - https://github.com/jenkinsci/jenkins/pull/1790 JENKINS-30742 - https://github.com/jenkinsci/jenkins/pull/1849 JENKINS-29014 - https://github.com/jenkinsci/jenkins/pull/1850 JENKINS-21720 - https://github.com/jenkinsci/jenkins/pull/1854 JENKINS-30569 - https://github.com/jenkinsci/jenkins/pull/1856

            csimons Christopher Simons
            kn122754 Karel Nenicka
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: