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

Limit HTTP proxy listen address + add real_ip_header config option #914

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ptrsvbd
Copy link

@ptrsvbd ptrsvbd commented Jun 26, 2023

The HTTP proxy should respect IPv4 address set in configuration and not listen on 0.0.0.0 so it won't interfere with other daemons listening on the same port on different (specific) IP address.

When Evilginx is running behind a reverse HTTP proxy, the information on client IP address is lost. The client with IP address A.A.A.A connects to proxy at B.B.B.B, which then makes connection to Evilginx at C.C.C.C. The client IP address in Evilginx is determined from source address of the TCP connection and so B.B.B.B is logged and used in blacklist instead of A.A.A.A. The new config option real_ip_header adds possibility to read information on original client IP address from a specified HTTP header set by the reverse HTTP proxy (e.g. "X-Forwarded-For")

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