Merge pull request #68 from jenkinsci-cert/SECURITY-273

[FIX SECURITY-273] Require admin permission to check update sites
This commit is contained in:
Jesse Glick committed 2016-04-26 15:57:12 -04:00
commit 53dd9046c9
1 file changed
+1
@@ -890,6 +890,7 @@ public abstract class PluginManager extends AbstractModelObject implements OnMas
@Restricted(NoExternalUse.class)
@RequirePOST public HttpResponse doCheckUpdatesServer() throws IOException {
Jenkins.getInstance().checkPermission(Jenkins.ADMINISTER);
for (UpdateSite site : Jenkins.getInstance().getUpdateCenter().getSites()) {
FormValidation v = site.updateDirectlyNow(DownloadService.signatureCheck);
if (v.kind != FormValidation.Kind.OK) {