[tor-commits] [torflow/master] Add some comments about potential timeout issues.
mikeperry at torproject.org
mikeperry at torproject.org
Thu Nov 17 23:50:09 UTC 2011
commit 1efe897f89b6ff113862dd0d418a17873962b836
Author: Mike Perry <mikeperry-git at fscked.org>
Date: Mon Nov 14 18:11:35 2011 -0800
Add some comments about potential timeout issues.
---
NetworkScanners/BwAuthority/bwauthority_child.py | 7 ++++++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/NetworkScanners/BwAuthority/bwauthority_child.py b/NetworkScanners/BwAuthority/bwauthority_child.py
index c42e163..65063a9 100755
--- a/NetworkScanners/BwAuthority/bwauthority_child.py
+++ b/NetworkScanners/BwAuthority/bwauthority_child.py
@@ -225,8 +225,11 @@ def speedrace(hdlr, start_pct, stop_pct, circs_per_node, save_every, out_dir,
hdlr.new_exit()
attempt += 1
- # FIXME: This noise is due to a difficult to find Tor bug that
+ # TODO: This noise is due to a difficult to find Tor bug that
# causes some exits to hang forever on streams :(
+ # FIXME: Hrmm, should we change the reason on this? Right now,
+ # 7 == TIMEOUT, which means we do not count the bandwidth of this
+ # stream.. however, we count it as 'successful' below
timer = threading.Timer(max_fetch_time, lambda: hdlr.close_streams(7))
timer.start()
url = choose_url(start_pct)
@@ -240,6 +243,8 @@ def speedrace(hdlr, start_pct, stop_pct, circs_per_node, save_every, out_dir,
plog('WARN', 'Timer exceeded limit: ' + str(delta_build) + '\n')
build_exit = hdlr.get_exit_node()
+ # FIXME: Timeouts get counted as 'sucessful' here, but do not
+ # count in the SQL stats!
if ret == 1 and build_exit:
successful += 1
plog('DEBUG', str(start_pct) + '-' + str(stop_pct) + '% circuit build+fetch took ' + str(delta_build) + ' for ' + str(build_exit))
More information about the tor-commits
mailing list