Socket error on Azure app service about guerrillantp HOT 6 OPEN

The "invalid pointer address" looks like an internal NET problem. I don't use Azure. Could you try writing a small program that just calls Connect() on a UDP socket? If it fails, we know it's not GuerrillaNTP-related.

from guerrillantp.

Hi Robert,

The error has been identified and can be resolved with a few small changes to your code. For some strange reason, Azure always tries to connect via IPV6, I think if you create an overload of the NtpClient method to include the possibility of choosing the range of addresses the problem will be solved, the following code does not generate an error in Azure:

var teste = new DnsEndPoint("ntp02.oal.ul.pt", 123);
var socket = new Socket(AddressFamily.InterNetwork, SocketType.Dgram, ProtocolType.Udp);
socket.Connect(teste);
socket.Close();

If i Remove the AddressFamily Azure throws one exception error.

Just add the parameter to NtpClient and change all socket connections to include the parameter if defined *if not defined use the default value AddressFamily.Unspecified
Ex:

NtpClient client = new NtpClient("ntp02.oal.ul.pt", 123, AddressFamily.InterNetwork);
or
NtpClient client = new NtpClient("ntp02.oal.ul.pt", AddressFamily.InterNetwork);

Hope this helps you to solve the issue

Cheers
Jorge

from guerrillantp.

Good analysis. Would you be able to make a pull request with the API changes you are proposing?

from guerrillantp.

Hi Robert

Created the pull request, if you use it and after you update the Nudget Package I'll try it on production server.

Cheers

Jorge

from guerrillantp.

Hi Robert

Did you have the opportunity to check the pull request I made regarding this error?

Still saying that needs to be approved by a maintainer

Cheers

Jorge

from guerrillantp.

I am a bit busy, but it's on my priority TODO list. I will get to it as soon as possible.

from guerrillantp.