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

PluginSetupWizard hangs if the default update site is not resolvable

      If the 'default' UpdateSite ID cannot be resolved due to whatever reason, Jenkins Installation wizard hangs infinitely.

      As a solution, I propose to return a fatal error to the WebUI.

          [JENKINS-34675] PluginSetupWizard hangs if the default update site is not resolvable

          Oleg Nenashev created issue -
          Oleg Nenashev made changes -
          Link New: This issue is related to JENKINS-34674 [ JENKINS-34674 ]

          Daniel Beck added a comment -

          What specifically does 'cannot be resolved' mean? Network issue? Or there is no such configured update site?

          Daniel Beck added a comment - What specifically does 'cannot be resolved' mean? Network issue? Or there is no such configured update site?

          Oleg Nenashev added a comment -

          danielbeck
          It's a follow-up to JENKINS-34674, which may expose the code issue heavily. If the requested UpdateSite ID is not resolvable (e.g. no JENKINS_HOME/updates/UPDATESITE_ID.json), the current logic does not handle it effectively.

          PR: https://github.com/jenkinsci/jenkins/pull/2319

          Oleg Nenashev added a comment - danielbeck It's a follow-up to JENKINS-34674 , which may expose the code issue heavily. If the requested UpdateSite ID is not resolvable (e.g. no JENKINS_HOME/updates/UPDATESITE_ID.json), the current logic does not handle it effectively. PR: https://github.com/jenkinsci/jenkins/pull/2319
          Oleg Nenashev made changes -
          Labels New: installation ui

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/java/hudson/model/UpdateCenter.java
          war/src/main/js/pluginSetupWizardGui.js
          war/src/main/js/util/jenkins.js
          http://jenkins-ci.org/commit/jenkins/5d1f81c56c9635ec53dce2b60563047837361251
          Log:
          [FIXED JENKINS-34675] - Prevent hanging of the WebUI if the default Update Site ID cannot be resolved.

          Also hardens the Java code a bit.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/model/UpdateCenter.java war/src/main/js/pluginSetupWizardGui.js war/src/main/js/util/jenkins.js http://jenkins-ci.org/commit/jenkins/5d1f81c56c9635ec53dce2b60563047837361251 Log: [FIXED JENKINS-34675] - Prevent hanging of the WebUI if the default Update Site ID cannot be resolved. Also hardens the Java code a bit.
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          test/src/test/java/hudson/model/UpdateCenterConnectionStatusTest.java
          http://jenkins-ci.org/commit/jenkins/25f6c2cf6d771668fe1caae1c11c82a2b80284e7
          Log:
          JENKINS-34675 - Fix the Unit test

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: test/src/test/java/hudson/model/UpdateCenterConnectionStatusTest.java http://jenkins-ci.org/commit/jenkins/25f6c2cf6d771668fe1caae1c11c82a2b80284e7 Log: JENKINS-34675 - Fix the Unit test

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/java/hudson/model/UpdateCenter.java
          test/src/test/java/hudson/model/UpdateCenterConnectionStatusTest.java
          war/src/main/js/pluginSetupWizardGui.js
          war/src/main/js/util/jenkins.js
          http://jenkins-ci.org/commit/jenkins/0b3dbfce64a3cae69e96beb6f7a437db587e401a
          Log:
          Merge pull request #2319 from oleg-nenashev/JENKINS-34674

          [JENKINS-34674,JENKINS-34675] - Handling of the default update site ID

          Compare: https://github.com/jenkinsci/jenkins/compare/b9bb52ffce51...0b3dbfce64a3

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/java/hudson/model/UpdateCenter.java test/src/test/java/hudson/model/UpdateCenterConnectionStatusTest.java war/src/main/js/pluginSetupWizardGui.js war/src/main/js/util/jenkins.js http://jenkins-ci.org/commit/jenkins/0b3dbfce64a3cae69e96beb6f7a437db587e401a Log: Merge pull request #2319 from oleg-nenashev/ JENKINS-34674 [JENKINS-34674,JENKINS-34675] - Handling of the default update site ID Compare: https://github.com/jenkinsci/jenkins/compare/b9bb52ffce51...0b3dbfce64a3
          Oleg Nenashev made changes -
          Labels Original: installation ui New: installation lts-candidate ui

            oleg_nenashev Oleg Nenashev
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: