Fix audit log to record client real IP instead of ALB internal IP#37
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughnginxプロキシの Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
…logs Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Summary
X-Forwarded-Forfrom$remote_addrto$proxy_add_x_forwarded_forto preserve the CloudFront→ALB IP chain instead of overwriting it with ALB's internal IPTrustedProxiesto RFC 1918 ranges soc.ClientIP()correctly extracts the first non-private IP from the X-Forwarded-For chainX-Forwarded-Port(Nginx's$server_port= 8080) from audit logremotefield, log IP onlyTest plan
docker build --target test runner/passes with 100% coverageremote=in audit logs shows client global IP instead of10.x.x.x:8080🤖 Generated with Claude Code
Summary by CodeRabbit
リリースノート