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

Improve explanation of Identifiable Aborts #26

Closed
real-or-random opened this issue Jun 8, 2022 · 3 comments
Closed

Improve explanation of Identifiable Aborts #26

real-or-random opened this issue Jun 8, 2022 · 3 comments
Assignees

Comments

@real-or-random
Copy link
Collaborator

  • For example, it currently doesn't say anything about blaming the aggregator. If we keep the InvalidContributionError, then we should elaborate in the text.
  • Also, it currently says "This standard specifies a partial signature verification algorithm to identify disruptive signers. It is incompatible with third-party nonce aggregation because the individual nonce is required for partial verification." That's not entirely true. The aggregator can still identify disruptive parties. We could also refer to ROAST now for a more formal discussion of IA.
  • "All the aggregator node can do is prevent the signing session from succeeding by sending out incorrect aggregate nonces." It can also send a wrong final sig.

Originally posted by @real-or-random in #9 (comment)

@real-or-random real-or-random self-assigned this Jun 8, 2022
@real-or-random
Copy link
Collaborator Author

Let's first try to merge #21, then I can take care of this.

@real-or-random
Copy link
Collaborator Author

The first two points have been solved by #67.

"All the aggregator node can do is prevent the signing session from succeeding by sending out incorrect aggregate nonces." It can also send a wrong final sig.

Indeed, and at the moment we don't mention at all that that the aggregator can also be used in the second round. We should probably do this.

@real-or-random
Copy link
Collaborator Author

Remaining point solved by #70

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

1 participant