Skip to content

Conversation

@samholmes
Copy link
Contributor

@samholmes samholmes commented Jul 9, 2025

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

none

cursor[bot]

This comment was marked as outdated.

@paullinator paullinator force-pushed the sam/edge-blockbook-servers branch from 2d4b40e to 35235b5 Compare July 14, 2025 23:23
@paullinator paullinator enabled auto-merge July 14, 2025 23:23
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: WebSocket vs HTTP Endpoint Mismatch

The blockbookServers configuration was updated to wss://dash-eusa1.edge.app, but the corresponding engineInfo.serverConfigs still references https://dash-wusa1.edge.app. This creates an inconsistency between the WebSocket and HTTP endpoints for the Dash service, potentially leading to connectivity issues or accessing different regional servers.

src/common/utxobased/info/dash.ts#L42-L43

'wss://dash5.trezor.io',
'wss://dash-eusa1.edge.app',

Fix in CursorFix in Web


Bug: Litecoin Server Config Mismatch

Litecoin server configuration is inconsistent: defaultSettings.blockbookServers was updated to wss://ltc-eusa1.edge.app, but engineInfo.serverConfigs still references https://ltc-wusa1.edge.app. This mismatch between WebSocket and HTTP endpoints for the same Edge service (eusa1 vs wusa1 regions) could cause connectivity issues if the old wusa1 servers are decommissioned.

src/common/utxobased/info/litecoin.ts#L40-L41

'wss://ltc5.trezor.io',
'wss://ltc-eusa1.edge.app',

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎

@paullinator paullinator merged commit 2442171 into master Jul 14, 2025
3 checks passed
@paullinator paullinator deleted the sam/edge-blockbook-servers branch July 14, 2025 23:27
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.

3 participants