From 6fdb179d84659fb14bacebf08c386cf21ec1b9d4 Mon Sep 17 00:00:00 2001 From: Yawning Angel Date: Wed, 18 Feb 2015 12:51:07 +0000 Subject: Fix compute_frac_paths_available, when ExitNodes is not set. --- changes/bug14918 | 3 +++ 1 file changed, 3 insertions(+) (limited to 'changes/bug14918') diff --git a/changes/bug14918 b/changes/bug14918 index 8e196cce5c..82f585a36d 100644 --- a/changes/bug14918 +++ b/changes/bug14918 @@ -5,3 +5,6 @@ - Do not warn that we are missing directory information if we have decided not to fetch directory information due to a long period of idleness. Fixes part of bug 14918; bugfix on 0.2.6.2-alpha. + - When deciding whether we have any exit nodes, only do processing for + ExitNodes when the ExitNodes option is actually set. Fixes part of + bug 14918; bugfix on 0.2.6.2-alpha. -- cgit v1.2.3-54-g00ecf From 8571e86d27577e348bdb6398eda6190c2c63c0ed Mon Sep 17 00:00:00 2001 From: Yawning Angel Date: Wed, 18 Feb 2015 13:58:13 +0000 Subject: Fix bootstrap directory information logging. `dir_info_status` is used from main.c:directory_info_has_arrived() to provide useful (INFO/NOTICE) level logging to users, and should always be updated regardless of the rate limiting. --- changes/bug14918 | 5 +++++ src/or/nodelist.c | 24 ++++++++++++++---------- 2 files changed, 19 insertions(+), 10 deletions(-) (limited to 'changes/bug14918') diff --git a/changes/bug14918 b/changes/bug14918 index 82f585a36d..6edd425820 100644 --- a/changes/bug14918 +++ b/changes/bug14918 @@ -8,3 +8,8 @@ - When deciding whether we have any exit nodes, only do processing for ExitNodes when the ExitNodes option is actually set. Fixes part of bug 14918; bugfix on 0.2.6.2-alpha. + + o Minor bugfixes (logging during bootstrap): + - Correctly update and log the amount of directory information that + has been fetched during the bootstrap process. Fixes part of bug + 14918; bugfix on 0.2.6.2-alpha. diff --git a/src/or/nodelist.c b/src/or/nodelist.c index e9335063f1..ab05bce420 100644 --- a/src/or/nodelist.c +++ b/src/or/nodelist.c @@ -1685,12 +1685,14 @@ update_router_have_minimum_dir_info(void) static ratelim_t last_warned = RATELIM_INIT(NOTICE_DIR_INFO_STATUS_INTERVAL); char *suppression_msg = NULL; + + tor_snprintf(dir_info_status, sizeof(dir_info_status), + "We need more %sdescriptors: we have %d/%d, and " + "can only build %d%% of likely paths. (We have %s.)", + using_md?"micro":"", num_present, num_usable, + (int)(paths*100), status); + if ((suppression_msg = rate_limit_log(&last_warned, time(NULL)))) { - tor_snprintf(dir_info_status, sizeof(dir_info_status), - "We need more %sdescriptors: we have %d/%d, and " - "can only build %d%% of likely paths. (We have %s.)", - using_md?"micro":"", num_present, num_usable, - (int)(paths*100), status); if (!should_delay_dir_fetches(options, NULL) && !directory_too_idle_to_fetch_descriptors(options, now)) { log_warn(LD_NET, "%s%s", dir_info_status, suppression_msg); @@ -1706,12 +1708,14 @@ update_router_have_minimum_dir_info(void) static ratelim_t last_warned = RATELIM_INIT(NOTICE_DIR_INFO_STATUS_INTERVAL); char *suppression_msg = NULL; + + tor_snprintf(dir_info_status, sizeof(dir_info_status), + "We have enough %sdescriptors: we have %d/%d, and " + "can build %d%% of likely paths. (We have %s.)", + using_md?"micro":"", num_present, num_usable, + (int)(paths*100), status); + if ((suppression_msg = rate_limit_log(&last_warned, time(NULL)))) { - tor_snprintf(dir_info_status, sizeof(dir_info_status), - "We have enough %sdescriptors: we have %d/%d, and " - "can build %d%% of likely paths. (We have %s.)", - using_md?"micro":"", num_present, num_usable, - (int)(paths*100), status); log_info(LD_NET, "%s%s", dir_info_status, suppression_msg); tor_free(suppression_msg); } -- cgit v1.2.3-54-g00ecf