fix: correct X-Forward-For typo to X-Forwarded-For in nginx configs#9019
fix: correct X-Forward-For typo to X-Forwarded-For in nginx configs#9019cybertron288 wants to merge 1 commit intomakeplane:previewfrom
Conversation
|
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This PR #8935 already fixes this. |
Closes #8934
What
Corrects the
real_ip_headerdirective inapps/web,apps/admin, andapps/spacenginx configs from the non-existentX-Forward-Forheader to the standardX-Forwarded-For.Why
X-Forward-Foris not a real HTTP header, so nginx never replaces$remote_addrwith the actual client IP. As a result,limit_req_zone $binary_remote_addrrate-limits per-proxy instead of per-client and access logs record the proxy IP. See #8934.Notes