[tor-commits] [doctor/master] Check for signatures of all voting authorities

[atagar at torproject.org]

commit 9146460a6ad6634fca8d0593e5a8b549a5ca6c76
Author: Damian Johnson <atagar at torproject.org>
Date:   Sun Dec 20 15:00:59 2015 -0800

    Check for signatures of all voting authorities
    
    Oops! Dannenberg hasn't been voting in the consensus for the last month due to
    its v3ident change. It's still up and providing a consensus so we didn't raise
    an alarm. Tweaking our 'has all signatures' check to check for all voting
    authorities rather than just the participants.
---
 consensus_health_checker.py |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/consensus_health_checker.py b/consensus_health_checker.py
index 377fa0a..1962d10 100755
--- a/consensus_health_checker.py
+++ b/consensus_health_checker.py
@@ -511,9 +511,9 @@ def has_all_signatures(latest_consensus, consensuses, votes):
   "Check that the consensuses have signatures for authorities that voted on it."
 
   issues = []
+  voting_authorities = set([authority.v3ident for authority in DIRECTORY_AUTHORITIES.values() if authority.v3ident])
 
   for consensus_of, consensus in consensuses.items():
-    voting_authorities = set([authority.fingerprint for authority in consensus.directory_authorities])
     signing_authorities = set([sig.identity for sig in consensus.signatures])
     missing_authorities = set()