[tor-bugs] #933 [Tor Client]: MapAddress for domains?
Tor Bug Tracker & Wiki
torproject-admin at torproject.org
Tue Dec 21 21:08:33 UTC 2010
#933: MapAddress for domains?
--------------------------+-------------------------------------------------
Reporter: Aleph0 | Owner: mwenge
Type: enhancement | Status: needs_review
Priority: minor | Milestone: Tor: 0.2.3.x-final
Component: Tor Client | Version: 0.2.0.33
Resolution: None | Keywords:
Parent: |
--------------------------+-------------------------------------------------
Comment(by nickm):
Almost there!
The code to handle the * case of config_register_addressmaps seems to
allow too much: It skips over the * starting "*example.com", not just the
* starting "*.example.com".
I think that the current syntax is still too hard to read: Having .mit.edu
be distinct from mit.edu will IMO be too easy for people to screw up.
Can't we just have the * be mandatory, so that "a.b.c" stays the same, but
"." becomes "*" and ".example.com" becomes "*.example.com"?
Now that we can have more than one rule apply to the same address, we
should be really explicit in the documentation about what we do when that
happens. I think the rule is, "If any rule applies, we apply whichever
rule was listed first, then check again and apply the one listed first,
and so on." Is that right? (Do we have a unit test for this case?)
Minor style:
* We initialize null pointers to NULL, not 0. (
addressmap_match_superdomains)
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/933#comment:8>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list