Skip to content

fix(desktop): trust native roots for relay websockets - #22

Merged
BrianInAz merged 1 commit into
mainfrom
hotfix/desktop-relay-connection-indicator
Aug 9, 2026
Merged

fix(desktop): trust native roots for relay websockets#22
BrianInAz merged 1 commit into
mainfrom
hotfix/desktop-relay-connection-indicator

Conversation

@BrianInAz

Copy link
Copy Markdown
Owner

Summary

  • configure native desktop WebSockets to use the operating system certificate store
  • align relay WebSocket trust behavior with the existing HTTP client
  • add an opt-in network regression test for OS-trusted WSS endpoints

Root cause

The HTTP relay queries succeeded because the HTTP client used platform certificate trust, while tokio-tungstenite was compiled with Mozilla WebPKI roots only. On systems where the relay chain is trusted through the OS certificate store, the native WSS handshake failed with UnknownIssuer. Cached/query data could still populate channels and DMs, leaving the sidebar to report that the relay was unreachable.

Validation

  • native WSS regression failed with UnknownIssuer before the feature change and passed afterward against wss://buzz.bjzy.me
  • native WebSocket module: 6 passed, 1 ignored
  • full Tauri test suite: 2,114 passed, 15 ignored
  • Clippy passed with warnings denied
  • Rust formatting and diff checks passed
  • rodio diagnostics: 3 passed
  • installed 0.5.3-bjzy acceptance build retained an established relay socket through multiple heartbeat intervals; channels/DMs loaded and the warning card remained absent

@cursor

cursor Bot commented Aug 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@BrianInAz
BrianInAz merged commit 5882061 into main Aug 9, 2026
23 checks passed
@BrianInAz
BrianInAz deleted the hotfix/desktop-relay-connection-indicator branch August 9, 2026 03:23
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