commit 08dbde23c1b8e9f2415723cb08d882ec7add07c8 Author: Damian Johnson <ata...@torproject.org> Date: Sat Nov 4 15:27:59 2017 -0700
Don't use inference resolution if proc is unavailable Oops. On reflection many platforms (OSX, BSD, etc) lack proc contents so we shouldn't attempt to use them when unavailable. Otherwise the connection panel will be delayed. Caught thanks to a report on tor-relays@. --- nyx/panel/connection.py | 2 -- nyx/tracker.py | 9 ++++++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/nyx/panel/connection.py b/nyx/panel/connection.py index ed0ec11..0fb4889 100644 --- a/nyx/panel/connection.py +++ b/nyx/panel/connection.py @@ -17,8 +17,6 @@ import nyx.panel import nyx.popups import nyx.tracker -import stem.util.log - from nyx import nyx_interface, tor_controller from nyx.curses import WHITE, NORMAL, BOLD, HIGHLIGHT from nyx.menu import MenuItem, Submenu, RadioMenuItem, RadioGroup diff --git a/nyx/tracker.py b/nyx/tracker.py index 8ef928a..a108433 100644 --- a/nyx/tracker.py +++ b/nyx/tracker.py @@ -510,10 +510,13 @@ class ConnectionTracker(Daemon): # resolution. Otherwise connection resolution by inference is the only game # in town. + self._resolvers = [] + if tor_controller().get_conf('DisableDebuggerAttachment', None) == '0': - self._resolvers = [CustomResolver.INFERENCE] + connection.system_resolvers() - else: - self._resolvers = [CustomResolver.INFERENCE] + self._resolvers = connection.system_resolvers() + + if stem.util.proc.is_available(): + self._resolvers = [CustomResolver.INFERENCE] + self._resolvers stem.util.log.info('Operating System: %s, Connection Resolvers: %s' % (os.uname()[0], ', '.join(self._resolvers))) _______________________________________________ tor-commits mailing list tor-commits@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits