Dynatrace / Dynatrace-AppMon-Puppet-DEPRECATED

Installs the Dynatrace Application Monitoring solution using Puppet.
MIT License
20 stars 12 forks source link

The PID file shouldn't be created in a temporary folder #37

Open tux-o-matic opened 7 years ago

tux-o-matic commented 7 years ago

In the init.d service script, there is a hard coded value for DT_PIDFILE_DIR whichis /tmp. Writing the pid file to a temporary folder is a dangerous thing, there is no persistency guarantee and a system or administrator could easily delete the PID file. This would lead to a situation where the service seams dead when it is in fact still running, Puppet would then try to restart it and face port conflict.