Handle case where requestedActions is null

cwperks · cwperks · commit 394c47a59a39 · 2024-09-19T14:47:34.000-04:00
Signed-off-by: Craig Perkins &lt;cwperx@amazon.com&gt;
diff --git a/server/src/main/java/org/opensearch/plugins/PluginInfo.java b/server/src/main/java/org/opensearch/plugins/PluginInfo.java
@@ -249,7 +249,11 @@ This works for currently supported range notations (=,~)
         out.writeStringCollection(extendedPlugins);
         out.writeBoolean(hasNativeController);
         if (out.getVersion().onOrAfter(Version.CURRENT)) {
-            Settings.writeSettingsToStream(requestedActions, out);
+            if (requestedActions != null) {
+                Settings.writeSettingsToStream(requestedActions, out);
+            } else {
+                Settings.writeSettingsToStream(Settings.EMPTY, out);
+            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,11 @@ This works for currently supported range notations (=,~)`
`249`	`249`	`out.writeStringCollection(extendedPlugins);`
`250`	`250`	`out.writeBoolean(hasNativeController);`
`251`	`251`	`if (out.getVersion().onOrAfter(Version.CURRENT)) {`
`252`		`- Settings.writeSettingsToStream(requestedActions, out);`
	`252`	`+ if (requestedActions != null) {`
	`253`	`+ Settings.writeSettingsToStream(requestedActions, out);`
	`254`	`+ } else {`
	`255`	`+ Settings.writeSettingsToStream(Settings.EMPTY, out);`
	`256`	`+ }`
`253`	`257`	`}`
`254`	`258`	`}`
`255`	`259`