Suppress Jenkins Security Scan warnings (#134)

Co-authored-by: Daniel Beck <daniel-beck@users.noreply.github.com>
jenkinsci · Apr 17, 2023 · f39d652 · f39d652
1 parent 1723c7a
commit f39d652
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/main/java/hudson/security/GlobalMatrixAuthorizationStrategy.java b/src/main/java/hudson/security/GlobalMatrixAuthorizationStrategy.java
@@ -217,8 +217,9 @@ protected GlobalMatrixAuthorizationStrategy create() {
             return new GlobalMatrixAuthorizationStrategy();
         }
 
+        @SuppressWarnings("lgtm[jenkins/csrf]")
         @Restricted(NoExternalUse.class)
-        public FormValidation doCheckName(@QueryParameter String value ) {
+        public FormValidation doCheckName(@QueryParameter String value) {
             return doCheckName_(value, Jenkins.get(), Jenkins.ADMINISTER);
         }
 

diff --git a/src/main/java/org/jenkinsci/plugins/matrixauth/AuthorizationMatrixNodeProperty.java b/src/main/java/org/jenkinsci/plugins/matrixauth/AuthorizationMatrixNodeProperty.java
@@ -179,6 +179,7 @@ public String getDisplayName() {
         }
 
         @Restricted(DoNotUse.class)
+        @SuppressWarnings("lgtm[jenkins/csrf]")
         public FormValidation doCheckName(@AncestorInPath Computer computer, @QueryParameter String value) {
             // Computer isn't a DescriptorByNameOwner before Jenkins 2.78, and then @AncestorInPath doesn't work
             return doCheckName_(value,