Opened 2 months ago

Last modified 7 weeks ago

#28795 new task

Generate a new fallback list in 2019 and backport it to all supported Tor versions

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: fallback, 029-backport, 034-backport, 035-backport, 040-backport
Cc: teor, starlight@… Actual Points:
Parent ID: #28793 Points:
Reviewer: Sponsor:

Description (last modified by teor)

We usually get two people to run:

scripts/maint/updateFallbackDirs.py > src/or/fallback_dirs.inc 2> fallback_dirs.log

from two different locations, then merge the lists.

Please attach the logs to this ticket.

Child Tickets

Change History (6)

comment:1 Changed 2 months ago by teor

Component: Core Tor/Fallback ScriptsCore Tor/Tor
Description: modified (diff)
Keywords: 029-backport 034-backport 035-backport 040-backport added

comment:2 Changed 2 months ago by starlight

Cc: starlight@… added

comment:3 Changed 2 months ago by teor

Type: defecttask

comment:4 Changed 2 months ago by starlight

to view full explanations for reasons why relays are excluded this edit is needed

@@ -2214,4 +2219,4 @@                                                           
 def process_default():                                                         
-  logging.basicConfig(level=logging.WARNING)
-  logging.getLogger('stem').setLevel(logging.WARNING)
+  logging.basicConfig(level=logging.INFO)
+  logging.getLogger('stem').setLevel(logging.INFO)
   whitelist = {'data': read_from_file(WHITELIST_FILE_NAME, MAX_LIST_FILE_SIZE),
@@ -2235,3 +2240,3 @@
   if get_command() == 'check_existing':
-    logging.warning(msg, *args)
+    logging.info(msg, *args)
   else:

otherwise little is revealed

comment:5 Changed 7 weeks ago by teor

We don't use check_existing mode when generating the list, so the final part of the patch shouldn't be needed.

Maybe we should make warn the default, and add an argument for info. Or the other way around.

comment:6 in reply to:  5 Changed 7 weeks ago by teor

This is the ticket for list generation. I opened #28986 so we don't forget your change.

Replying to teor:

We don't use check_existing mode when generating the list, so the final part of the patch shouldn't be needed.

But it would be good to be consistent.

Maybe we should make warn the default, and add an argument for info. Or the other way around.

Let's make info-level the default, so we don't have to remember extra options.

Note: See TracTickets for help on using tickets.