[tor-bugs] #20082 [Core Tor/Tor]: Lower initial descriptor upload delay for ephemeral services
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue Oct 11 23:52:16 UTC 2016
#20082: Lower initial descriptor upload delay for ephemeral services
-------------------------------------------------+-------------------------
Reporter: twim | Owner:
Type: enhancement | Status:
| needs_revision
Priority: Medium | Milestone: Tor:
| 0.2.???
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: tor-hs, research, proposal-needed?, | Actual Points:
TorCoreTeam201610, 030-proposed |
Parent ID: | Points:
Reviewer: | Sponsor:
| SponsorR-can
-------------------------------------------------+-------------------------
Changes (by teor):
* status: needs_review => needs_revision
* keywords: tor-hs, research, proposal-needed?, TorCoreTeam201609 => tor-
hs, research, proposal-needed?, TorCoreTeam201610, 030-proposed
Comment:
Thanks for these patches. Given the discussion in #20262, I think we can
proceed with removing this code that never worked anyway.
A few comments on patch 0001:
* Please don't remove the lower delay for test networks. Instead, make it
0.
* All references to rend_service_reveal_startup_time() should be removed,
because it is no longer used.
And on 0002:
* We generally don't include numbers in the code like
`old_stabilizing_period = (time_t) 30;`, instead, let's keep the old
#define, but change its name to include OLD.
* The warning message is unhelpful, because the user can't do anything to
fix it. It will most likely be triggered when the network is slow or
unreliable - let's say that, and tell them that their service may be
unreliable as a result. (The message that maybe the "new stabilizing
period is too short" is for developers, it should go in a comment above
the log message.)
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20082#comment:23>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list