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

PrivateSend Enhancement: Up default round count to 4 and allow user to mix up to 16 rounds #2128

Merged
merged 2 commits into from
Jul 12, 2018

Conversation

PastaPastaPasta
Copy link
Member

@PastaPastaPasta PastaPastaPasta commented Jun 17, 2018

This is intended for 12.4
This PR consists of 2 commits
1:
Allows a user to mix up to 16 rounds
It appears the entirety of the codebase supports up to 16 rounds, except for the form where you enter how many rounds you would like it to do. This commit changes that, and allows users to use the core wallet to mix up to 16 rounds. This will potentially increase the upper bound of privacy for users who are highly concerned, and could increase liquidity in general.

2:
Ups the default for PS mixing to 4 rounds. fixes #2055
Although 2 rounds of private send mixing does have its use case and is fine to use depending on the situation, it does not make a good default. Although the recent improvements to PrivateSend could make 2 rounds more viable and private, 2 rounds is generally accepted as not private to a sufficient degree. As such, myself and many community members believe that the default should be increased to not give users a false sense of security.

PastaPastaPasta and others added 2 commits May 24, 2018 01:00
All codebase appears to accept up to 16 rounds, however this form was capped at 8; restricting users to 2-8 rounds. This commit allows a user to actually use up to 16 rounds in practice. This will greatly enhance the privacy of those very worried about their privacy, as well as possibly increasing liquidity and increasing the privacy of all users.
@UdjinM6 UdjinM6 added this to the 12.4 milestone Jun 17, 2018
@UdjinM6
Copy link

UdjinM6 commented Jun 17, 2018

Assigned 12.4 for now though this is a bit more complex issue than it seems at the first sight.

When wallet is starting for the first time users are prompted with a config dialog (DarksendConfig) which allows them to pick 16 rounds as a "Maximum Privacy" option. Users who picked less aggressive settings initially and who need more rounds (up to 16) later, can override GUI limits with cmd-line or dash.conf via privatesendrounds param. Maybe it's ok to let everyone use 16 rounds without learning about dash.conf and all the advanced options but I'm not sure about that tbh.

Regarding default number of rounds, there are 3 options in DarksendConfig (which set initial settings) - 2, 8 and 16 rounds respectively i.e. changes for DEFAULT_PRIVATESEND_ROUNDS in this PR won't have any effect for GUI users actually. DarksendConfig needs some refactoring (e.g. that's what I have in my local repo UdjinM6@b1e0a7b) or should be removed completely for these settings to take effect.

@PastaPastaPasta
Copy link
Member Author

PastaPastaPasta commented Jun 17, 2018

@UdjinM6 I have used multiple installs of the core wallet, quite often. And I have never seen this DarksendConfig in action. I have installed core 12.2.3 on my laptop which has never seen any blockchain before, and still I do not see any pop-up when the wallet has first been launched.

In regards to the default, I had a local branch which had more PS changes than this commit has, but I am fairly sure that that branch (when compiled locally) did have the default at 4, and did appear to allow users to mix up to 16 rounds

My VM for compiling is having problems, but once I get those smoothed out I'll test this branch and see how it acts

@UdjinM6
Copy link

UdjinM6 commented Jun 17, 2018

Hmm... That's interesting... Looks like the code that should open DarksendConfig is broken. Congrats, you've found a bug! 🎉 😄

@PastaPastaPasta
Copy link
Member Author

Yeah, I just assumed that is was deprecated

@UdjinM6 UdjinM6 mentioned this pull request Jun 18, 2018
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK for 12.4

@UdjinM6 UdjinM6 merged commit 43091a3 into dashpay:develop Jul 12, 2018
@PastaPastaPasta PastaPastaPasta deleted the ps-upDefault-upMax branch July 13, 2018 02:19
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Apr 25, 2019
…o mix up to 16 rounds (dashpay#2128)

* Allows a user to mix up to 16 rounds

All codebase appears to accept up to 16 rounds, however this form was capped at 8; restricting users to 2-8 rounds. This commit allows a user to actually use up to 16 rounds in practice. This will greatly enhance the privacy of those very worried about their privacy, as well as possibly increasing liquidity and increasing the privacy of all users.

* Up the default for PS mixing to 4 rounds.
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