[tor-commits] [tor] 02/08: Retain all routerinfos listed in the consensus.

[gitolite role]

This is an automated email from the git hooks/post-receive script.

dgoulet pushed a commit to branch main
in repository tor.

commit db7d067ab178e90163bd013c3892ca707a846433
Author: Nick Mathewson <nickm at torproject.org>
AuthorDate: Tue Nov 9 08:49:07 2021 -0500

    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));
   }
 

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.