search

logdna / logdna-agent

LogDNA Agent streams from log files to your LogDNA account. Works with Linux, Windows, and macOS Servers
https://logdna.com
MIT License
138 stars 55 forks source link

Don't check for root user when program.config isset #187

Closed matthiaz closed 4 years ago

matthiaz commented 4 years ago

If a user defines a program.config, there is really no need to check the root user. We can assume the user wants to put the config in a folder that is not /etc. no? I don't see any other reason to have the index.js file check for root.

smusali commented 4 years ago

@matthiaz, thanks for bringing this up! There have been similar requests from other customers as well.

Honestly, it is not about just /etc/logdna.conf since we also tail /var/log/logdna-agent.log and/or /var/log/ by default; so, there might be so many other components requiring sudo access.

For the rest of this year, we are planning to make some changes including adding the option to run the agent as a non-root user but there is no exact ETA for now.