Skip to content

fix(browser): honor forwarded websocket host#29

Merged
taigrr merged 2 commits into
masterfrom
cd/browser-forwarded-host
May 11, 2026
Merged

fix(browser): honor forwarded websocket host#29
taigrr merged 2 commits into
masterfrom
cd/browser-forwarded-host

Conversation

@taigrr
Copy link
Copy Markdown
Owner

@taigrr taigrr commented May 6, 2026

Summary

  • use X-Forwarded-Host when building the browser viewer websocket URL
  • accept comma-separated forwarded headers by using the first proxy value
  • guard the package-level websocket upgrader and stabilize websocket tests that were racing the server setup

Testing

  • go test ./...
  • go test -race ./...
  • staticcheck ./...

@taigrr taigrr merged commit 1b292bd into master May 11, 2026
@taigrr taigrr deleted the cd/browser-forwarded-host branch May 11, 2026 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant