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

Feature Request: Option to rewrite source ip dynamically to match incoming packet source address #1

Closed
technomancer-101 opened this issue Jul 4, 2023 · 5 comments

Comments

@technomancer-101
Copy link

Hi there,

I'd like to start with THANK YOU. Your program allows us to run some client software over a VPN that inexplicably uses a broadcast packet to locate its host server every single time you open it.

Any chance you would be able to implement an option for the source IP to be dynamically rewritten based on the origin of the broadcast packet? Everything works great as-is, but I can only use it for one host at a time. I would love to help out, but unfortunately I don't know Go.

Thanks.

@hwipl
Copy link
Owner

hwipl commented Jul 4, 2023

Hey, thanks for the positive feedback :)

Just to make sure I understand your request correctly: You want to run the program on some host in your network. It receives broadcast packets from other hosts in the network and forwards them over a VPN tunnel. Reply packets could go directly from the VPN tunnel to the original hosts, but the source address rewriting gets into your way (for more than one other host). So, you would basically need an option to just keep the original source IP address in the packets, right?

@technomancer-101
Copy link
Author

Yep, you got it. That's exactly what I'm hoping for.
Other than the fact that it can only point at one original IP right now, it's working perfectly.

Thanks again :)

@hwipl
Copy link
Owner

hwipl commented Jul 5, 2023

I added the command line argument "-keep-source-ip" that should disable the source address rewriting. Could you test if it works for you?

@technomancer-101
Copy link
Author

Testing looks good. Opened our client from a handful of computers simultaneously and all packets are relaying correctly between the server and each client respectively.
Thank you again, this helps us a lot!

@hwipl
Copy link
Owner

hwipl commented Jul 5, 2023

Great, closing this issue for now. If anything breaks, feel free to re-open or create a new one :)

@hwipl hwipl closed this as completed Jul 5, 2023
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