Opened 6 months ago

Last modified 4 days ago

#22760 assigned defect

Fix extra-info flags on fallbacks

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Normal Keywords: fallback
Cc: Actual Points:
Parent ID: Points: 0.5
Reviewer: Sponsor:

Description (last modified by teor)

See the first comment for details.

Child Tickets

Change History (5)

comment:1 Changed 5 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 5 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 5 months ago by teor

Owner: set to teor
Status: newassigned

comment:4 Changed 3 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 4 days 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.

Note: See TracTickets for help on using tickets.