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

CLI is not parsing arguments with remote user authentication set

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • cli
    • None

    Description

      CLI options are parsed before the remote authentication is set to securityContextHolder. Side effect is that AbstractProject options are populated with default authentication, i.e. anonymous that may not have READ permission on jobs.

      Attachments

        Issue Links

          Activity

            main issue here is that options need to be parsed for CliAuthenticator to compute --username and --password
            but SecurityContext.authorization need to be set to populate AbstractProject

            ndeloof Nicolas De Loof added a comment - main issue here is that options need to be parsed for CliAuthenticator to compute --username and --password but SecurityContext.authorization need to be set to populate AbstractProject

            Code changed in jenkins
            User: Nicolas De Loof
            Path:
            core/src/main/java/hudson/cli/CLICommand.java
            http://jenkins-ci.org/commit/jenkins/c7991585bcc2d640f2befd56104224320f5ed0fa
            Log:
            JENKINS-14745 use transportAuthentication to parse command arguments

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nicolas De Loof Path: core/src/main/java/hudson/cli/CLICommand.java http://jenkins-ci.org/commit/jenkins/c7991585bcc2d640f2befd56104224320f5ed0fa Log: JENKINS-14745 use transportAuthentication to parse command arguments
            dogfood dogfood added a comment -

            Integrated in jenkins_main_trunk #1856

            Result = UNSTABLE

            dogfood dogfood added a comment - Integrated in jenkins_main_trunk #1856 Result = UNSTABLE
            evernat evernat added a comment -

            This one is also committed, and so I suppose this is also fixed.

            evernat evernat added a comment - This one is also committed, and so I suppose this is also fixed.

            People

              ndeloof Nicolas De Loof
              ndeloof Nicolas De Loof
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: