Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-17929] Pick up transport authentication from groovysh …
…command, by overriding run rather than main.

Originally-Committed-As: 3896aabc0bee39fd94366f5b2dd6bc191c040e40
  • Loading branch information
jglick committed Dec 5, 2013
1 parent 6a7dcd8 commit f94319e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion test/src/main/java/hudson/cli/CLICommandInvoker.java
Expand Up @@ -26,13 +26,13 @@

import hudson.Extension;
import hudson.model.User;
import hudson.security.ACL;
import hudson.security.Permission;
import hudson.security.GlobalMatrixAuthorizationStrategy;

import java.io.ByteArrayOutputStream;
import java.io.InputStream;
import java.io.PrintStream;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
import java.util.List;
Expand Down Expand Up @@ -129,6 +129,8 @@ private void setAuth() {
rule.jenkins.setAuthorizationStrategy(auth);

command.setTransportAuth(user().impersonate());
// Otherwise it is SYSTEM, which would be relevant for a command overriding main:
ACL.impersonate(Jenkins.ANONYMOUS);
}

public User user() {
Expand Down

0 comments on commit f94319e

Please sign in to comment.