From 7880d5f93145b74ff18c16dc81cd3a7c89a20d64 Mon Sep 17 00:00:00 2001 From: David Buzinski Date: Wed, 27 Aug 2025 10:37:20 -0400 Subject: [PATCH 1/2] fix security finding from codeql scan --- src/main/java/com/mathworks/ci/tools/MatlabInstaller.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java b/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java index 53378d4b..0c666107 100644 --- a/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java +++ b/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java @@ -36,6 +36,7 @@ import org.kohsuke.stapler.DataBoundConstructor; import org.kohsuke.stapler.DataBoundSetter; import org.kohsuke.stapler.QueryParameter; +import org.kohsuke.stapler.verb.POST; public class MatlabInstaller extends ToolInstaller { @@ -292,6 +293,7 @@ public boolean isApplicable(Class toolType) { return toolType == MatlabInstallation.class; } + @POST public FormValidation doCheckRelease(@QueryParameter String value) { Jenkins.get().checkPermission(Jenkins.ADMINISTER); if (value.isEmpty()) { From af33b22345bca0126a08885fca6881a97686f26b Mon Sep 17 00:00:00 2001 From: David Buzinski Date: Wed, 27 Aug 2025 10:39:46 -0400 Subject: [PATCH 2/2] fix indent --- src/main/java/com/mathworks/ci/tools/MatlabInstaller.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java b/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java index 0c666107..2cfebdd1 100644 --- a/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java +++ b/src/main/java/com/mathworks/ci/tools/MatlabInstaller.java @@ -293,7 +293,7 @@ public boolean isApplicable(Class toolType) { return toolType == MatlabInstallation.class; } - @POST + @POST public FormValidation doCheckRelease(@QueryParameter String value) { Jenkins.get().checkPermission(Jenkins.ADMINISTER); if (value.isEmpty()) {