From 4446a369b13b4152ae5e93cfa934f11ad7a7362c Mon Sep 17 00:00:00 2001 From: Luke Parker Date: Sat, 2 Dec 2023 09:09:16 -0500 Subject: [PATCH] Remove old TODO --- coordinator/src/tributary/handle.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/coordinator/src/tributary/handle.rs b/coordinator/src/tributary/handle.rs index 7887df75..e3d5eba7 100644 --- a/coordinator/src/tributary/handle.rs +++ b/coordinator/src/tributary/handle.rs @@ -175,6 +175,8 @@ pub(crate) async fn handle_application_tx< }; // If they've already published a TX for this attempt, slash + // This shouldn't be reachable since nonces were made inserted by the coordinator, yet it's a + // cheap check to leave in for safety if DataDb::get(txn, genesis, data_spec, &signed.signer.to_bytes()).is_some() { fatal_slash::(txn, genesis, signed.signer.to_bytes(), "published data multiple times"); return Accumulation::NotReady; @@ -409,7 +411,6 @@ pub(crate) async fn handle_application_tx< } } - // TODO: Only accept one of either InvalidDkgShare/DkgConfirmed per signer // TODO: Ban self-accusals Transaction::InvalidDkgShare { attempt, accuser, faulty, blame, signed } => { let range = spec.i(signed.signer).unwrap();