Opened 3 years ago

Last modified 22 months ago

#20619 needs_review defect

bwauths don't handle NEWCONSENSUS event correctly

Reported by: teor Owned by: tom
Priority: Medium Milestone:
Component: Core Tor/pytorctl Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: #17064 Points:
Reviewer: Sponsor:

Description

Instead, they throw an error about a missing path_selector every time a new consensus arrives.

(It's possible some of my other code changes might be causing this error to surface.)

Child Tickets

TicketStatusOwnerSummaryComponent
#20603closedaagbsnTorFlow's PathSupport can use path_selector when uninitialisedCore Tor/Torflow

Attachments (1)

10002-Make-sure-path_selector-is-initialised-before-using-.patch (2.7 KB) - added by teor 3 years ago.
Patch that initialises path_selector, and tries to regenerate it when it's missing

Download all attachments as: .zip

Change History (8)

Changed 3 years ago by teor

Patch that initialises path_selector, and tries to regenerate it when it's missing

comment:1 Changed 3 years ago by teor

Status: newneeds_review

comment:2 Changed 22 months ago by teor

Owner: changed from aagbsn to tom
Status: needs_reviewassigned

I would recommend this as a stability fix for testing.

comment:3 Changed 22 months ago by teor

Parent ID: #17064

comment:4 Changed 22 months ago by teor

(This is a pytorctl patch.)

comment:5 Changed 22 months ago by teor

Component: Core Tor/TorflowCore Tor/pytorctl

comment:6 Changed 22 months ago by teor

Status: assignedneeds_review

If you prefer this as a branch, it's bug20619 on https://github.com/teor2345/pytorctl.git

comment:7 Changed 22 months ago by teor

After we commit this, we need to update the TorCtl submodule in torflow, see #24225.

Note: See TracTickets for help on using tickets.