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

[FEAT] getting real client IP #144

Closed
venkatamutyala opened this issue Nov 30, 2024 · 6 comments
Closed

[FEAT] getting real client IP #144

venkatamutyala opened this issue Nov 30, 2024 · 6 comments

Comments

@venkatamutyala
Copy link
Contributor

I'm using chisel operator with ingress nginx it looks like by default I can only see the IP of the chisel pods. Is there a way to get the client IP? I'd like to leverage it to do some advance operations based on IP within my nginx setup.

Copy link

linear bot commented Nov 30, 2024

@korewaChino
Copy link
Member

korewaChino commented Dec 1, 2024

I don't think so, unless Chisel has an option to pass a HTTP header when it detects HTTP traffic.

Chisel is a first and foremost TCP and UDP tunnel, so I don't think this is the case. The chisel operator also treats the tunnel like any normal TCP traffic by specifically setting ip:port/tcp in the args, so it does not process any headers.

@korewaChino
Copy link
Member

I found an issue related here jpillora/chisel#266

@venkatamutyala
Copy link
Contributor Author

venkatamutyala commented Dec 2, 2024

Thanks Cappy. I had a few minutes to do some more research, and it appears if Chisel supported the PROXY Protocol (v2?), it would send over the data needed for my ingress (e.g., nginx) to detect the IP. I opened up a ticket here: jpillora/chisel#540

ref:
https://inlets.dev/blog/2022/09/02/real-client-ips-with-proxy-protocol.html
https://aws.amazon.com/blogs/networking-and-content-delivery/preserving-client-ip-address-with-proxy-protocol-v2-and-network-load-balancer/

@korewaChino
Copy link
Member

Will this require any changes to the operator?

@venkatamutyala
Copy link
Contributor Author

I think at this time it makes sense to close this out until the support exists upstream.

@venkatamutyala venkatamutyala closed this as not planned Won't fix, can't repro, duplicate, stale Dec 5, 2024
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

No branches or pull requests

2 participants