Repository navigation
docs: describe request inspection, and correct a limitation that stopped being true - #100
Merged
Merged
Conversation
…ped being true README said pasu does no L7 content inspection. It does now, and a limitations list that understates what a security tool inspects is as misleading as one that overstates it — a reader deciding whether pasu covers their case would decide wrongly in both directions. So that entry is split into what is actually true: the kernel layer is L3/L4 and cannot see inside TLS; the proxy layer inspects content, but only in request shapes it parses; detection is regex and checksums, so PII without a pattern is not caught; and it blocks rather than masks. The Inspector plugin point, the two new crates and the Presidio import were in no README, no docs page and no changelog entry. The CHANGELOG Unreleased section was empty. Both READMEs now carry a request-inspection section with the commands, and the changelog carries a Known gaps heading rather than only the additions. Every command in that section was run against the built binary rather than written from memory. Doing that caught one thing: on a rule file it refuses, the error told the operator to call read_lossy — a library function a person holding a CLI cannot call. The library no longer names a remedy, because its two callers have different moves available, and each now adds its own. The proxy says to remove the recognizers or lower --presidio-min-score, and prints the current value. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
README said pasu does no L7 content inspection. It does now.
A limitations list that understates what a security tool inspects misleads as badly as one that overstates it — a reader deciding whether pasu covers their case would decide wrongly in both directions.
That entry is now split into what is actually true:
What had no documentation at all
The
Inspectorplugin point,pasu-pii-krbecoming reachable, andpasu-inspect-presidioappeared in no README, no docs page, and no changelog entry.## [Unreleased]was empty.Both READMEs now carry a request-inspection section with the commands, why the request body is the only place content is readable, and what does not survive a Presidio import. The changelog carries a Known gaps heading, not only the additions.
Running the commands caught something
Every command in the new section was run against the built binary rather than written from memory. On a rule file it refuses, the error told the operator to call
read_lossy— a library function a person holding a CLI cannot call.The library no longer names a remedy. Its two callers have different moves available: one can call
read_lossy, the other can only change a flag or the file, and naming either is advice the other cannot take. Each now adds its own:And the documented workaround does what the docs say it does:
🤖 Generated with Claude Code