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

IPv6 SocketUDT #58

Open
aurelienmaury opened this issue Oct 2, 2013 · 6 comments
Open

IPv6 SocketUDT #58

aurelienmaury opened this issue Oct 2, 2013 · 6 comments

Comments

@aurelienmaury
Copy link

The com.barchart.udt.SocketUDT class states clearly:

note: current implementation supports IPv4 only (no IPv6)

Is there any hope to see a IPv6 ready SocketUDT in a future release ?

Btw, thanks for this great piece of work, I play with the Netty integration and take a lot of fun.

@Andrei-Pozolotin
Copy link
Contributor

thanks for checking in.
adding ipv6 support is a straightforward change, but it touches on many places.
there were no need to support ipv6 internally here, and there were no one else interested.
you are the first one :-) I could definitely help, but someone like you would have to take the lead.
you would want to fork a branch and start breaking/fixing relevant unit tests affected by new ipv6 support.
cheers!

@aurelienmaury
Copy link
Author

Nice hook ;-)

I add this to my TODO list. If you have hints to give me, about where to start and pitfalls, it'd be nice.

@Andrei-Pozolotin
Copy link
Contributor

:-)
to give you any hints I have to make context switch myself. :-)
lets make a deal: when you feel like really doing it I will be there also.

@Holdo
Copy link

Holdo commented Sep 9, 2016

So... what's up? :)
I guess I'll pick this up then because... wait for it... I preferIPv6Addresses

@dlucek
Copy link

dlucek commented Sep 12, 2016

Michael:
IP v 6 support would be great.

On Fri, Sep 9, 2016 at 5:30 AM, Michal Holič notifications@github.com
wrote:

So... what's up? :)
I guess I'll pick this up then because... wait for it... I
preferIPv6Addresses


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#58 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAd-va2PbtjbXyH5YzYM93CK3xA84ZHfks5qoTVZgaJpZM4BDlya
.

dl

Holdo added a commit to Holdo/barchart-udt that referenced this issue Sep 13, 2016
@Holdo
Copy link

Holdo commented Sep 13, 2016

Should be done #86

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

4 participants