starred/windmill
1
0
Fork 0
mirror of https://github.com/windmill-labs/windmill.git synced 2026-10-30 08:16:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

perf: build log-mask automaton as a contiguous nfa (#12088)

Browse Source 
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Ruben Fiszel
2026-10-28 22:31:08 +01:00
committed by GitHub
co-authored by Claude Opus 5.5
parent d03142ba39
commit 97de072e7f
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/windmill-common/src/sensitive_log_masks.rs
+4
View File
@@ -184,7 +184,11 @@ fn compile(secrets: &HashSet<String>) -> CompiledMasks {
})
.collect();
// Built once per job (its token is always a secret) and searched over little log text, so
// the default DFA's build cost dominates; a contiguous NFA searches as fast here since
// the literal prefilter does the scanning.
let ac = aho_corasick::AhoCorasickBuilder::new()
.kind(Some(aho_corasick::AhoCorasickKind::ContiguousNFA))
.match_kind(aho_corasick::MatchKind::LeftmostLongest)
.build(sorted.iter().map(|s| s.as_str()))
.expect("failed to build aho-corasick automaton");