#20077: Make is_sensitive_dir_purpose and purpose_needs_anonymity consistent -------------------------------------------------+------------------------- Reporter: teor | Owner: Type: defect | Status: | needs_revision Priority: Medium | Milestone: Tor: | 0.3.0.x-final Component: Core Tor/Tor | Version: Severity: Normal | Resolution: Keywords: refactor, 030-proposed, | Actual Points: TorCoreTeam201610 | Parent ID: | Points: 1 Reviewer: teor | Sponsor: -------------------------------------------------+-------------------------
Comment (by chelseakomlo): Ok! Here are the latest changes. https://github.com/chelseakomlo/tor_patches/commit/3343b20cc5cf73a09fc3c5d7b07841821c81473c I had one question when doing this: Outside of directory.c, this functionality is used in two other places, neither of which test whether the router purpose is a bridge (as is done throughout directory.c). Specifically, this change feels slightly dirty: https://github.com/chelseakomlo/tor_patches/commit/3343b20cc5cf73a09fc3c5d7b07841821c81473c #diff-0798d3d17392dc5c15f3f58a5fc6b29aR2392 But I'm not sure if making two interfaces for this (one specifically for callers who only want to check the dir_purpose) is overkill. -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20077#comment:6> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online _______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs