BEE-52312 - SuggestedChanges

jenkinsci · Nov 12, 2024 · a1700c5 · a1700c5
1 parent d5d3cba
commit a1700c5
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/src/main/java/hudson/slaves/CommandLauncher.java b/src/main/java/hudson/slaves/CommandLauncher.java
@@ -101,12 +101,6 @@ public CommandLauncher(String command) throws Descriptor.FormException {
      * @param env       environment variables for the launcher to include when it runs the command
      */
     public CommandLauncher(String command, EnvVars env) {
-        try {
-            checkSandbox();
-        } catch (Descriptor.FormException ex) {
-            throw new RuntimeException(ex);
-        }
-
         this.agentCommand = command;
     	this.env = env;
         ScriptApproval.get().preapprove(command, SystemCommandLanguage.get());
@@ -305,11 +299,12 @@ public boolean filterType(@NonNull Class<?> contextClass, @NonNull Descriptor de
         }
 
         private boolean isCreatingNewObject() {
-            if (Stapler.getCurrentRequest() != null) {
-                List<Ancestor> ancs = Stapler.getCurrentRequest().getAncestors();
+            var req = Stapler.getCurrentRequest();
+            if (req != null) {
+                List<Ancestor> ancs = req.getAncestors();
                 for (Ancestor anc : ancs) {
                     if (anc.getObject() instanceof ComputerSet) {
-                        String uri = Stapler.getCurrentRequest().getOriginalRequestURI();
+                        String uri = req.getOriginalRequestURI();
                         if (uri.endsWith("createItem")) {
                             return true;
                         }