Changes log filenames from ID to (first) domain

[joewesch]

Closes: #746

This change would be slightly not backwards compatible as the log filenames would change for all new logs and all existing files would be abandoned and not pruned. Any guidance or assistance to add this as a setting in the UI or something rather than just change the default would be welcomed.

[KGBist2000]

Hi Joewesch, nice change. Did you checked what will happen, if more then one domain is entered into the gui by creating a new host ?

[joewesch]

Hello @KGBist2000.

Yes, it uses the alphabetically first domain in the filename. For example, if you create a proxy host with the 2 domains beta.example.com and alpha.example.com the log files would be proxy-host-alpha.example.com_access.log and proxy-host-alpha.example.com_error.log.

I have been using this method for quite some time with no problems starting shortly after I opened the linked issue. I only just recently was reminded I never put in a PR via a recent comment on the issue.

[csantospminds]

I'm very interested in this, any idea if it's possible to change the log filename from ID to domain? Adding the option to read the logs in the GUI would also be cool because right now it's difficult to find a particular log for a dozen or more vhosts.

[hirenshah]

Could this be expanded to name the conf files in a similar manner?

[pXius]

Is this PR still being considered?

[joewesch]

Pinging @jc21 to see if there is any feedback?

[github-actions[bot]]

PR is now considered stale. If you want to keep it open, please comment :+1:

[nginxproxymanagerci[bot]]

Docker Image for build 10 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-2606

[!NOTE] Ensure you backup your NPM instance before testing this image! Especially if there are database changes. This is a different docker image namespace than the official image.

[!WARNING] Changes and additions to DNS Providers require verification by at least 2 members of the community!