Pinger.New() can only create privileged icmp.PacketConn

[realnedsanders]

Pinger.New() runs connectICMP() which runs icmp.ListenPacket(network, address) using static value "ip4:icmp" as network. Per icmp.ListenPacket() def, this only allows usage of privileged endpoints.

ListenPacket listens for incoming ICMP packets addressed to address. See net.Dial for the syntax of address.

For non-privileged datagram-oriented ICMP endpoints, network must be "udp4" or "udp6". The endpoint allows to read, write a few limited ICMP messages such as echo request and echo reply. Currently only Darwin and Linux support this.

[kfn-d0]

Hi! I have been looking into this and i believe i have way to implement it

My idea is to introduce a NewUDP() constructor (and a more generic NewPinger) that allows users to specify udp4 or udp6 networks. I have also figured out how to handle the ICMP ID correlation for unprivileged sockets by using the local port and I have already tested this logic successfully in a Docker environment

TestPinger_Privileged
pinger_test.go:16: Skipping privileged test.
SKIP: TestPinger_Privileged (0.00s)

TestPinger_Unprivileged
Pinging (unprivileged) 8.8.8.8...
PASS: TestPinger_Unprivileged (0.04s)

I wouldd love to know if NewUDP() works out of the box on macOS without sudo