Skip to content

Commit cc32b8c

Browse files
committed
fix minor anonymizer bug
Signed-off-by: xinyual <xinyual@amazon.com>
1 parent c30d5d0 commit cc32b8c

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

ppl/src/main/java/org/opensearch/sql/ppl/utils/PPLQueryDataAnonymizer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,7 @@ public String visitTableFunction(TableFunction node, String context) {
251251
public String visitSearch(Search node, String context) {
252252
String source = node.getChild().get(0).accept(this, context);
253253
String queryString = node.getQueryString();
254-
String anonymized = queryString.replaceAll(":\\S+", ":" + MASK_LITERAL);
254+
String anonymized = queryString.replaceAll(":\\S+", "=" + MASK_LITERAL);
255255
return StringUtils.format("%s %s", source, anonymized);
256256
}
257257

ppl/src/test/java/org/opensearch/sql/ppl/utils/PPLQueryDataAnonymizerTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public class PPLQueryDataAnonymizerTest {
3333

3434
@Test
3535
public void testSearchCommand() {
36-
assertEquals("source=table a:***", anonymize("search source=t a=1"));
36+
assertEquals("source=table a=***", anonymize("search source=t a=1"));
3737
}
3838

3939
@Test
@@ -795,7 +795,7 @@ private String anonymizeStatement(String query, boolean isExplain) {
795795
@Test
796796
public void testSearchWithAbsoluteTimeRange() {
797797
assertEquals(
798-
"source=table (@timestamp:*** AND (@timestamp:***",
798+
"source=table (@timestamp=*** AND (@timestamp=***",
799799
anonymize("search source=t earliest='2012-12-10 15:00:00' latest=now"));
800800
}
801801

0 commit comments

Comments
 (0)