Opened 9 months ago

Last modified 2 months ago

#22760 assigned defect

Fix extra-info flags on fallbacks

Reported by: teor Owned by:
Priority: Low Milestone: Tor: 0.3.4.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Minor Keywords: fallback
Cc: Actual Points:
Parent ID: #24786 Points: 0.5
Reviewer: Sponsor:

Description (last modified by teor)

See the first comment for details.

Child Tickets

Change History (7)

comment:1 Changed 9 months ago by teor

There are some more bugs we should fix here:

  • fallback_dir_server_new() shouldn't mark all fallbacks with ALL_DIRINFO
  • SKIP_MISSING_TRUSTED_EXTRAINFO() / router_supports_extrainfo() should believe the node, and only use router_digest_is_trusted_dir_type() if the node is NULL. (This allows descriptors to override hard-coded settings.)

comment:2 Changed 9 months ago by teor

We will backport the new fallback list format to 0.2.8 and later, so we might need two versions of this code:

  • a backport version that ignores the extrainfo tag
  • a master version that parses the tag, uses it to set the extrainfo type via fallback_dir_server_new(), and correctly handles the extrainfo type when bootstrapping.

comment:3 Changed 9 months ago by teor

Owner: set to teor
Status: newassigned

comment:4 Changed 6 months ago by teor

Milestone: Tor: 0.3.2.x-finalTor: 0.3.3.x-final

I'm not going to get time to do these in 0.3.2.
Moving them to 0.3.3.

comment:5 Changed 3 months ago by teor

Description: modified (diff)
Keywords: fallback added
Parent ID: #22759
Summary: Parse or ignore falback extra-info markersFix extra-info flags on fallbacks
Type: enhancementdefect
Version: Tor:

Since we never download extra-infos until we have the consensus, we should just fix the bugs listed in the first comment, and call this done.

comment:6 Changed 2 months ago by teor

Milestone: Tor: 0.3.3.x-finalTor: 0.3.4.x-final
Owner: teor deleted
Parent ID: #24786

We could do this as part of the next fallback rebuild if we wanted to.
I won't get time to do it in 0.3.3.

comment:7 Changed 2 months ago by teor

Priority: MediumLow
Severity: NormalMinor
Note: See TracTickets for help on using tickets.