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

include mixing and tickerbuyer info in walletinfo #2165

Open
chappjc opened this issue Jun 3, 2022 · 2 comments
Open

include mixing and tickerbuyer info in walletinfo #2165

chappjc opened this issue Jun 3, 2022 · 2 comments

Comments

@chappjc
Copy link
Member

chappjc commented Jun 3, 2022

For DEX we would like to know:

  1. what accounts are configured as the mixedaccount and changeaccount
  2. if mixing is actually enabled (csppserver && mixchange, not with the ticker buyer)
  3. If the tickerbuyer is running (enableticketbuyer) because we don't want it running
  4. If tickerbuyer is running, the purchaseaccount (in the off chance it's not the mixed account that DEX uses)

I think it would be ideal to have the above information available either via walletinfo or another RPC. Ideally, not one of the RPCs that has to compute balance (e.g. listaccounts or getbalance) because for large wallets these are relatively slow... definitely not getstakeinfo.

The ideal configuration for DEX is mixing configured with just mixchange, not ticker buyer (we cannot feasibly fund orders from a mixed account that is automatically spending coins for tickets). So we want to discover this as well as confirm which accounts are actually mixing/change/normal.

CC @itswisdomagain

A JSON-RPC request to stop the ticker buyer would also be of use.

@jrick
Copy link
Member

jrick commented Jun 3, 2022

Is this intended for use with a user-controlled cli wallet? I would imagine with decrediton, that since all of these details are managed by decrediton, they could also be provided to the dex library.

@chappjc
Copy link
Member Author

chappjc commented Jun 3, 2022

No Decrediton involvement. It's not required to run DEX.

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

No branches or pull requests

2 participants