[tor-commits] [stem/master] Don't fail validation if consensus method 1 isn't present

atagar at torproject.org atagar at torproject.org
Thu Aug 28 16:47:45 UTC 2014


commit b3820fe7859fa9666b1352ecdb3916ab51ec7b6d
Author: Damian Johnson <atagar at torproject.org>
Date:   Thu Aug 28 09:48:25 2014 -0700

    Don't fail validation if consensus method 1 isn't present
    
    Change from proposal 215. Tor no longer requires that consensus method 1 is
    around as a fallback. Pointed out thanks to Sebastian.
---
 docs/change_log.rst              |    1 +
 stem/descriptor/networkstatus.py |    3 ---
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/docs/change_log.rst b/docs/change_log.rst
index b52a377..4ec16a0 100644
--- a/docs/change_log.rst
+++ b/docs/change_log.rst
@@ -43,6 +43,7 @@ The following are only available within Stem's `git repository
  * **Descriptors**
 
   * Improved speed for parsing consensus documents by around 30% (:trac:`12859`)
+  * Don't fail if consensus method 1 is not present, as it is no longer required (:spec:`fc8a6f0`)
 
  * **Utilities**
 
diff --git a/stem/descriptor/networkstatus.py b/stem/descriptor/networkstatus.py
index 115f07b..c861236 100644
--- a/stem/descriptor/networkstatus.py
+++ b/stem/descriptor/networkstatus.py
@@ -690,9 +690,6 @@ class _DocumentHeader(object):
             raise ValueError("A network status document's consensus-methods must be a list of integer values, but was '%s'" % value)
 
         self.consensus_methods = consensus_methods
-
-        if validate and not (1 in self.consensus_methods):
-          raise ValueError('Network status votes must include consensus-method version 1')
       elif keyword == 'consensus-method':
         # "consensus-method" Integer
 



More information about the tor-commits mailing list