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 suggestion: IPv6 support #4

Open
sbrl opened this issue Feb 14, 2021 · 4 comments
Open

Feature suggestion: IPv6 support #4

sbrl opened this issue Feb 14, 2021 · 4 comments

Comments

@sbrl
Copy link

sbrl commented Feb 14, 2021

I just tried to use simpleproxy with an IPv6 remote, but it didn't appear to like it very much. A quick glance at the source code suggests that IPv6 is not supported yet (e.g. parse_host_port doesn't check for IPv6 addresses I guess).

IPv6 is the future, so it'd be great if it was supported here.

@vlad1
Copy link
Collaborator

vlad1 commented Feb 15, 2021

This is correct, simpleproxy doesn't support IPv6 at the moment, unfortunately. I'll see if this can be implemented.

@cross
Copy link

cross commented Apr 27, 2022

I find I have a need for this myself. I am using simple proxy 3.4 on FreeBSD 12. If this is still the primary location for this project, I can assist by adding that support. (I'm 90% sure)

@vzaliva
Copy link
Owner

vzaliva commented May 3, 2022

Sorry for the delayed response. Yes, this is the primary repository. Contributions are always welcome. Please submit at PR and we will review it. Please keep code portable, so it will compile on Linux or other Unix-es as well. Thanks!

@cross
Copy link

cross commented Jun 12, 2022

Okay. I'm working on this, for anyone's information. I thought it would take a few hours, but I had to dust off this knowledge in my head, and then found some peculiarities. I think there may be some FreeBSD issues that I'll have to consider separately on other Ones.

A question for you @vzaliva , what are your feelings on whether a "disable IPv6" should be a compile-time option, and do you require a -4 and/or -6 command-line option to prohibit address families? (The more of this I keep/add, the more complicated the code looks...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants