Skip to content

Commit

Permalink
Merge pull request #27 from Minh-Chien/master
Browse files Browse the repository at this point in the history
[STUDIO-2561] Fix SECURITY-2844 issue
  • Loading branch information
duyluonganh authored Sep 30, 2022
2 parents 9717106 + c09f66b commit 0ee4b34
Showing 1 changed file with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
import hudson.FilePath;
import hudson.Launcher;
import hudson.model.TaskListener;
import hudson.remoting.Callable;
import org.jenkinsci.remoting.RoleChecker;
import jenkins.security.MasterToSlaveCallable;

import java.util.HashMap;
import java.util.Map;
Expand All @@ -27,7 +26,7 @@ public static boolean executeKatalon(
String xvfbConfiguration) {
Logger logger = new JenkinsLogger(taskListener);
try {
return launcher.getChannel().call(new Callable<Boolean, Exception>() {
return launcher.getChannel().call(new MasterToSlaveCallable<Boolean, Exception>() {
@Override
public Boolean call() throws Exception {

Expand All @@ -54,9 +53,6 @@ public Boolean call() throws Exception {
}
return true;
}
@Override
public void checkRoles(RoleChecker roleChecker) throws SecurityException {
}
});
} catch (Exception e) {
String stackTrace = Throwables.getStackTraceAsString(e);
Expand Down

0 comments on commit 0ee4b34

Please sign in to comment.