Skip to content

fix: pass broker API key to WebSocket connection#73

Merged
khaliqgant merged 3 commits intomainfrom
fix/broker-ws-auth
Apr 7, 2026
Merged

fix: pass broker API key to WebSocket connection#73
khaliqgant merged 3 commits intomainfrom
fix/broker-ws-auth

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Apr 7, 2026

Summary

  • The broker's /ws endpoint sits behind the auth middleware, so the dashboard's hybrid WebSocket connection was rejected with no auth header
  • Pass RELAY_BROKER_API_KEY through handleHybridWebSocket and set it as an x-api-key header on the broker WebSocket connection

Test plan

  • Run agent-relay up and verify the dashboard WebSocket connects without auth errors
  • Confirm real-time agent events stream to the dashboard

🤖 Generated with Claude Code


Open with Devin

khaliqgant and others added 2 commits April 7, 2026 10:08
The broker's /ws endpoint is behind the auth middleware, so the
dashboard's hybrid WebSocket connection was getting rejected. Pass
RELAY_BROKER_API_KEY as an x-api-key header when connecting.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@khaliqgant khaliqgant merged commit 8d7fb2a into main Apr 7, 2026
1 check passed
@khaliqgant khaliqgant deleted the fix/broker-ws-auth branch April 7, 2026 08:14
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