From 380ae2b74b8c8762acc075d8d22bc5d6320655e1 Mon Sep 17 00:00:00 2001 From: rl1987 Date: Sun, 23 Dec 2018 19:21:37 +0200 Subject: [PATCH] Stop logging 'Your Guard' in circpathbias.c --- changes/bug28895 | 5 +++++ src/feature/client/circpathbias.c | 14 +++++++------- 2 files changed, 12 insertions(+), 7 deletions(-) create mode 100644 changes/bug28895 diff --git a/changes/bug28895 b/changes/bug28895 new file mode 100644 index 0000000000..25fb167b2e --- /dev/null +++ b/changes/bug28895 @@ -0,0 +1,5 @@ + o Minor bugfixes (usability): + - Stop saying "Your Guard ..." in pathbias_measure_{use,close}_rate() + as that confusingly suggests that mentioned guard node is under control + and responsibility of end user, which it is not. Fixes bug 28895; + bugfix on Tor 0.3.0.1-alpha. diff --git a/src/feature/client/circpathbias.c b/src/feature/client/circpathbias.c index 9f2ed9347c..eaeef5fe5d 100644 --- a/src/feature/client/circpathbias.c +++ b/src/feature/client/circpathbias.c @@ -1332,7 +1332,7 @@ pathbias_measure_use_rate(entry_guard_t *guard) if (pathbias_get_dropguards(options)) { if (!pb->path_bias_disabled) { log_warn(LD_CIRC, - "Your Guard %s is failing to carry an extremely large " + "Guard %s is failing to carry an extremely large " "amount of stream on its circuits. " "To avoid potential route manipulation attacks, Tor has " "disabled use of this guard. " @@ -1356,7 +1356,7 @@ pathbias_measure_use_rate(entry_guard_t *guard) } else if (!pb->path_bias_use_extreme) { pb->path_bias_use_extreme = 1; log_warn(LD_CIRC, - "Your Guard %s is failing to carry an extremely large " + "Guard %s is failing to carry an extremely large " "amount of streams on its circuits. " "This could indicate a route manipulation attack, network " "overload, bad local network connectivity, or a bug. " @@ -1380,7 +1380,7 @@ pathbias_measure_use_rate(entry_guard_t *guard) if (!pb->path_bias_use_noticed) { pb->path_bias_use_noticed = 1; log_notice(LD_CIRC, - "Your Guard %s is failing to carry more streams on its " + "Guard %s is failing to carry more streams on its " "circuits than usual. " "Most likely this means the Tor network is overloaded " "or your network connection is poor. " @@ -1437,7 +1437,7 @@ pathbias_measure_close_rate(entry_guard_t *guard) if (pathbias_get_dropguards(options)) { if (!pb->path_bias_disabled) { log_warn(LD_CIRC, - "Your Guard %s is failing an extremely large " + "Guard %s is failing an extremely large " "amount of circuits. " "To avoid potential route manipulation attacks, Tor has " "disabled use of this guard. " @@ -1461,7 +1461,7 @@ pathbias_measure_close_rate(entry_guard_t *guard) } else if (!pb->path_bias_extreme) { pb->path_bias_extreme = 1; log_warn(LD_CIRC, - "Your Guard %s is failing an extremely large " + "Guard %s is failing an extremely large " "amount of circuits. " "This could indicate a route manipulation attack, " "extreme network overload, or a bug. " @@ -1485,7 +1485,7 @@ pathbias_measure_close_rate(entry_guard_t *guard) if (!pb->path_bias_warned) { pb->path_bias_warned = 1; log_warn(LD_CIRC, - "Your Guard %s is failing a very large " + "Guard %s is failing a very large " "amount of circuits. " "Most likely this means the Tor network is " "overloaded, but it could also mean an attack against " @@ -1510,7 +1510,7 @@ pathbias_measure_close_rate(entry_guard_t *guard) if (!pb->path_bias_noticed) { pb->path_bias_noticed = 1; log_notice(LD_CIRC, - "Your Guard %s is failing more circuits than " + "Guard %s is failing more circuits than " "usual. " "Most likely this means the Tor network is overloaded. " "Success counts are %ld/%ld. Use counts are %ld/%ld. "