[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #20077 [Core Tor/Tor]: Make is_sensitive_dir_purpose and purpose_needs_anonymity consistent
#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@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs