Opened 4 years ago

Closed 4 years ago

#16286 closed defect (fixed)

[warn] Router $9695DFC35FFEB861329B9F1AB04C46397020CE31~moria1 at 128.31.0.34 uploaded a descriptor with no Ed25519 key, when we previously knew an Ed25519 for it.

Reported by: arma Owned by: nickm
Priority: Very Low Milestone: Tor: 0.2.7.x-final
Component: Core Tor/Tor Version:
Severity: Keywords: TorCoreTeam201508
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Starting moria1 on git master I get:

Jun 03 10:25:40.272 [warn] Router $9695DFC35FFEB861329B9F1AB04C46397020CE31~moria1 at 128.31.0.34 uploaded a descriptor with no Ed25519 key, when we previously knew an Ed25519 for it. Ignoring for now, since Tor 0.2.6 is under development.
Jun 03 10:25:40.272 [warn] Router $9695DFC35FFEB861329B9F1AB04C46397020CE31~moria1 at 128.31.0.34 uploaded a descriptor with no Ed25519 key, when we previously knew an Ed25519 for it. Ignoring for now, since Tor 0.2.6 is under development.
Jun 03 10:25:40.272 [warn] Router $9695DFC35FFEB861329B9F1AB04C46397020CE31~moria1 at 128.31.0.34 uploaded a descriptor with no Ed25519 key, when we previously knew an Ed25519 for it. Ignoring for now, since Tor 0.2.6 is under development.

I suspect this word 'uploaded' is not right, and instead that this is moria1 loading cached descriptors on startup.

Child Tickets

Change History (4)

comment:1 Changed 4 years ago by nickm

Keywords: TorCoreTeam201508 added
Owner: set to nickm
Priority: normaltrivial
Status: newaccepted

comment:2 Changed 4 years ago by nickm

Status: acceptedneeds_review

My branch bug16286 has a fix for this, I think

comment:3 in reply to:  2 Changed 4 years ago by isis

Replying to nickm:

My branch bug16286 has a fix for this, I think


Hello nickm!

I reviewed your patch. It's quite a simple change (it basically just changes dirserv_router_get_status() to take a log severity level, rather than always logging everything at the LOG_WARN level. It looks fine to me. I also built it and ran make test.

comment:4 Changed 4 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Thanks! Just merged it to master.

Note: See TracTickets for help on using tickets.