Skip to content

fix: honor default mask strategy in rust pii filter#23

Open
lucarlig wants to merge 9 commits intomainfrom
fix/pii-filter-default-mask-strategy
Open

fix: honor default mask strategy in rust pii filter#23
lucarlig wants to merge 9 commits intomainfrom
fix/pii-filter-default-mask-strategy

Conversation

@lucarlig
Copy link
Copy Markdown
Collaborator

@lucarlig lucarlig commented Apr 10, 2026

Summary

This fixes the Rust pii_filter package so built-in PII patterns honor default_mask_strategy instead of forcing hardcoded mask strategies.

Root Cause

Built-in pattern definitions in plugins/rust/python-package/pii_filter/src/patterns.rs stored explicit mask_strategy values, so detection never fell back to config.default_mask_strategy.

What Changed

  • Changed built-in Rust pattern definitions to inherit default_mask_strategy
  • Kept explicit mask-strategy overrides for custom patterns only
  • Updated Rust regression tests to verify global fallback behavior and masked output

Closes #22
Related to IBM/mcp-context-forge#3820

Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
Signed-off-by: lucarlig <luca.carlig@ibm.com>
@lucarlig lucarlig marked this pull request as ready for review April 10, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Rust pii_filter ignores default_mask_strategy for built-in patterns

1 participant