Opened 5 months ago

Last modified 13 days 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, 029-backport, 032-backport, 031-unreached-backport
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

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 (8)

comment:1 Changed 5 months ago by dgoulet

Owner: set to dgoulet
Status: newaccepted

comment:2 Changed 5 months ago by dgoulet

Status: acceptedneeds_review

Branch: bug25116_033_01

comment:3 Changed 5 months ago by nickm

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

comment:4 Changed 5 months 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 5 months 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 5 months ago by dgoulet

Sure, 029 branch: bug25116_029_01

Commit 7ce8d5513ba38825. Hope that is ok!

comment:7 Changed 5 months 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.

comment:8 Changed 13 days ago by teor

Keywords: 031-unreached-backport added; 031-backport removed

0.3.1 is end of life, there are no more backports.
Tagging with 031-unreached-backport instead.

Note: See TracTickets for help on using tickets.