[tor-bugs] #23814 [Core Tor/Tor]: Remove non-exponential backoff directory download implementation

[Tor Bug Tracker & Wiki]

#23814: Remove non-exponential backoff directory download implementation
-----------------------------+------------------------------------
 Reporter:  teor             |          Owner:  nickm
     Type:  defect           |         Status:  needs_revision
 Priority:  Medium           |      Milestone:  Tor: 0.3.3.x-final
Component:  Core Tor/Tor     |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:  review-group-31  |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:  asn              |        Sponsor:
-----------------------------+------------------------------------

Comment (by nickm):

 > I'm a bit curious on why we needed to do 5b55e15 if the function works
 fine and we had tests for it. Is it to simplify the code? Should we do it
 as part of another ticket, or we feel fine doing it in this one? Code
 looks reasonable anyhow. We should also remove mention of max_delay from
 func doc of next_random_exponential_delay().

 I'm fine removing max_delay it in this ticket -- My rationale was that
 with the removal of , it was never anything besides INT_MAX, and so wasn't
 actually serving any purpose.

 Working on the other issues you raise above.

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