[tor-commits] [doctor/master] Exclude authorities on .0 versions from check.

karsten at torproject.org karsten at torproject.org
Wed Mar 6 15:37:59 UTC 2013


commit 9cdb65f51c2eafcab660c337f587693a85553b7a
Author: Karsten Loesing <karsten.loesing at gmx.net>
Date:   Wed Mar 6 16:37:41 2013 +0100

    Exclude authorities on .0 versions from check.
    
    Authorities often run not-yet-released tor versions, which have version
    strings ending in "-dev".  We ignore status tags in version checks, so
    a version string "0.2.4.10-alpha-dev" is okay as long as "0.2.4.10-alpha"
    is recommended.  But "0.2.5.0-alpha-dev" was not okay, because there's no
    recommended version "0.2.5.0-alpha".  Let's pretend versions ending in
    ".0" (plus status tag) are always okay.  Shouldn't produce too many false
    negatives, but saves us from writing lots of version string parsing code.
    
    Fixes #8393.
---
 src/org/torproject/doctor/Checker.java |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/org/torproject/doctor/Checker.java b/src/org/torproject/doctor/Checker.java
index 3adc678..5681e56 100644
--- a/src/org/torproject/doctor/Checker.java
+++ b/src/org/torproject/doctor/Checker.java
@@ -509,7 +509,8 @@ public class Checker {
           authorityVersion = authorityVersion.substring("Tor ".length());
         }
         if (!recommendedVersions.contains(
-              authorityVersion.split("-", 2)[0])) {
+              authorityVersion.split("-", 2)[0]) &&
+            !authorityVersion.split("-", 2)[0].endsWith(".0")) {
           unrecommendedVersions.add(nickname + "=" + authorityVersion);
         }
       }



More information about the tor-commits mailing list