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

Show original IP address of requests in AWS logs #1131

Merged
merged 1 commit into from
Feb 17, 2021
Merged

Show original IP address of requests in AWS logs #1131

merged 1 commit into from
Feb 17, 2021

Conversation

andreyv
Copy link
Member

@andreyv andreyv commented Aug 22, 2019

See also the previous discussion: https://gitter.im/cms-dev/cms?at=5bfa905597a8982b9a606c79

For now I implemented a bare-bones approach with the hidden config setting. It can be developed further:

  • Either by documenting the new setting in cms.conf
  • Or by refactoring num_proxies_used to include both CWS and AWS

I can rework this PR if needed.


This change is Reviewable

@andreyv
Copy link
Member Author

andreyv commented Feb 17, 2021

Let's make the new setting separate. Initializing it from num_proxies_used by default may have security implications.

@andreyv andreyv merged commit 9236084 into cms-dev:master Feb 17, 2021
@andreyv andreyv deleted the aws-original-ip branch February 17, 2021 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant