Opened 3 weeks ago

Last modified 2 weeks ago

#25116 merge_ready defect

hs: circuit_log_ancient_one_hop_circuits() should probably not log single onion service rendezvous circuit

Reported by: dgoulet Owned by: dgoulet
Priority: Medium Milestone: Tor: 0.3.2.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-hs, sos, easy, 031-backport 029-backport 032-backport
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Assume an email server where clients often keep a connection open and regularly exchange traffic on them.

Making that email server an .onion, RP circuits will stay open for a while and more than 1800 seconds which is the cutoff of circuit_log_ancient_one_hop_circuits() to log single hop circuits.

I think we want to ignore to log anything service related in there. Some v3 services have started seeing that heartbeat more and more in the last days.

Related: #8387

Child Tickets

Change History (7)

comment:1 Changed 3 weeks ago by dgoulet

Owner: set to dgoulet
Status: newaccepted

comment:2 Changed 2 weeks ago by dgoulet

Status: acceptedneeds_review

Branch: bug25116_033_01

comment:3 Changed 2 weeks ago by nickm

LGTM. Is this a bug against some earlier branch, though?

comment:4 Changed 2 weeks ago by dgoulet

Yes goes back to 029 but I wasn't sure this would qualify for a backport. If yes, I'm happy to provide an 029 branch.

comment:5 Changed 2 weeks ago by nickm

Let's have the branch against 0.2.9, so that we can backport it in case anybody complains about getting these errors in earlier versions. It's a simple enough fix.

comment:6 Changed 2 weeks ago by dgoulet

Sure, 029 branch: bug25116_029_01

Commit 7ce8d5513ba38825. Hope that is ok!

comment:7 Changed 2 weeks ago by nickm

Keywords: 031-backport 029-backport 032-backport added
Milestone: Tor: 0.3.3.x-finalTor: 0.3.2.x-final
Status: needs_reviewmerge_ready

great! merging to 0.3.3, marking for possible backport.

Note: See TracTickets for help on using tickets.