Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show both client config and local config bitcoin nodes in guardian UI #366

Open
wbobeirne opened this issue Jan 30, 2024 · 2 comments
Open
Assignees

Comments

@wbobeirne
Copy link
Collaborator

This is a common point of confusion, there are 2 bitcoin nodes that get configured but we only show one of them. We should probably show both, and distinguish which is which.

Somewhat related is fedimint/fedimint#3531 for configuring both.

@elsirion
Copy link
Contributor

elsirion commented Feb 2, 2024

For context: one is used for consensus, this is typically a local bitcoind, while the other is given to clients to fetch the deposit status of their transactions from. That one is typically an esplora instance rn because we don't support electrum for that (incompatible merkle proof formats that I never got around to converting into each other fedimint/fedimint#4223) and bitcoind isn't really built for being a public API service.

@Kodylow Kodylow self-assigned this Aug 10, 2024
@Kodylow
Copy link
Member

Kodylow commented Aug 10, 2024

Note: local config isn't even exposed to the admin UI, needs a new api endpoint to expose it. In the meantime if this is just the client info it probably shouldn't even be exposed at all in the gaurdian UI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants