Opened 3 years ago

Closed 3 years ago

Last modified 17 months ago

#19530 closed defect (fixed)

Bug: Directory m��V�W�ٱq����/(��>� delivered a consensus, but a routerstatus could not be found for it.

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

Description

Running a normal Tor client from git master, using a normal bridge, I consistently get

Jun 29 09:16:52.626 [warn] authority_certs_fetch_missing(): Bug: Directory m��V�W�ٱq����/(��>� delivered a consensus, but a routerstatus could not be found for it. (on Tor 0.2.9.0-alpha-dev 3cdc8bfa2c995b3b)

There are two problems. First, we shouldn't be printing an identity digest without some sort of encoding. Second, the comments around dir_hint make it sound like it wasn't expecting to get a non-null dir_hint while we're using a bridge.

Looks like these issues were introduced in #18963.

Child Tickets

Change History (5)

comment:1 Changed 3 years ago by arma

"git show 730cfeb6" for those who want to investigate more

comment:2 Changed 3 years ago by teor

Owner: set to teor
Status: newassigned

I guess that makes it me then.

comment:3 Changed 3 years ago by teor

Actual Points: 0.5
Points: 0.5
Status: assignedneeds_review

Please see my branch bug19530-v2 on https://github.com/teor2345/tor.git

It hex-encodes the fingerprint before printing it, and correctly supports retrying a successful bridge when downloading certificates.

It also makes some consequential changes to the code to make it simpler and more robust.

comment:4 Changed 3 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

merged to master!

comment:5 Changed 17 months ago by teor

Cc: teor removed

Remove useless CC

Note: See TracTickets for help on using tickets.