Browse Source

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
	doc/tor.1.txt
Nick Mathewson 13 years ago
parent
commit
f1a004797e
3 changed files with 12 additions and 4 deletions
  1. 6 0
      changes/bug2408
  2. 3 3
      doc/tor.1.txt
  3. 3 1
      src/or/config.c

+ 6 - 0
changes/bug2408

@@ -0,0 +1,6 @@
+  o Major bugfixes
+    - Ignore and warn about "PublishServerDescriptor hidserv" torrc
+      options.  The 'hidserv' argument never controlled publication
+      of hidden service descriptors.  Bugfix on 0.2.0.1-alpha.
+
+

+ 3 - 3
doc/tor.1.txt

@@ -909,9 +909,9 @@ is non-zero):
     If set to a path, only the specified path will be executed.
     (Default: tor-fw-helper)
 
-**PublishServerDescriptor** **0**|**1**|**v1**|**v2**|**v3**|**bridge**|**hidserv**,**...**::
+**PublishServerDescriptor** **0**|**1**|**v1**|**v2**|**v3**|**bridge**,**...**::
     This option specifies which descriptors Tor will publish when acting as
-    a relay or hidden service. You can
+    a relay. You can
     choose multiple arguments, separated by commas.
  +
     If this option is set to 0, Tor will not publish its
@@ -919,7 +919,7 @@ is non-zero):
     out your server, or if you're using a Tor controller that handles directory
     publishing for you.) Otherwise, Tor will publish its descriptors of all
     type(s) specified. The default is "1",
-    which means "if running as a server or a hidden service, publish the
+    which means "if running as a server, publish the
     appropriate descriptors to the authorities".
 
 **ShutdownWaitLength** __NUM__::

+ 3 - 1
src/or/config.c

@@ -2889,7 +2889,9 @@ compute_publishserverdescriptor(or_options_t *options)
     else if (!strcasecmp(string, "bridge"))
       *auth |= BRIDGE_AUTHORITY;
     else if (!strcasecmp(string, "hidserv"))
-      *auth |= HIDSERV_AUTHORITY;
+      log_warn(LD_CONFIG,
+               "PublishServerDescriptor hidserv is invalid. See "
+               "PublishHidServDescriptors.");
     else if (!strcasecmp(string, "") || !strcmp(string, "0"))
       /* no authority */;
     else