Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log PiHole dns stats to /var/log/pihole #27

Merged
merged 2 commits into from Jul 29, 2015
Merged

Log PiHole dns stats to /var/log/pihole #27

merged 2 commits into from Jul 29, 2015

Conversation

Fourdee
Copy link
Contributor

@Fourdee Fourdee commented Jul 28, 2015

Log PiHole dns stats to /var/log/pihole

  • Removes the requirement for rsyslog to be installed.
  • Prevents reading the shared logfile /var/log/daemon.log
  • Ensures PiHole dns stats are kept in its own logfile.

Log PiHole dns stats to /var/log/pihole
Removes the requirement for rsyslog and reading the shared logfile /var/log/daemon.log
@jacobsalmela
Copy link
Contributor

I do like the idea of the Pi-hole having its own log file. However, I would like the .log file extension added so it's easy to identify as a log file. Also, the file path change would need to be made here for anyone that is using the LCD display.

Will you commit those additions?

@Fourdee Fourdee mentioned this pull request Jul 29, 2015
@Fourdee
Copy link
Contributor Author

Fourdee commented Jul 29, 2015

Hi Jake,

As per your requests, all files have been updated to reflect /var/log/pihole.log
Web interface: pi-hole/web#1
LCD screen: #28

jacobsalmela added a commit that referenced this pull request Jul 29, 2015
Log Pi-hole DNS stats to /var/log/pihole.log instead of the shared /var/log/daemon.log
@jacobsalmela jacobsalmela merged commit 1a57e6f into pi-hole:master Jul 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants