Bugfix: dyndns tor servers were needing to wait 18 hours before they

could start doing reachability testing using the new IP address and
ports. This is because we were using the internal descriptor to learn
what to test, yet we were only rebuilding the descriptor once we
decided we were reachable.


svn:r5723
This commit is contained in:
Roger Dingledine 2006-01-03 23:34:42 +00:00
parent d10f2b7267
commit 6b2a6a8239

View File

@ -579,13 +579,18 @@ decide_if_publishable_server(time_t now)
/** Initiate server descriptor upload as reasonable (if server is publishable, /** Initiate server descriptor upload as reasonable (if server is publishable,
* etc). <b>force</b> is as for router_upload_dir_desc_to_dirservers. * etc). <b>force</b> is as for router_upload_dir_desc_to_dirservers.
*
* We need to rebuild the descriptor if it's dirty even if we're not
* uploading, because our reachability testing *uses* our descriptor to
* determine what IP address and ports to test.
*/ */
void void
consider_publishable_server(time_t now, int force) consider_publishable_server(time_t now, int force)
{ {
int rebuilt = router_rebuild_descriptor(0);
if (decide_if_publishable_server(now)) { if (decide_if_publishable_server(now)) {
set_server_advertised(1); set_server_advertised(1);
if (router_rebuild_descriptor(0) == 0) if (rebuilt == 0)
router_upload_dir_desc_to_dirservers(force); router_upload_dir_desc_to_dirservers(force);
} else { } else {
set_server_advertised(0); set_server_advertised(0);