r12012@catbus: nickm | 2007-02-28 18:25:18 -0500

 Twiddle constants wrt uploading hidden service descriptors.


svn:r9694

3 files changed, 7 insertions, 3 deletions

diff --git a/ChangeLog b/ChangeLog
index 183a01b9f..35414b221 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -13,6 +13,10 @@ Changes in version 0.1.2.9-??? - 2007-??-??
       uptime and bandwidth cutoffs.
     - Stop listing hibernating servers in the v1 directory.
 
+  o Minor bugfixes (hidden services):
+    - Upload hidden service descriptors slightly less often, to reduce
+      load on authorities.
+
   o Minor bugfixes (other):
     - Fix an assert that could trigger if a controller quickly set then
       cleared EntryNodes.  (Bug found by Udo van den Heuvel.)
diff --git a/src/or/config.c b/src/or/config.c
index 56f4ed120..fa9f56e08 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -2272,7 +2272,7 @@ validate_ports_csv(smartlist_t *sl, const char *name, char **msg)
 
 /** Lowest allowable value for RendPostPeriod; if this is too low, hidden
  * services can overload the directory system. */
-#define MIN_REND_POST_PERIOD (5*60)
+#define MIN_REND_POST_PERIOD (10*60)
 
 /** Highest allowable value for RendPostPeriod. */
 #define MAX_DIR_PERIOD (MIN_ONION_KEY_LIFETIME/2)
diff --git a/src/or/rendservice.c b/src/or/rendservice.c
index c9aa171b6..3b509a0e4 100644
--- a/src/or/rendservice.c
+++ b/src/or/rendservice.c
@@ -1091,9 +1091,9 @@ rend_consider_services_upload(time_t now)
     }
     if (service->next_upload_time < now ||
         (service->desc_is_dirty &&
-         service->desc_is_dirty < now-5)) {
+         service->desc_is_dirty < now-30)) {
       /* if it's time, or if the directory servers have a wrong service
-       * descriptor and ours has been stable for 5 seconds, upload a
+       * descriptor and ours has been stable for 30 seconds, upload a
        * new one of each format. */
       upload_service_descriptor(service, 0);
       service->next_upload_time = now + rendpostperiod;