[tor-commits] [tor/master] Revert "Disable (Cell, DirReq, Entry, ExitPort)Statistics on bridges"

nickm at torproject.org nickm at torproject.org
Tue Jun 5 14:51:31 UTC 2012


commit 2468a1bd2c3a989c4225897e58b5bf09ac08ba61
Author: Nick Mathewson <nickm at torproject.org>
Date:   Tue Jun 5 10:47:05 2012 -0400

    Revert "Disable (Cell,DirReq,Entry,ExitPort)Statistics on bridges"
    
    This reverts commit 981e896dd2eaf69798bb503c271306ee779dd6d2.
    
    Apparently Karsten still needs DirReqStatistics for bridges; see
---
 changes/bug5824 |    3 ---
 src/or/config.c |    5 ++---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/changes/bug5824 b/changes/bug5824
deleted file mode 100644
index a75e8ed..0000000
--- a/changes/bug5824
+++ /dev/null
@@ -1,3 +0,0 @@
-  o Minor bugfixes:
-    - Bridges now never collect statistics that were designed for
-      relays. Fix for bug 5824; bugfix on 0.2.1.17-rc.
diff --git a/src/or/config.c b/src/or/config.c
index 03186bf..090d96c 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -1703,9 +1703,8 @@ options_act(const or_options_t *old_options)
     time_t now = time(NULL);
     int print_notice = 0;
 
-    /* If we aren't acting as a server, or we are a bridge, we can't collect
-     * stats anyway. */
-    if (!public_server_mode(options)) {
+    /* If we aren't acting as a server, we can't collect stats anyway. */
+    if (!server_mode(options)) {
       options->CellStatistics = 0;
       options->DirReqStatistics = 0;
       options->EntryStatistics = 0;



More information about the tor-commits mailing list