Skip to content

chore(pii): drop the over-long text example from the mask client's outage comment #9989

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
waleedlatif1 merged 1 commit into from
Nov 13, 2026
+3 −2
Merged

chore(pii): drop the over-long text example from the mask client's outage comment #9989

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions apps/sim/lib/guardrails/mask-client.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,8 +92,9 @@ function isRetryableChunkError(error: unknown): boolean {
* Statuses that mean the service is down rather than that one request was bad:
* the load balancer has no healthy target (502/503/504), or the mask-batch route
* could not reach Presidio (503). A 500 is retried but never read as an outage —
* Presidio answers 500 for deterministic per-input failures too (an over-long text
* spaCy refuses), and one such chunk must not scrub the rest of the payload.
* Presidio answers 500 for deterministic per-input failures too (an input that
* raises inside a recognizer), and one such chunk must not scrub the rest of the
* payload.
*/
const OUTAGE_STATUSES = new Set([502, 503, 504])

Expand Down
Loading