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

[Nakamoto] signer keys must be associated with PoX addresses, not stackers #4290

Closed
jcnelson opened this issue Jan 25, 2024 · 3 comments
Closed

Comments

@jcnelson
Copy link
Member

jcnelson commented Jan 25, 2024

Per #4271 and #4269, the reward set needs to bind PoX addresses to signing keys in a many-to-many relationship:

  • The reward set decides which signing keys act on behalf of a PoX address (with an associated "partial punishment" tactic for partial signer failures, described below)

  • The reward set decides which PoX addresses a signing key can sign for. A signing key failure would affect all its bound PoX addresses.

In the event that a PoX address has multiple signing keys, and some signing keys fail, then we deduct the proportional rewards by doing this:

Right now, the idea is to have PoX payouts to the address cease if the associated signing key has failed to produce a signature for K Bitcoin blocks, and resume if they have resumed signing for K Bitcoin blocks. Are you thinking that instead of stopping the payout altogether, we flip a weighted coin (e.g. using the VRF) on PoX payouts to the offending address such that the probability of the payout being forfeited is proportional to the fraction of STX represented by the misbehaving signing keys?

@setzeus
Copy link
Collaborator

setzeus commented Feb 2, 2024

Resurface during unblock, the problem is clear but requirements for Clarity / associated PoX-4 contracts a bit unclear.

@kantai
Copy link
Member

kantai commented Feb 3, 2024

This relationship is already implemented in next. The only requirements for PoX-4 not implemented are that signing keys need to approve any request that sets the signing key. This is captured in #4247 and implemented by #4277

@saralab saralab assigned jcnelson, kantai and hstove and unassigned jcnelson and kantai Feb 8, 2024
@hstove hstove closed this as completed Feb 9, 2024
@hstove
Copy link
Contributor

hstove commented Feb 9, 2024

Closed via #4277

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

5 participants