Skip to content

Commit 762a84f

Browse files
committed
ok but do the check right this time
1 parent d59d0f9 commit 762a84f

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/IndicesAndAliasesResolver.java

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -285,12 +285,9 @@ ResolvedIndices resolveIndicesAndAliases(
285285
*/
286286
assert indicesRequest.indices() == null || indicesRequest.indices().length == 0
287287
: "indices are: " + Arrays.toString(indicesRequest.indices()); // Arrays.toString() can handle null values - all good
288-
String indexName = ((PutMappingRequest) indicesRequest).getConcreteIndex().getName();
289-
if (authorizedIndices.check(indexName)) {
290-
resolvedIndicesBuilder.addLocal(
291-
getPutMappingIndexOrAlias((PutMappingRequest) indicesRequest, authorizedIndices::check, metadata)
292-
);
293-
}
288+
resolvedIndicesBuilder.addLocal(
289+
getPutMappingIndexOrAlias((PutMappingRequest) indicesRequest, authorizedIndices::check, metadata)
290+
);
294291
} else if (indicesRequest instanceof final IndicesRequest.Replaceable replaceable) {
295292
final IndicesOptions indicesOptions = indicesRequest.indicesOptions();
296293

0 commit comments

Comments
 (0)