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

Introduce mixsplitlimit config option #2107

Merged
merged 1 commit into from Dec 17, 2021
Merged

Conversation

jrick
Copy link
Member

@jrick jrick commented Nov 15, 2021

This allows changing the previously-hardcoded value of 10 allowed CSPP
client connections per any change downmixing amount.

Copy link
Member

@chappjc chappjc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My testnet wallet is moving along with this set to 24 right now.
Although I don't understand the wording "per change amount" in the config help message for mixsplitlimit

@jrick
Copy link
Member Author

jrick commented Nov 15, 2021

perhaps "CoinShuffle++ server connection limit per mixed output value" but this only applies to the account downmixing, and not the mixes that occur when buying the tickets.

I agree that we need something better though, as not all uses of the account mixing is for ticket change

This allows changing the previously-hardcoded value of 10 allowed CSPP
client connections per any change downmixing amount.
@jrick jrick merged commit f4a9e9f into decred:master Dec 17, 2021
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.

None yet

2 participants