Include the ngx_http_log_module in the nginx build

[nealdb]

---

Desired Behavior

Include the ngx_http_log_module in the nginx build to allow full urls to be shown and correct IPs when behind Cloudflare

Current Behavior

Since the default behaviour of SWAG is to place logging for all sites proxied it can be hard to determine which sites are being requested.
I'd like to be able to use the $host variable in the logs to show the site being requested

Also since putting my sites behind Cloadflare's DNS proxy I only see Cloudflare IP's in my logs. the ngx_http_log_module would allow use of the $http_x_forwarded_for too which would show the originating IP

Alternatives Considered

I was considering alternate log files per site but wasn't sure if the dashboard would pick them up

[github-actions]

Thanks for opening your first issue here! Be sure to follow the bug or feature issue templates!

[aptalca]

What makes you think it's not included?

[nemchik]

We have a docker mod https://github.com/linuxserver/docker-mods/tree/swag-cloudflare-real-ip that does what you're asking. If you have trouble getting it setup we can help on our discord.

[nealdb]

Thanks for the mod I've now enabled that and it's logging the real IP

But how can I get the full hostname logged so I can see which of the multiple sites swag is proxying added to the logs? As the logs only show the request after the hostname eg /remote.php/dav/files/gloria/ not https://mybloodyawesomeshiz.site/remote.php/dav/files/neal/

@aptalca I can tell it's not included as it's not in the list of included modules if I run nginx -V

[nemchik]

@aptalca I can tell it's not included as it's not in the list of included modules if I run nginx -V

In that case, it must not be required, because the mod doesn't install anything, it just adds a configuration file for nginx, which just enables features that are already installed (but not enabled by default).

But how can I get the full hostname logged so I can see which of the multiple sites swag is proxying added to the logs? As the logs only show the request after the hostname eg /remote.php/dav/files/gloria/ not https://mybloodyawesomeshiz.site/remote.php/dav/files/neal/

You would have to change the default log format found in nginx.conf which is configured as access_log. The default value we ship in the config is here https://github.com/linuxserver/docker-baseimage-alpine-nginx/blob/8ec61214bf1294e9d5aa93d18b9837f26442d1a0/root/defaults/nginx/nginx.conf.sample#L73 which does not include any specific formatting (it falls back to the default nginx format for the log) but you can modify that line in your config to achieve what you want.

[nealdb]

Ahh gotcha thanks. I'd specified a new log format in my nginx.conf but hadn't told nginx to use that format.

So problem between chair and keyboard.
Thanks again