Opened 5 years ago

Closed 5 years ago

#4602 closed defect (fixed)

Arm's torrc validation misses alternate Log default

Reported by: atagar Owned by: atagar
Priority: Medium Milestone:
Component: Core Tor/Nyx Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Tor reports a default log value ("Log notice stdout") if none is defined as per...
https://trac.torproject.org/projects/tor/ticket/2362

However, with the deb it defaults to "Log notice file /var/log/tor/log" instead, causing validation to think that this is an issue. I should account for this too.

Thanks to Sebastian, here's the part that determines this...

12:31 < Sebastian> atagar:
12:31 < Sebastian> + if (debian_running_as_debiantor()) {
12:31 < Sebastian> + if (!options->Logs && !from_setconf)
12:31 < Sebastian> + config_line_append(&options->Logs, "Log", "notice file /var/log/tor/log");

Child Tickets

Change History (1)

comment:1 Changed 5 years ago by atagar

  • Resolution set to fixed
  • Status changed from new to closed

Issue reproed on a Tor Cloud image and fixed...
https://gitweb.torproject.org/arm.git/commitdiff/13b9ffe93748f4af0cd886ba330a256e07de2b2f

Thanks Sebastian and koolfy!

Note: See TracTickets for help on using tickets.