Opened 8 years ago

Last modified 2 years ago

#3037 new enhancement

Internal checks to detect client streams/circuits whose sock request vanished

Reported by: arma Owned by:
Priority: Medium Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-client, debugging, diagnostic
Cc: Actual Points:
Parent ID: Points: 3
Reviewer: Sponsor:

Description

Tor has a cycle for handling streams, where it attaches the stream to a circuit, gives up after a while, attaches it to another, etc.

Tor also has a timeout after which it hangs up on the socks connection, and some logic to notice that a socks connection has hung up so it should cancel streams.

Long ago, it was quite common for me to see the "keep trying" cycle continue even after the socks request had vanished. Those were bugs.

I haven't seen them so much recently, but it would be smart to add some logic to check the consistency of "for each thing we're trying to handle, is there still a connection around that's asking us to handle it." If we had that logic, it might help catch the root cause of #2983.

Child Tickets

Change History (9)

comment:1 Changed 8 years ago by nickm

Milestone: Tor: 0.2.3.x-finalTor: 0.2.4.x-final

Worth doing, but it's a feature.

comment:2 Changed 7 years ago by nickm

Keywords: tor-client added

comment:3 Changed 7 years ago by nickm

Component: Tor ClientTor

comment:4 Changed 6 years ago by nickm

Milestone: Tor: 0.2.4.x-finalTor: 0.2.5.x-final

comment:5 Changed 5 years ago by nickm

Milestone: Tor: 0.2.5.x-finalTor: 0.2.???

comment:6 Changed 3 years ago by teor

Milestone: Tor: 0.2.???Tor: 0.3.???

Milestone renamed

comment:7 Changed 3 years ago by nickm

Keywords: tor-03-unspecified-201612 added
Milestone: Tor: 0.3.???Tor: unspecified

Finally admitting that 0.3.??? was a euphemism for Tor: unspecified all along.

comment:8 Changed 2 years ago by nickm

Keywords: tor-03-unspecified-201612 removed

Remove an old triaging keyword.

comment:9 Changed 2 years ago by nickm

Keywords: debugging diagnostic added
Points: 3
Severity: Normal
Note: See TracTickets for help on using tickets.