[tor-commits] [tor/release-0.2.8] Confim we want certificates from fallbacks

nickm at torproject.org nickm at torproject.org
Thu May 12 19:35:50 UTC 2016


commit 797ece042d9666beae3ffc801b148e8bebcd88c0
Author: teor (Tim Wilson-Brown) <teor2345 at gmail.com>
Date:   Wed May 11 13:08:45 2016 -0400

    Confim we want certificates from fallbacks
    
    Comment-only change
---
 src/or/routerlist.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 85e9e7d..d40d704 100644
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -913,7 +913,8 @@ authority_certs_fetch_missing(networkstatus_t *status, time_t now)
 
     if (smartlist_len(fps) > 1) {
       resource = smartlist_join_strings(fps, "", 0, NULL);
-      /* XXX - do we want certs from authorities or mirrors? - teor */
+      /* We want certs from mirrors, because they will almost always succeed.
+       */
       directory_get_from_dirserver(DIR_PURPOSE_FETCH_CERTIFICATE, 0,
                                    resource, PDS_RETRY_IF_NO_SERVERS,
                                    DL_WANT_ANY_DIRSERVER);
@@ -959,7 +960,8 @@ authority_certs_fetch_missing(networkstatus_t *status, time_t now)
 
     if (smartlist_len(fp_pairs) > 1) {
       resource = smartlist_join_strings(fp_pairs, "", 0, NULL);
-      /* XXX - do we want certs from authorities or mirrors? - teor */
+      /* We want certs from mirrors, because they will almost always succeed.
+       */
       directory_get_from_dirserver(DIR_PURPOSE_FETCH_CERTIFICATE, 0,
                                    resource, PDS_RETRY_IF_NO_SERVERS,
                                    DL_WANT_ANY_DIRSERVER);





More information about the tor-commits mailing list