diff options
| author | srdusr <[email protected]> | 2026-05-20 09:29:00 +0200 |
|---|---|---|
| committer | srdusr <[email protected]> | 2026-05-20 09:29:00 +0200 |
| commit | 51811b67018515366bd56f3c3b21aed11d906db2 (patch) | |
| tree | 86f0bb0a95be73e7e6ca45ce82c3989932d82844 /internal/proxy | |
| parent | 1ac926d9936947e7b7e1cadaf2fcdeeda52b8c83 (diff) | |
| download | mitmux-51811b67018515366bd56f3c3b21aed11d906db2.tar.gz mitmux-51811b67018515366bd56f3c3b21aed11d906db2.zip | |
Target scope: filter what gets recorded, not what gets proxied
The proxy captured and stored literally everything with no way to
exclude unrelated traffic - every CDN asset, analytics beacon, and
third-party tracker request on a real engagement pollutes history and
search right alongside the traffic that actually matters.
internal/scope: Rule{Enabled, Pattern, IsRegex} and InScope(rules,
host). A non-regex pattern matches by case-insensitive substring
against the host - "example.com" matches "example.com",
"www.example.com", and "api.example.com" alike, covering "this domain
and its subdomains" without inventing a separate wildcard syntax.
IsRegex mirrors the same toggle match-and-replace rules already use,
for one consistent mental model across both rule types in this tool.
An empty or all-disabled rule set means everything is in scope - the
behavior before scope existed at all, unchanged, so a fresh install or
a user who never opens the scope view keeps recording everything
rather than silently nothing.
Deliberately a recording filter, not access control: out-of-scope
traffic still proxies completely normally, reaching its destination and
the client exactly as before. internal/proxy's forward() already writes
the response to the client before record() ever runs, so the scope
check (new in record()) can only affect whether the exchange gets
stored, never whether it happens. Blocking out-of-scope traffic outright
would be a materially different, much riskier feature - a wrong scope
pattern could silently break the very traffic someone's trying to test,
which is a far worse failure mode than a noisier history. Repeat/Intrude
(recordRaw, a separate function from record()) deliberately don't go
through the scope check at all: a user explicitly resending or fuzzing
a specific request wants to see the result regardless of scope, which
exists to cut passive-capture noise, not second-guess a deliberate
action.
internal/store: new scope_rules table (CREATE TABLE IF NOT EXISTS, no
migration needed - it's a new table, not a new column on an existing
one) plus List/Add/SetEnabled/Delete, mirroring the existing
match-and-replace rules CRUD exactly. internal/ipc: scope_list/
scope_add/scope_delete/scope_toggle request types and matching Client
methods; scope_add validates a regex pattern compiles before persisting,
same reasoning and same fix as the earlier rules_save validation (an
invalid regex should be rejected up front, not silently never match at
apply time with zero feedback).
TUI: 's' from the history list opens scope management, mirroring the
Rules view's own list+form pattern but simpler (add-only, no
edit-in-place - a pattern and a regex toggle don't need a five-field
form, delete-and-re-add covers changing one).
Verified live in tmux against a running daemon: added a substring scope
rule for one host, sent requests to both a matching and a non-matching
host - the non-matching one proxied successfully (client got its 200)
but was never recorded, the matching one was recorded normally;
confirmed a Repeater resend of the excluded host WAS recorded despite
being out of scope; toggled the rule off and confirmed recording
resumed for everything; added and confirmed a regex-mode rule saves and
displays correctly; deleted a rule and confirmed the list returns to
empty ("no rules means everything is recorded").
go build/vet/gofmt/test/mod tidy all clean.
Diffstat (limited to 'internal/proxy')
| -rw-r--r-- | internal/proxy/proxy.go | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/internal/proxy/proxy.go b/internal/proxy/proxy.go index 216e947..2e80ed2 100644 --- a/internal/proxy/proxy.go +++ b/internal/proxy/proxy.go @@ -38,6 +38,7 @@ import ( "mitmux/internal/ca" "mitmux/internal/rules" + "mitmux/internal/scope" "mitmux/internal/store" ) @@ -509,6 +510,18 @@ func (s *Server) record(started time.Time, duration time.Duration, scheme, host if s.store == nil { return } + // Scope only filters what gets recorded here - the request has + // already been forwarded and its response already written to the + // client by the time record() runs (see forward()), so an + // out-of-scope host still proxies completely normally, it just + // doesn't clutter history. Repeat/Intrude (recordRaw, a different + // function) deliberately don't go through this check: a user + // explicitly resending or fuzzing a specific request wants to see + // the result regardless of scope, which exists to cut passive- + // capture noise, not to second-guess a deliberate action. + if scopeRules, err := s.store.ListScopeRules(); err == nil && !scope.InScope(scopeRules, host) { + return + } e := &store.Entry{ StartedAt: started, |