[tor-commits] [arm/release] fix: mixing up the relay and client shutdown opts

atagar at torproject.org atagar at torproject.org
Sun Jul 17 06:08:32 UTC 2011


commit 2a1daaa179f0c25c15e0d71cb43d7ecbc92e129d
Author: Damian Johnson <atagar at torproject.org>
Date:   Mon Jul 11 13:27:05 2011 -0700

    fix: mixing up the relay and client shutdown opts
    
    There's a separate shutdown option for clients vs relays (RSHUTDOWN vs
    CSHUTDOWN), the reason being that they have a different default and
    descriptions. The template was mistakenly taking the logical or of the
    two options rather than picking the one for our relay type.
---
 src/cli/wizard.py               |    7 +++++++
 src/resources/torrcTemplate.txt |    2 +-
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/src/cli/wizard.py b/src/cli/wizard.py
index b50105e..6caf490 100644
--- a/src/cli/wizard.py
+++ b/src/cli/wizard.py
@@ -600,6 +600,13 @@ def getTorrc(relayType, config, disabledOpt):
     if config[Options.IM].getValue(): policyCategories.append("im")
     if config[Options.MISC].getValue(): policyCategories.append("misc")
   
+  # uses the CSHUTDOWN or RSHUTDOWN option based on if we're running as a
+  # client or not
+  if relayType == RelayType.CLIENT:
+    templateOptions["SHUTDOWN"] = templateOptions[Options.CSHUTDOWN.upper()]
+  else:
+    templateOptions["SHUTDOWN"] = templateOptions[Options.RSHUTDOWN.upper()]
+  
   if policyCategories:
     isEncryptedOnly = not config[Options.PLAINTEXT].getValue()
     
diff --git a/src/resources/torrcTemplate.txt b/src/resources/torrcTemplate.txt
index c31b209..d23ed0b 100644
--- a/src/resources/torrcTemplate.txt
+++ b/src/resources/torrcTemplate.txt
@@ -7,7 +7,7 @@
 # Descriptions of all of these configuraiton attibutes (and many more) are
 # available in the tor man page.
 
-[IF RSHUTDOWN | CSHUTDOWN]
+[IF SHUTDOWN]
   [NEWLINE]
   # The following flag tells arm to shut down tor when it quits.
   # ARM_SHUTDOWN





More information about the tor-commits mailing list