[or-cvs] r10089: Remove the _UploadExtraInfo option, since I tried turning it (in tor/trunk: . doc src/or)

[nickm at seul.org]

Author: nickm
Date: 2007-05-01 16:41:27 -0400 (Tue, 01 May 2007)
New Revision: 10089

Modified:
   tor/trunk/
   tor/trunk/doc/TODO
   tor/trunk/src/or/config.c
   tor/trunk/src/or/or.h
   tor/trunk/src/or/router.c
Log:
 r12625 at catbus:  nickm | 2007-05-01 16:41:23 -0400
 Remove the _UploadExtraInfo option, since I tried turning it on and moria[12] seem not to have exploded.



Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r12625] on 8246c3cf-6607-4228-993b-4d95d33730f1

Modified: tor/trunk/doc/TODO
===================================================================
--- tor/trunk/doc/TODO	2007-05-01 20:29:32 UTC (rev 10088)
+++ tor/trunk/doc/TODO	2007-05-01 20:41:27 UTC (rev 10089)
@@ -77,8 +77,8 @@
         o Implement cache on disk.
       o Have routers upload extra-info documents to authorities running
         version 0.2.0.0-alpha-dev (r10070) or later.
-        . Implement, but make it option-controlled.
-        - Make it always-on once it seems to work.
+        o Implement, but make it option-controlled.
+        o Make it always-on once it seems to work.
       - Implement option to download and cache extra-info documents.
       - Drop bandwidth history from router-descriptors
     - 105: Version negotiation for the Tor protocol (finalize by Jun 1)

Modified: tor/trunk/src/or/config.c
===================================================================
--- tor/trunk/src/or/config.c	2007-05-01 20:29:32 UTC (rev 10088)
+++ tor/trunk/src/or/config.c	2007-05-01 20:41:27 UTC (rev 10089)
@@ -262,9 +262,6 @@
   VAR("__DisablePredictedCircuits",BOOL,DisablePredictedCircuits,"0"),
   VAR("__LeaveStreamsUnattached", BOOL,LeaveStreamsUnattached, "0"),
 
-  /* XXXX020 Testing only. This will become "always on" once we confirm
-   * that it works. */
-  VAR("_UploadExtraInfo", BOOL, _UploadExtraInfo, "0"),
   { NULL, CONFIG_TYPE_OBSOLETE, 0, NULL }
 };
 #undef VAR

Modified: tor/trunk/src/or/or.h
===================================================================
--- tor/trunk/src/or/or.h	2007-05-01 20:29:32 UTC (rev 10088)
+++ tor/trunk/src/or/or.h	2007-05-01 20:41:27 UTC (rev 10089)
@@ -1881,8 +1881,6 @@
  /** If true, we try resolving hostnames with weird characters. */
   int ServerDNSAllowNonRFC953Hostnames;
 
-  /*XXXX020 remove me once no longer needed */
-  int _UploadExtraInfo;
 } or_options_t;
 
 /** Persistent state for an onion router, as saved to disk. */

Modified: tor/trunk/src/or/router.c
===================================================================
--- tor/trunk/src/or/router.c	2007-05-01 20:29:32 UTC (rev 10088)
+++ tor/trunk/src/or/router.c	2007-05-01 20:41:27 UTC (rev 10089)
@@ -730,7 +730,6 @@
   extrainfo_t *ei;
   char *msg;
   size_t desc_len, extra_len = 0, total_len;
-  int post_extra;
 
   ri = router_get_my_routerinfo();
   if (!ri) {
@@ -738,7 +737,6 @@
     return;
   }
   ei = router_get_my_extrainfo();
-  post_extra = ei && get_options()->_UploadExtraInfo;
   if (!get_options()->PublishServerDescriptor)
     return;
   if (!force && !desc_needs_upload)
@@ -746,11 +744,11 @@
   desc_needs_upload = 0;
 
   desc_len = ri->cache_info.signed_descriptor_len;
-  extra_len = (ei && post_extra) ? ei->cache_info.signed_descriptor_len : 0;
+  extra_len = ei ? ei->cache_info.signed_descriptor_len : 0;
   total_len = desc_len + extra_len + 1;
   msg = tor_malloc(total_len);
   memcpy(msg, ri->cache_info.signed_descriptor_body, desc_len);
-  if (ei && post_extra) {
+  if (ei) {
     memcpy(msg+desc_len, ei->cache_info.signed_descriptor_body, extra_len);
   }
   msg[desc_len+extra_len] = 0;