Merge remote-tracking branch 'dgoulet/bug14202_026_v1'

This commit is contained in:
Nick Mathewson 2015-02-02 10:16:48 -05:00
commit 55639bc67f
2 changed files with 4 additions and 8 deletions

3
changes/bug14202 Normal file
View File

@ -0,0 +1,3 @@
o Minor cleanup
- Remove workaround in dirserv_thinks_router_is_hs_dir() that was only
for version <= 0.2.2.24 which is now deprecated.

View File

@ -1305,14 +1305,7 @@ dirserv_thinks_router_is_hs_dir(const routerinfo_t *router,
else
uptime = real_uptime(router, now);
/* XXX We shouldn't need to check dir_port, but we do because of
* bug 1693. In the future, once relays set wants_to_be_hs_dir
* correctly, we can revert to only checking dir_port if router's
* version is too old. */
/* XXX Unfortunately, we need to keep checking dir_port until all
* *clients* suffering from bug 2722 are obsolete. The first version
* to fix the bug was 0.2.2.25-alpha. */
return (router->wants_to_be_hs_dir && router->dir_port &&
return (router->wants_to_be_hs_dir &&
uptime >= get_options()->MinUptimeHidServDirectoryV2 &&
router_is_active(router, node, now));
}