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

fix(coinjoin): do not ban inputs locally after unsuccessful round #8855

Merged
merged 1 commit into from
Jul 4, 2023

Conversation

szymonlesisz
Copy link
Contributor

@szymonlesisz szymonlesisz commented Jul 4, 2023

Description

solve https://www.notion.so/satoshilabs/Coinjoin-55b6fb9efaa540e98a01473d62ef8337?p=89ed3a4a3ba34d52937dfe1d85ebb3ad&pm=s

Distinguish noted and banned inputs, do not ban inputs locally after unsuccessful round, try again after randomized time and detain (ban) input if coordinator say so

Related issue #8307

@szymonlesisz szymonlesisz added the coinjoin Related to coinjoin feature label Jul 4, 2023
Copy link
Contributor

@marekrjpolak marekrjpolak left a comment

Choose a reason for hiding this comment

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

Fire at will.

@szymonlesisz
Copy link
Contributor Author

/rebase

@github-actions
Copy link

github-actions bot commented Jul 4, 2023

@szymonlesisz
Copy link
Contributor Author

fixed unit tests (also output addresses are now detained)

Copy link
Contributor

@marekrjpolak marekrjpolak left a comment

Choose a reason for hiding this comment

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

Still good 👍

@szymonlesisz
Copy link
Contributor Author

/rebase

@github-actions
Copy link

github-actions bot commented Jul 4, 2023

@matejkriz matejkriz merged commit e2a5d3e into develop Jul 4, 2023
@matejkriz matejkriz deleted the fix/cj-noted-inputs branch July 4, 2023 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coinjoin Related to coinjoin feature
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants