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

CLI is not parsing arguments with remote user authentication set

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • cli
    • None

      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.

          [JENKINS-14745] CLI is not parsing arguments with remote user authentication set

          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

          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/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 added a comment -

          Integrated in jenkins_main_trunk #1856

          Result = UNSTABLE

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

          evernat added a comment -

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

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

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

              Created:
              Updated:
              Resolved: