Skip to content

fix: update Insight API URLs for sweep balance#605

Merged
bfoss765 merged 1 commit intomasterfrom
fix/sweep-balance-url
Mar 3, 2026
Merged

fix: update Insight API URLs for sweep balance#605
bfoss765 merged 1 commit intomasterfrom
fix/sweep-balance-url

Conversation

@bfoss765
Copy link
Copy Markdown
Contributor

@bfoss765 bfoss765 commented Feb 27, 2026

Summary

  • Update INSIGHT_URL from insight.dash.org/insight-api-dash (404) to insight.dash.org/insight-api (200)
  • Update INSIGHT_FAILOVER_URL from insight.dash.show/api (unreachable) to insight.dash.org/insight-api (200)
  • Update TESTNET_INSIGHT_URL from testnet-insight.dashevo.org/insight-api-dash (502) to insight.testnet.networks.dash.org/insight-api (200)

Context

Users reported "Couldn't sweep balance - Unexpected response from (null)" when using Import Private Key. All three Insight API endpoints were returning errors (404, connection refused, 502).

Test plan

  • Import a private key on mainnet and verify sweep balance succeeds
  • Import a private key on testnet and verify sweep balance succeeds
  • Verify UTXO lookups return valid responses

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes
    • Updated blockchain network endpoints for improved service reliability and connectivity.

The old Insight API endpoints are no longer valid, causing "Couldn't
sweep balance" errors when importing private keys. Update all three
URLs to their current endpoints.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8e36a58 and e272a36.

📒 Files selected for processing (1)
  • DashSync/shared/Models/Managers/Service Managers/DSInsightManager.m

📝 Walkthrough

Walkthrough

Three Insight API endpoint URLs are updated in the DSInsightManager configuration file. The changes redirect the primary, failover, and testnet API calls to updated endpoint addresses, with modifications to both the domain and API path structures.

Changes

Cohort / File(s) Summary
Insight API Endpoint URLs
DashSync/shared/Models/Managers/Service Managers/DSInsightManager.m
Updated three API endpoint URL constants: INSIGHT_URL, INSIGHT_FAILOVER_URL, and TESTNET_INSIGHT_URL to point to new endpoint addresses with modified domain names and API paths.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Hop, hop, hop—new paths we take,
Through insight.dash our queries make,
From old domains to fresh new routes,
The API endpoints bear sweet fruits! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and specifically describes the main change: updating Insight API URLs to fix sweep balance functionality that was failing due to invalid endpoints.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/sweep-balance-url

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bfoss765 bfoss765 merged commit 9d962f2 into master Mar 3, 2026
1 of 8 checks passed
@bfoss765 bfoss765 deleted the fix/sweep-balance-url branch March 3, 2026 18:34
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