-
Notifications
You must be signed in to change notification settings - Fork 22
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 support #25
Comments
I like it! I haven't spent much time looking at IPv6, my fault. If it's that simple then it needs to be rolled out into the whole suite of MMDVM programs for future proofing. You're code looks good. |
Changed some lines for Windows/VisualStudio 2019. No problem for compiling Current code does not have IPv4<-->IPv6 fallback. Do we have to implement If there is no problem, I want to issue pull-request. But is there any |
I have added an "IPv6" branch for testing purposes. |
Now I issuing pull request to IPv6 branch. |
@jg1uaa I did compile the IPv6 branch successfully but get a TCP error 97 when trying to use an IPv6 address in DAPNETGateway.ini. Is there any format we need to have in the .ini? Using a domain name yields the same result ... :( |
@phl0 this is my DAPNETGateway.ini
dapnet.afu.rwth-aachen.de has IPv4 address only. |
Jepp, I have an own instance of DAPNET running on my own server. That is IPv6 capable and the core process listens on IPv6 socket. I did try with that host. |
@phl0 oh, it is bug of TCPSocket.cpp. It always creates IPv4 socket... |
@jg1uaa roger that. Just tested successfully. Go ahead an PR that. My DAPNETGateway is noch connected to my test instance using IPv6: |
Hello,
Current code is strongly dependent on sockaddr_in and limited to IPv4 support.
I am trying to rewrite with sockaddr_storage to add IPv6 support.
There is a modified code at https://github.com/jg1uaa/DAPNETGateway and tested with MMDVM-POCSAG emulator http://www.uaa.org.uk/gomitext/2019/20190923/fakemmdvmhost-pocsag.c
At least they looks working on Linux/OpenBSD (Windows and MacOS not tested), I hope they will be a hint to future expansion of DAPNETGateway/MMDVMHost.
The text was updated successfully, but these errors were encountered: