Opened 22 months ago

Closed 7 months ago

#25632 closed enhancement (implemented)

Improve stem torrc logging options for integration testing

Reported by: dmr Owned by: atagar
Priority: Medium Milestone:
Component: Archived/Stem Version:
Severity: Normal Keywords: testing easy
Cc: dmr Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


As suggested by teor in #25631, utilizing the following options in torrc will improve logging for the purpose of integration testing in stem:

  • ProtocolWarnings 1
  • SafeLogging 0
  • LogTimeGranularity 1

The former (ProtocolWarnings 1) should be used in general with a local test relay (i.e. during development, independent of integration testing), and the log [warn]ings surfaced automatically in integration tests, which I believe currently isn't the case. The [warn]ings should potentially be treated as test failures when emitted (with exception of tests that directly try to violate the protocol, and check that Tor responds as such).

The latter two will help with debugging when a problem is encountered.

Child Tickets

Change History (4)

comment:1 Changed 21 months ago by atagar

Keywords: testing easy added

comment:2 Changed 7 months ago by atagar

Resolution: worksforme
Status: newclosed

teor has been working with me on Tor/Stem CI and hasn't mentioned this of late so gonna guess this is no longer needed. If so, it's a trivial tweak to make.

comment:3 Changed 7 months ago by teor

Resolution: worksforme
Status: closedreopened

No, these changes are still useful.

I'm not sure what we want to do about ProtocolWarnings, but having them in the logs or console output would be helpful.

comment:4 Changed 7 months ago by atagar

Resolution: implemented
Status: reopenedclosed

Sounds good! Done...

Feel free to reopen if you need anything else.

Note: See TracTickets for help on using tickets.