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.

Sign up for GitHub

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

Update TSS tests to use new registration flow #1026

Merged
merged 18 commits into from
Aug 26, 2024
Merged

Update TSS tests to use new registration flow #1026

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
e582265
Remove signing integration test
HCastano Aug 22, 2024
86fdd96
Update ETH integration test to use new registration flow
HCastano Aug 22, 2024
805143b
Get program config test working again
HCastano Aug 23, 2024
16cdeec
Delete `test_store_share` test
HCastano Aug 23, 2024
bf3f712
Fix `test_fail_infinite_program`
HCastano Aug 23, 2024
9ba38e7
Fix `test_device_key_proxy` test
HCastano Aug 23, 2024
02496c5
Get `test_sign_tx_no_chain` mostly passing
HCastano Aug 23, 2024
47c181f
Move request limit tests to their own test
HCastano Aug 23, 2024
4685ac0
Rename failing signing test
HCastano Aug 23, 2024
87ea4ff
Move a failing connection test out of `test_sign_tx_no_chain`
HCastano Aug 23, 2024
1a75496
Ignore failing test
HCastano Aug 23, 2024
1d695c6
Update test name to better reflect what it covers
HCastano Aug 23, 2024
c8a2fc3
Rename `generic_msg` to `signature_request`
HCastano Aug 23, 2024
a47156c
Fix test for rejecting a TSS node who is not part of the signing comm…
ameba23 Aug 26, 2024
3d77e12
Fix used import and variable warnings
HCastano Aug 26, 2024
2ac232a
Use `test_client` for registration in all non-registration tests
HCastano Aug 26, 2024
ea6dbd4
Rustfmt
HCastano Aug 26, 2024
8c46c82
Move `update_programs` call to relevant location
HCastano Aug 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions crates/threshold-signature-server/src/signing_client/protocol_transport.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,7 @@ async fn handle_initial_incoming_ws_message(
};

{
// Check that the given public key matches the public key we got in the
// UserTransactionRequest or register transaction
// Check that the given public key is of the ones we are expecting for this protocol session
let mut listeners = app_state
.listener_state
.listeners
Expand All @@ -186,8 +185,7 @@ async fn handle_initial_incoming_ws_message(
// Make the signing process fail, since one of the commitee has misbehaved
listeners.remove(&msg.session_id);
return Err(SubscribeErr::Decryption(
"Public key does not match that given in UserTransactionRequest or register \
transaction"
"Public key does not match any of those expected for this protocol session"
.to_string(),
));
}
Expand Down
Loading
Loading