Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#3183 closed defect (fixed)

heartbeat message inappropriate for bridge

Reported by: keb Owned by:
Priority: Medium Milestone:
Component: Core Tor/Tor Version: Tor: 0.2.3.1-alpha
Severity: Keywords: asn heartbeat bridge consensus tor-bridge
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

[11:39] j what does this mean, is it important? if so, what do I do to fix it? "[Notice] Heartbeat: It seems like we are not in the cached consensus."
...
[13:08] rransom Then you compiled without bufferevents enabled, and it was still broken.
...
[13:09] rransom (Or you just hit the same intermittent brokenness that we've had for ages, even on 0.2.2.x--- A bridge shouldn't be in the consensus.
[13:09] rransom j
: Were you running a bridge or a public relay?
[13:11] j I'm running a bridge
...
[13:12] rransom j
: If you were running a bridge, there may have been nothing wrong with it.

Child Tickets

Change History (5)

comment:1 in reply to:  description Changed 9 years ago by rransom

Replying to keb:

[11:39] j__ what does this mean, is it important? if so, what do I do to fix it? "[Notice] Heartbeat: It seems like we are not in the cached consensus."
...
[13:08] rransom Then you compiled without bufferevents enabled, and it was still broken.
...
[13:09] rransom (Or you just hit the same intermittent brokenness that we've had for ages, even on 0.2.2.x--- A bridge shouldn't be in the consensus.
[13:09] rransom j__: Were you running a bridge or a public relay?
[13:11] j__ I'm running a bridge
...
[13:12] rransom j__: If you were running a bridge, there may have been nothing wrong with it.

comment:2 Changed 9 years ago by asn

Status: newneeds_review

comment:3 Changed 9 years ago by nickm

Resolution: fixed
Status: needs_reviewclosed

Hi! Looks ok to me. Two quick things:

It's way easier for me to merge a branch by name than to merge a commit by digest. This time I guessed the branch name "heartbeat", but in the future it wold be great if you can say the branch name as well as or instead of just linking to a gitorious pge.

Second, this needs a changes file. I added one for bug3183 (since I was already merging it).

comment:4 Changed 8 years ago by nickm

Keywords: tor-bridge added

comment:5 Changed 8 years ago by nickm

Component: Tor BridgeTor
Note: See TracTickets for help on using tickets.