diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-06-03 00:35:02 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-06-03 00:35:02 +0000 |
| commit | eb089ef4c9416bd3d5642bb5a4437355ffa2fdd0 (patch) | |
| tree | 8e4fbd833defa346f506c95a80345b3ab0fc379a /internal/handlers/claudomator_proxy.go | |
| parent | 3a9263abd0a5b2b759a8f691f001ff71f5417748 (diff) | |
fix: add CSRF and WebSocket origin protection to Claudomator proxy
Token-based CSRF is impractical for a reverse proxy whose UI doesn't
know Doot's session tokens, so state-changing requests to /claudomator/*
are now validated against the configured WebAuthn origin via Origin/Referer
header. WebSocket upgrades reject mismatched or missing Origin headers
before hijacking the connection. Both checks are no-ops when WebAuthnOrigin
is unset (local dev).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/handlers/claudomator_proxy.go')
| -rw-r--r-- | internal/handlers/claudomator_proxy.go | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/internal/handlers/claudomator_proxy.go b/internal/handlers/claudomator_proxy.go index bfbbabc..f3f0cd2 100644 --- a/internal/handlers/claudomator_proxy.go +++ b/internal/handlers/claudomator_proxy.go @@ -13,7 +13,7 @@ import ( // targetURL, stripping the "/claudomator" prefix from the path. WebSocket // upgrade requests are handled via raw TCP hijacking to support long-lived // connections. -func NewClaudomatorProxy(targetURL string) http.Handler { +func NewClaudomatorProxy(targetURL, allowedOrigin string) http.Handler { target, err := url.Parse(targetURL) if err != nil { panic("claudomator: invalid target URL: " + err.Error()) @@ -50,6 +50,10 @@ func NewClaudomatorProxy(targetURL string) http.Handler { return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { if strings.EqualFold(r.Header.Get("Upgrade"), "websocket") { + if allowedOrigin != "" && r.Header.Get("Origin") != allowedOrigin { + http.Error(w, "forbidden", http.StatusForbidden) + return + } proxyWebSocket(w, r, target) return } |
