Skip to content

fix: skip disconnect for OKX on connect failure#11

Merged
satoshai-dev merged 2 commits intomainfrom
fix/okx-disconnect-wrong-provider
Feb 25, 2026
Merged

fix: skip disconnect for OKX on connect failure#11
satoshai-dev merged 2 commits intomainfrom
fix/okx-disconnect-wrong-provider

Conversation

@satoshai-dev
Copy link
Copy Markdown
Owner

Summary

  • OKX uses its own SDK directly and never calls setSelectedProviderId, so on connect failure getSelectedProvider() would return the previously connected provider
  • Guard the catch block to only disconnect/clear for non-OKX providers

Closes #6

🤖 Generated with Claude Code

Closes #6

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@satoshai-dev satoshai-dev changed the base branch from feat/update-stacks-connect-8.2.5 to main February 25, 2026 15:03
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@satoshai-dev satoshai-dev merged commit b8eeec9 into main Feb 25, 2026
1 check passed
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.

OKX connect failure disconnects wrong provider

1 participant