Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config: add trusted_proxies to Caddyfile #146

Merged
merged 1 commit into from
Jan 26, 2025
Merged

Conversation

chuckadams
Copy link
Contributor

Pull Request

What changed?

Adds this stanza to the Caddyfile:

    servers {
        trusted_proxies static private_ranges
        trusted_proxies_strict
    }

As usual with Caddy, the writeup is more typing than the change itself 😛

Why did it change?

This enables the client_ip field in caddy logfiles, which is the actual originating IP, not the ingress/balancer/proxy. Docker desktop users might not see any difference, but it should do the trick on staging/prod.

Did you fix any specific issues?

none

CERTIFICATION

By opening this pull request, I do agree to abide by the Code of Conduct and be bound by the terms of the Contribution Guidelines in effect on the date and time of my contribution as proven by the revision information in GitHub.

@chuckadams chuckadams merged commit 179cdc1 into main Jan 26, 2025
1 check passed
@chuckadams chuckadams deleted the chuck/dev/log-proxy-ips branch January 26, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant