Skip to content

Commit f868409

Browse files
authored
chore(pii): drop the over-long text example from the mask client's outage comment (#9989)
Presidio now masks texts over spaCy's length limit instead of answering 500, so the comment names a failure that still produces a deterministic 500.
1 parent 3c4c4de commit f868409

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

‎apps/sim/lib/guardrails/mask-client.ts‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,9 @@ function isRetryableChunkError(error: unknown): boolean {
9292
* Statuses that mean the service is down rather than that one request was bad:
9393
* the load balancer has no healthy target (502/503/504), or the mask-batch route
9494
* could not reach Presidio (503). A 500 is retried but never read as an outage —
95-
* Presidio answers 500 for deterministic per-input failures too (an over-long text
96-
* spaCy refuses), and one such chunk must not scrub the rest of the payload.
95+
* Presidio answers 500 for deterministic per-input failures too (an input that
96+
* raises inside a recognizer), and one such chunk must not scrub the rest of the
97+
* payload.
9798
*/
9899
const OUTAGE_STATUSES = new Set([502, 503, 504])
99100

0 commit comments

Comments
 (0)