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

Customize the IPs and private keys #4

Open
KusakabeShi opened this issue Jan 4, 2024 · 4 comments · Fixed by #5
Open

Customize the IPs and private keys #4

KusakabeShi opened this issue Jan 4, 2024 · 4 comments · Fixed by #5

Comments

@KusakabeShi
Copy link

KusakabeShi commented Jan 4, 2024

I want an option to customize the IPs and wireguard keys for em_proxy while initialization.

Add 4 new optional paramater for em_proxy:

Name Default Value
em_proxy server_privatekey UI+sCDEketXDq6vOAidLe0mYiHogMh1TA2zg3CqlxEA=
em_proxy client_publickey ow01dGyrgRSrjln9bGb6fx0FwY5XKSVMTaoKQ2GDSkM=
em_proxy server_ip 10.0.7.1
em_proxy client_ip 10.0.7.10

make these value customizable instead of hardcoded in em_prixy
image
image

@KusakabeShi
Copy link
Author

By the way, em_proxy client_ip seems automatic detected so we doesn't need to make it configureable.

image

@arthurmelton
Copy link
Contributor

The only place I can even find the server ip is in a test

std::net::TcpStream::connect(SocketAddrV4::new(Ipv4Addr::new(10, 7, 0, 1), 3000))

@KusakabeShi
Copy link
Author

KusakabeShi commented Jan 1, 2025

The scenario of changeable IP is the IP used by em_proxy( 10.0.7.1 ) may conflict with the real VPN (if I want to use em_proxy with real wireguard vpn).
In most case it won't be an issue but make it changeable to give us a chance to avoid the conflict would be better I think.

@arthurmelton
Copy link
Contributor

I believe the IP used for the user is set by one of the functions. Whenever you start the proxy you can set the IP to be whatever you want. If their is a program you are using that uses em_proxy, you would have to request that program to change the IP used. There fire I believe this issue can be closed.

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 a pull request may close this issue.

3 participants