AdguardTeam / AdGuardHome

Network-wide ads & trackers blocking DNS server
https://adguard.com/adguard-home/overview.html
GNU General Public License v3.0
25.63k stars 1.84k forks source link

Allow to change the default permissions on query logfile #7389

Closed nycom83 closed 3 weeks ago

nycom83 commented 3 weeks ago

Prerequisites

The problem

I have an external program that read query logfile to build a dashboard. But since the last version (v0.107.53), default permissions on query logfile have been changed from 0644 to 0600.

Proposed solution

Give the ability to change the default permissions on query log file

Alternatives considered and additional information

No response

EugeneOne1 commented 3 weeks ago

@nycom83, hello. The permissions for each file are checked on every run, but the actual change is only made when the configuration file (AdGuardHome.yaml) has invalid permissions. So you can freely manipulate the access to the querylog file, but AdGuard Home will print a warning message about it, as setting such permissions is not considered safe.

handymenny commented 3 weeks ago

@EugeneOne1 The issue is that the permissions revert to the default ones when the logs rotate: image