Skip to content

Commit

Permalink
removing jenkins security scan warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
PereBueno committed Jul 19, 2024
1 parent bdaf7a8 commit 15d9550
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -974,7 +974,7 @@ public FormValidation doTestConnection(
}

@RequirePOST
@SuppressWarnings("unused") // used by jelly
@SuppressWarnings({"unused", "lgtm[jenkins/csrf]"}) // used by jelly and already fixed jenkins security scan warning
public FormValidation doCheckSkipTlsVerify(@QueryParameter boolean skipTlsVerify) {
Jenkins.get().checkPermission(Jenkins.MANAGE);
try {
Expand All @@ -986,7 +986,7 @@ public FormValidation doCheckSkipTlsVerify(@QueryParameter boolean skipTlsVerify
}

@RequirePOST
@SuppressWarnings("unused") // used by jelly
@SuppressWarnings({"unused", "lgtm[jenkins/csrf]"}) // used by jelly and already fixed jenkins security scan warning
public FormValidation doCheckServerCertificate(@QueryParameter String serverCertificate) {
Jenkins.get().checkPermission(Jenkins.MANAGE);
try {
Expand Down

0 comments on commit 15d9550

Please sign in to comment.