From 87d66acdacca1e248c956cf4aa7675a2e357fc7a Mon Sep 17 00:00:00 2001 From: Ryan Ernst Date: Fri, 7 Feb 2025 11:51:26 -0800 Subject: [PATCH] Remove assertion about theoretical path separators (#122069) The only real path separators are either forward or back slash. Trying to use something else like newline fails to even parse as a path on windows. This commit removes testing of other separators. closes #121872 --- .../entitlement/runtime/policy/FileAccessTreeTests.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/libs/entitlement/src/test/java/org/elasticsearch/entitlement/runtime/policy/FileAccessTreeTests.java b/libs/entitlement/src/test/java/org/elasticsearch/entitlement/runtime/policy/FileAccessTreeTests.java index a79eeca3bcd7d..de3e2eafb7569 100644 --- a/libs/entitlement/src/test/java/org/elasticsearch/entitlement/runtime/policy/FileAccessTreeTests.java +++ b/libs/entitlement/src/test/java/org/elasticsearch/entitlement/runtime/policy/FileAccessTreeTests.java @@ -102,9 +102,6 @@ public void testForwardSlashes() { // Forward slashes also work assertThat(tree.canRead(path("a/b")), is(true)); assertThat(tree.canRead(path("m/n")), is(true)); - - // In case the native separator is a backslash, don't treat that as an escape - assertThat(tree.canRead(path("m\n")), is(false)); } FilesEntitlement entitlement(String... values) {