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

Jenkins.version saved only from doConfigSubmit

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      2.32.2. Set up Jenkins from the setup wizard. Install matrix-auth and configure admin to have everything but anonymous to have only discover, and Save. You will see config.xml as expected:

        <authorizationStrategy class="hudson.security.GlobalMatrixAuthorizationStrategy">
          <permission>hudson.model.Hudson.Administer:admin</permission>
          <permission>hudson.model.Hudson.Read:anonymous</permission>
          <permission>hudson.model.Item.Discover:anonymous</permission>
        </authorizationStrategy>
      

      Yet after restart if you revisit /configureSecurity, in the UI and reality, anonymous is granted Item.READ.

      Seems that the JENKINS-2324 workaround is incorrectly being applied, since Jenkins.isUpgradedFromBefore is broken, since version on config.xml is still 1.0!

      Workaround is to go to /configure and Save.

        Attachments

          Issue Links

            Activity

            jglick Jesse Glick created issue -
            jglick Jesse Glick made changes -
            Field Original Value New Value
            Link This issue relates to JENKINS-42556 [ JENKINS-42556 ]
            jglick Jesse Glick made changes -
            Link This issue blocks JENKINS-2324 [ JENKINS-2324 ]
            Hide
            jglick Jesse Glick added a comment -

            Fix should be to move

            version = VERSION;
            

            into save.

            Show
            jglick Jesse Glick added a comment - Fix should be to move version = VERSION; into save .
            jglick Jesse Glick made changes -
            Labels setup
            oleg_nenashev Oleg Nenashev made changes -
            Labels setup newbie-friendly setup
            liketic Li Ke made changes -
            Assignee Li Ke [ liketic ]
            liketic Li Ke made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            danielbeck Daniel Beck made changes -
            Link This issue is duplicated by JENKINS-17081 [ JENKINS-17081 ]
            Hide
            jglick Jesse Glick added a comment -

            If and when there is a PR for this, Link it here and set to In Review.

            Show
            jglick Jesse Glick added a comment - If and when there is a PR for this, Link it here and set to In Review .
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: kel
            Path:
            core/src/main/java/jenkins/model/Jenkins.java
            test/src/test/java/jenkins/install/InstallUtilTest.java
            test/src/test/java/jenkins/model/JenkinsTest.java
            http://jenkins-ci.org/commit/jenkins/27569a965a80caa1352e67c255065b5a67cb1a27
            Log:
            [Fix JENKINS-42577] Save jenkins version in Jenkins.save() (#3010)

            • [Fix JENKINS-42577] Save jenkins version in Jenkins.save()
            • [Fix JENKINS-42577] Fix unit test failures
            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: kel Path: core/src/main/java/jenkins/model/Jenkins.java test/src/test/java/jenkins/install/InstallUtilTest.java test/src/test/java/jenkins/model/JenkinsTest.java http://jenkins-ci.org/commit/jenkins/27569a965a80caa1352e67c255065b5a67cb1a27 Log: [Fix JENKINS-42577] Save jenkins version in Jenkins.save() (#3010) [Fix JENKINS-42577] Save jenkins version in Jenkins.save() [Fix JENKINS-42577] Fix unit test failures
            oleg_nenashev Oleg Nenashev made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            oleg_nenashev Oleg Nenashev made changes -
            Remote Link This issue links to "https://github.com/jenkinsci/jenkins/pull/3010 (Web Link)" [ 17754 ]
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            Jesse Glick Didn't notice your comment here. The PR is https://github.com/jenkinsci/jenkins/pull/3010 , it has been integrated towards the next weekly. If you have any concerns, please comment in the pull request

            Show
            oleg_nenashev Oleg Nenashev added a comment - Jesse Glick Didn't notice your comment here. The PR is https://github.com/jenkinsci/jenkins/pull/3010 , it has been integrated towards the next weekly. If you have any concerns, please comment in the pull request
            Hide
            danielbeck Daniel Beck added a comment -

            Fixed towards 2.80.

            Show
            danielbeck Daniel Beck added a comment - Fixed towards 2.80.
            danielbeck Daniel Beck made changes -
            Status In Review [ 10005 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            jglick Jesse Glick made changes -
            Link This issue depends on JENKINS-47139 [ JENKINS-47139 ]
            Hide
            jglick Jesse Glick added a comment -

            I presume you never confirmed that the originally reported bug was actually fixed, since the reproduction steps involve going through the setup wizard, which was broken by this as detailed in JENKINS-47139.

            Show
            jglick Jesse Glick added a comment - I presume you never confirmed that the originally reported bug was actually fixed, since the reproduction steps involve going through the setup wizard, which was broken by this as detailed in  JENKINS-47139 .

              People

              • Assignee:
                liketic Li Ke
                Reporter:
                jglick Jesse Glick
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: