Commit 80db7c4
committed
fix(config): add default catch-all decision to enable PII detection for all requests
This fixes the E2E PII detection test failures (0% detection rate) by ensuring
PII detection is always enabled, even when no specific decision matches.
Previously, requests with model='MoM' (used by E2E tests) did not match any
decision criteria, causing decisionName to be empty. This triggered the check:
if decisionName == '' { return false } // PII detection disabled
The fix adds a catch-all default_decision with:
- priority: 1 (lowest - matches only if nothing else does)
- type: 'always' (matches any request)
- pii_types_allowed: [] (blocks ALL PII for safety)
This ensures the 100 E2E PII test cases will be blocked correctly.
Fixes vllm-project#647 E2E test failures
Signed-off-by: Yossi Ovadia <[email protected]>1 parent e507895 commit 80db7c4
1 file changed
+18
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
359 | 359 | | |
360 | 360 | | |
361 | 361 | | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
| 375 | + | |
| 376 | + | |
| 377 | + | |
| 378 | + | |
| 379 | + | |
362 | 380 | | |
363 | 381 | | |
364 | 382 | | |
| |||
0 commit comments