gavinchou commented on code in PR #69467:
URL: https://github.com/apache/doris/pull/69467#discussion_r4453585020
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/NeedAuditEncryption.java:
##########
@@ -36,9 +37,30 @@ default String geneEncryptionSQL(String sql) {
if (!needAuditEncryption()) {
return sql;
}
+ return maskSql(sql, false);
Review Comment:
@starocean999 Updated in 2dc978f137959ff19585c248ecd980a6ae06171d.
Unicode range normalization now happens in NereidsParser.parseForEncryption
for both formatting modes and direct streaming callers. The audit entry point
still uses false, preserving its property rendering and hidden-key behavior.
The mode-specific conversion was removed to avoid double conversion.
Added regression coverage for the default audit API, both modes,
supplementary characters before and inside credentials, and multiple
direct-parser ranges. All 58 focused FE tests and Checkstyle pass. CI has been
requested for the new commit. Please take another look.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
