Skip to content

List accounts using all endpoints and persist the endpoints for correct signing - #16

Merged
AntiD2ta merged 6 commits into
masterfrom
endpoints-accounts-fix
Jan 15, 2026
Merged

AntiD2ta merged 6 commits into
masterfrom
endpoints-accounts-fix

Conversation

@AntiD2ta

@AntiD2ta AntiD2ta commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

This PR addresses critical issues with endpoint management and account persistence in distributed Dirk wallet setups that could occur in production distributed Dirk wallet setups where multiple endpoints serve different accounts. The changes ensure proper endpoint tracking for accounts and improve reliability when listing accounts across multiple endpoints.

Changes

Account Listing Improvements

  • Problem: Account listing only queried a single endpoint, potentially missing accounts on other endpoints in distributed setups.
  • Solution: Modified account listing to query all available endpoints and deduplicate results.

Endpoint Persistence for Correct Signing

  • Problem: Accounts were not properly maintaining references to their originating endpoints, which could cause signing operations to fail in distributed setups.
  • Solution: Modified account creation/retrieval logic to persist endpoint information with each account.
  • Impact: Ensures signing operations use the correct endpoint associated with each account.

Golangci-lint Migration

Migrated to golangci-lint v2 and updated code to pass new linting requirements.

@AntiD2ta AntiD2ta self-assigned this Jan 8, 2026
Comment thread mock/signerserver.go Outdated
@AntiD2ta
AntiD2ta merged commit 68a233b into master Jan 15, 2026
2 checks passed
@AntiD2ta
AntiD2ta deleted the endpoints-accounts-fix branch January 15, 2026 13:54
@AntiD2ta
AntiD2ta restored the endpoints-accounts-fix branch January 15, 2026 13:54
@AntiD2ta
AntiD2ta deleted the endpoints-accounts-fix branch January 15, 2026 14:33
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