Skip to content

fix: use correct websocket URL scheme (wss://) in hub configurations#1432

Merged
artemijspavlovs merged 1 commit intomainfrom
danwt/claude/fix-ws-urls
Dec 29, 2025
Merged

fix: use correct websocket URL scheme (wss://) in hub configurations#1432
artemijspavlovs merged 1 commit intomainfrom
danwt/claude/fix-ws-urls

Conversation

@danwt
Copy link
Contributor

@danwt danwt commented Dec 18, 2025

Summary

  • Fixed WsUrl fields in hub configurations to use proper WebSocket schemes
  • Changed https://wss:// for secure WebSocket connections
  • Changed http://ws:// for localhost

Context

Part of fix for #1414 ("roller reports tx failure, while it actually passed")

The WsUrl fields were incorrectly configured with HTTP(S) schemes instead of WebSocket schemes. This caused the CometBFT WebSocket client to fail immediately when attempting to connect, as it expected ws:// or wss:// URLs.

Changes

Environment Before After
Mainnet https://dymension.api.onfinality.io:443/public-ws wss://dymension.api.onfinality.io:443/public-ws
Blumbus https://rpc-blumbus.mzonder.com:443 wss://rpc-blumbus.mzonder.com:443
Local http://localhost:36657 ws://localhost:36657
Playground https://rpc-dymension-playground35.mzonder.com:443 wss://rpc-dymension-playground35.mzonder.com:443

Test plan

  • Verify WebSocket connections work with the new URLs
  • Test transaction monitoring on each environment

🤖 Generated with Claude Code

The WsUrl fields were incorrectly configured with https:// scheme
instead of wss:// (or ws:// for localhost). This caused WebSocket-based
transaction monitoring to fail immediately when attempting to connect.

Fixes part of #1414

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@danwt danwt requested a review from a team as a code owner December 18, 2025 14:57
@artemijspavlovs artemijspavlovs merged commit 39b987e into main Dec 29, 2025
2 checks passed
@artemijspavlovs artemijspavlovs deleted the danwt/claude/fix-ws-urls branch December 29, 2025 11:42
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.

2 participants