Revert "Fix tests failing due to JVM security exceptions (#41)"

This reverts commit 6369a78.
opensearch-project · Sep 12, 2024 · 0d4da5a · 0d4da5a
1 parent 6369a78
commit 0d4da5a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/build.gradle b/build.gradle
@@ -144,13 +144,8 @@ githubRelease.doFirst {
   }
 }
 
-tasks.withType(Test).configureEach { task ->
-    if (JavaVersion.current().compareTo(JavaVersion.VERSION_17) > 0) {
-        def policyFile = file("${projectDir}/src/main/plugin-metadata/plugin-security.policy").absolutePath
-        task.jvmArgs += [
-            "--add-opens", "java.base/java.lang=ALL-UNNAMED",
-            "-Djava.security.manager",
-            "-Djava.security.policy=" + policyFile
-        ]
-    }
+tasks.withType(Test) { task ->
+  if (JavaVersion.current().compareTo(JavaVersion.VERSION_17) > 0) {
+    task.jvmArgs += ["-Djava.security.manager=allow"]
+  }
 }
diff --git a/src/main/plugin-metadata/plugin-security.policy b/src/main/plugin-metadata/plugin-security.policy
@@ -20,11 +20,11 @@
 grant {
   // needed to generate runtime classes
   permission java.lang.RuntimePermission "createClassLoader";
-
+  
   // expression runtime
   permission org.opensearch.script.ClassPermission "java.lang.String";
   permission org.opensearch.script.ClassPermission "org.apache.lucene.expressions.Expression";
-  permission org.opensearch.script.ClassPermission "org.apache.lucene.expressions.js.ExpressionMath";
+  permission org.opensearch.script.ClassPermission "org.apache.lucene.expressions.js.ExpressionMath"
   permission org.opensearch.script.ClassPermission "org.apache.lucene.search.DoubleValues";
   // available functions
   permission org.opensearch.script.ClassPermission "java.lang.Math";