diff --git a/src/main/java/org/elasticsearch/bootstrap/ESPolicy.java b/src/main/java/org/elasticsearch/bootstrap/ESPolicy.java index e0d8dd873bf..cbe00b252a4 100644 --- a/src/main/java/org/elasticsearch/bootstrap/ESPolicy.java +++ b/src/main/java/org/elasticsearch/bootstrap/ESPolicy.java @@ -43,7 +43,7 @@ public class ESPolicy extends Policy { this.dynamic = dynamic; } - @Override @SuppressForbidden(reason = "I know what i am doing") + @Override @SuppressForbidden(reason = "fast equals check is desired") public boolean implies(ProtectionDomain domain, Permission permission) { // run groovy scripts with no permissions if ("/groovy/script".equals(domain.getCodeSource().getLocation().getFile())) { diff --git a/src/test/java/org/elasticsearch/script/GroovySecurityTests.java b/src/test/java/org/elasticsearch/script/GroovySecurityTests.java index 23b78ac9b6b..c89d6187ddb 100644 --- a/src/test/java/org/elasticsearch/script/GroovySecurityTests.java +++ b/src/test/java/org/elasticsearch/script/GroovySecurityTests.java @@ -41,7 +41,7 @@ public class GroovySecurityTests extends ElasticsearchIntegrationTest { @Override public void setUp() throws Exception { super.setUp(); - assumeTrue("security manager is enabled", System.getSecurityManager() != null); + assumeTrue("test requires security manager to be enabled", System.getSecurityManager() != null); } @Test