[tor-commits] [tor/master] relay: Avoid extra LOG_NOTICE for every new descriptor batch

[nickm@xxxxxxxxxxxxxx]

commit 380a95116a0e582eca0cc643eb0bca65c3820c86
Author: David Goulet <dgoulet@xxxxxxxxxxxxxx>
Date:   Thu Nov 16 10:51:41 2017 -0500

    relay: Avoid extra LOG_NOTICE for every new descriptor batch
    
    Commit 56c5e282a733912776f6dacbe4f5df66b4fb9606 suppressed that same log
    statement in directory_info_has_arrived() for microdescriptors so do the same
    for the descriptors. As the commit says, we already have the bootstrap
    progress for this.
    
    Fixes #23861
    
    Signed-off-by: David Goulet <dgoulet@xxxxxxxxxxxxxx>
---
 changes/bug23861   | 5 +++++
 src/or/directory.c | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/changes/bug23861 b/changes/bug23861
new file mode 100644
index 000000000..c6f017640
--- /dev/null
+++ b/changes/bug23861
@@ -0,0 +1,5 @@
+  o Minor bugfixes (logging, relay):
+    - Suppress a log notice when relay descriptors arrive. We already have a
+      bootstrap progress for this so no need to log notice everytime tor
+      receives relay descriptors. Microdescriptors behave the same. Fixes bug
+      23861; bugfix on 0.2.8.2-alpha.
diff --git a/src/or/directory.c b/src/or/directory.c
index 836f86298..b3dbb0d1e 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -2889,7 +2889,7 @@ handle_response_fetch_desc(dir_connection_t *conn,
                                   conn->router_purpose,
                                   conn->base_.address)) {
         time_t now = approx_time();
-        directory_info_has_arrived(now, 0, 0);
+        directory_info_has_arrived(now, 0, 1);
       }
     }
   }



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits