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

Signer changing TSS account #1072

Open
JesseAbram opened this issue Sep 27, 2024 · 2 comments
Open

Signer changing TSS account #1072

JesseAbram opened this issue Sep 27, 2024 · 2 comments
Assignees
Labels
Fix something that's not working as intended

Comments

@JesseAbram
Copy link
Member

Currently there is a btree map for tss_account as the key when signing in synedrion. There is also a way for a TSS to change their TSS account . If they are a signer and do this, this will break signing with them. This can be fixed if everyone updates their TSS key for them. Either a pre lookup to get their tss key and check and change them in the key share before signing if this is possible @fjarri , if so that is cheap and can happen wtv. If not then maybe a reshare triggered with a new TSS...... or the same one but if they do it too much then maybe slash @HCastano

@ameba23
Copy link
Contributor

ameba23 commented Sep 27, 2024

The problem with changing the account id in the keyshare, is that all other keyshares in the set have a list of the other party ids - so we would have to change it in all the keyshares.

The other problem is that if a validator wants to change their TSS account ID because they are moving the physical location of where their TS server is run, they would anyway have no way of bringing the keyshare with them.

I think the simplest solution would be to not allow chainging TSS account ID whilst being a signer.

Another option would be that changing TSS account ID triggers a reshare, replacing the keyshare associated with the old TS account with the new one.

@JesseAbram
Copy link
Member Author

yes both those are options, however, why can't before sign I do a lookup of all signers on the chain, get their TSS accounts, check it against the TSS accounts in the keyshare, and replace the ones that are not there with the new ones?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix something that's not working as intended
Projects
Status: 📋 Backlog
Development

No branches or pull requests

4 participants