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

Block tss chain when signer #1078

Merged
merged 3 commits into from
Sep 30, 2024
Merged

Block tss chain when signer #1078

merged 3 commits into from
Sep 30, 2024

Conversation

JesseAbram
Copy link
Member

Related #1072

Fixes 1072 but maybe not the best UX, open to leave the issue open and changes it, but it closes the vulnerability and was hella fast to implement

@JesseAbram JesseAbram requested review from ameba23 and HCastano and removed request for ameba23 September 30, 2024 15:51
@JesseAbram JesseAbram marked this pull request as ready for review September 30, 2024 15:51
Copy link
Contributor

@ameba23 ameba23 left a comment

Choose a reason for hiding this comment

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

I think this is a good move. Maybe we need to do the same for changing http endpoint.

@JesseAbram
Copy link
Member Author

I think this is a good move. Maybe we need to do the same for changing http endpoint.

I think I see your point here with tdx, but there may be cases where you can, for example port forwarding, but it brings up a good point, redesigning these flows with TDX in ming

@JesseAbram JesseAbram merged commit c5bb5cc into master Sep 30, 2024
7 checks passed
@JesseAbram JesseAbram deleted the block-tss-chain-when-signer branch September 30, 2024 17:50
ameba23 added a commit that referenced this pull request Oct 1, 2024
* master:
  Pregenerate keyshares sets for all possible initial signer comittees (#1073)
  Fix master build (#1079)
  Bump reqwest from 0.12.7 to 0.12.8 in the patch-dependencies group (#1082)
  Block tss chain when signer (#1078)
ameba23 added a commit that referenced this pull request Oct 2, 2024
* master:
  Bump clap from 4.5.18 to 4.5.19 in the patch-dependencies group (#1091)
  Avoid panic by checking that we have a non-signing validator before selecting one (#1083)
  Fix master build (#1088)
  Small fixes to `test-cli` (#1084)
  Pregenerate keyshares sets for all possible initial signer comittees (#1073)
  Fix master build (#1079)
  Bump reqwest from 0.12.7 to 0.12.8 in the patch-dependencies group (#1082)
  Block tss chain when signer (#1078)
  Run multiple test validator (#1074)
  Bump tempfile from 3.12.0 to 3.13.0 (#1076)
  Bump axum from 0.7.6 to 0.7.7 in the patch-dependencies group (#1075)
  Unignore register and sign integration test, and do a non-mock jumpstart (#1070)
  No unbonding when signer or next signer (#1031)
  Add `/relay_tx` endpoint (#1050)
  Fix how pre-generated keyshares are added for tests (#1061)
  Handle Provisioning Certification Keys (PCKs) (#1051)
  Bump async-trait from 0.1.82 to 0.1.83 in the patch-dependencies group (#1067)
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