Opened 6 months ago

Last modified 7 weeks ago

#29053 new enhancement

Use descriptor structs to deterministically create descriptor dumps

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: technical-debt
Cc: Actual Points:
Parent ID: Points: 2
Reviewer: Sponsor:

Description

We should make router_dump_router_to_string() and extrainfo_dump_to_string() deterministic transforms of routerinfo_t and extrainfo_t.

At the moment, these functions access options, internal state, and statistics files. If all those accesses occurred in router_build_fresh_unsigned_{router,extra}info() then the dump functions would be a lot easier to test.

Child Tickets

Change History (5)

comment:1 Changed 6 months ago by teor

I don't think we have time for this in 0.4.1, and I think it's out of scope for Sponsor V. But I'll ask the team next meeting.

comment:2 Changed 6 months ago by teor

Keywords: 041-proposed removed

I think our consensus is that this ticket would be cool, but it's out of scope for 0.4.1.

comment:3 Changed 6 months ago by teor

Parent ID: #29017

Un-parenting this, because it is out of scope for 041.

comment:4 Changed 7 weeks ago by gaba

Removing sponsor V as we do not have more time to include this tickets in the sponsor.

comment:5 Changed 7 weeks ago by gaba

Sponsor: SponsorV-can

Removing sponsor from tickets that we do not have time to fit in the remain of this sponsorship.

Note: See TracTickets for help on using tickets.