Przeglądaj źródła

Script: Load Whitelists as Resource (#87539)

Changes PainlessPlugin.class.getResourceAsStream to
PainlessPlugin.class.getResource to avoid leaking unclosed
input streams
Stuart Tettemer 3 lat temu
rodzic
commit
b05ed2fd51

+ 5 - 0
docs/changelog/87539.yaml

@@ -0,0 +1,5 @@
+pr: 87539
+summary: "Script: Load Whitelists as Resource"
+area: Infra/Scripting
+type: bug
+issues: []

+ 1 - 1
modules/lang-painless/src/main/java/org/elasticsearch/painless/PainlessPlugin.java

@@ -90,7 +90,7 @@ public final class PainlessPlugin extends Plugin implements ScriptPlugin, Extens
 
         for (ScriptContext<?> context : ScriptModule.CORE_CONTEXTS.values()) {
             List<Whitelist> contextWhitelists = new ArrayList<>();
-            if (PainlessPlugin.class.getResourceAsStream("org.elasticsearch.script." + context.name.replace('-', '_') + ".txt") != null) {
+            if (PainlessPlugin.class.getResource("org.elasticsearch.script." + context.name.replace('-', '_') + ".txt") != null) {
                 contextWhitelists.add(
                     WhitelistLoader.loadFromResourceFiles(
                         PainlessPlugin.class,