commit b3820fe7859fa9666b1352ecdb3916ab51ec7b6d Author: Damian Johnson atagar@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