[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #28932 [Core Tor/sbws]: Detect when sbws stalls and log backtrace
#28932: Detect when sbws stalls and log backtrace
---------------------------+-----------------------------------
Reporter: juga | Owner: (none)
Type: defect | Status: needs_revision
Priority: Medium | Milestone: sbws: 1.0.x-final
Component: Core Tor/sbws | Version: sbws: 1.0.2
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: #28663 | Points:
Reviewer: dgoulet | Sponsor:
---------------------------+-----------------------------------
Changes (by dgoulet):
* status: needs_review => needs_revision
Comment:
I think we should make `36` value into some sort of constant because it
isn't clear what is that value but also it is repeated twice.
I had to read the loop above to get that it is every 5 seconds and thus 36
is for 3 minutes. Either adding a clearer comment linking the two or
making it a constant and comment there.
The other thing is that, don't you want to exit after the bug is detected
or it will recover from the mistake?
Feel free to merge after that :).
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/28932#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs