[tor-bugs] #12538 [Tor]: Make all relays automatically be dir caches

Tor Bug Tracker & Wiki blackhole at torproject.org
Sun Dec 20 12:40:24 UTC 2015


#12538: Make all relays automatically be dir caches
-------------------------------------------------+-------------------------
 Reporter:  cypherpunks                          |          Owner:
     Type:  task                                 |         Status:
 Priority:  High                                 |  needs_review
Component:  Tor                                  |      Milestone:  Tor:
 Severity:  Normal                               |  0.2.8.x-final
 Keywords:  tor-guard, tor-relay, prop237,       |        Version:  Tor:
  026-triaged-1, sebastian-review,               |  unspecified
  027-triaged-1-out, 028-triage, 028-triaged,    |     Resolution:
  mike-can, pre028-patch, TorCoreTeam201512,     |  Actual Points:
  201511-deferred                                |         Points:
Parent ID:                                       |  medium/large
  Sponsor:                                       |
-------------------------------------------------+-------------------------

Comment (by teor):

 Replying to [comment:92 nickm]:
 > After squashing and merging to master, these tests failed:
 > {{{
 > nodelist/node_is_dir
 > config/directory_fetch
 > routerlist/pick_directory_server_impl
 > }}}
 >
 > I took a shot at fixing them, but I'd like somebody else to sanity-check
 whether my changes make sense in light of the test code.
 >
 > Please see the 3 latest commits on my branch "bug12538_merged":
 > {{{
 > f0a4282 fix routerlist/pick_directory_server_impl in light of 12538
 > f5f35e9 Fix config/directory_fetch after 12538 merge
 > a6c9fcc Fix nodelist/node_is_dir test wrt 12538.
 > }}}
 >
 > They're all very short.

 f0a4282 I can't understand why we're inverting the sense of the tests
 here. But I'm not familiar enough with the code or the tests to make a
 better suggestion.

 f5f35e9 is almost exactly what I expected to have to do once this merged.

 a6c9fcc looks plausible.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/12538#comment:93>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list