From db7d067ab178e90163bd013c3892ca707a846433 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Tue, 9 Nov 2021 08:49:07 -0500 Subject: [PATCH] Retain all routerinfos listed in the consensus. Previously we'd look at the routerstatus published_on field when deciding what to dump, which really has no point. If something's in the consensus with an ancient published date, then we do want to keep it. --- src/feature/nodelist/routerlist.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/feature/nodelist/routerlist.c b/src/feature/nodelist/routerlist.c index c00f7ffb26..2d6ae13711 100644 --- a/src/feature/nodelist/routerlist.c +++ b/src/feature/nodelist/routerlist.c @@ -1924,11 +1924,9 @@ routerlist_remove_old_routers(void) retain = digestset_new(n_max_retain); } - cutoff = now - OLD_ROUTER_DESC_MAX_AGE; /* Retain anything listed in the consensus. */ if (consensus) { SMARTLIST_FOREACH(consensus->routerstatus_list, routerstatus_t *, rs, - if (rs->published_on >= cutoff) digestset_add(retain, rs->descriptor_digest)); }