AttributeError with 1.2.0 release on Windows

[romana]

AttributeError with 1.2.0 release on Windows

[jbrendel]

Thank you for reporting this issue. I'll have a look at it.

[jbrendel]

Recently, IPv6 support was added to multi-ping. The issue seems to stem from that. The code works for me as is and the IPPROTO_ICMPV6 symbol is defined for me.

@cleonello and @geokal, since I can't reproduce it here, could you please help me out and run a small test for me? Could you please start a Python shell and import the socket module, then check one attribute value for me?

>>> import socket
>>> socket.has_ipv6

What output do you get?

@geokal, could you please also let me know what version of Ubuntu you are running?

[mametov]

This solves the issue:
Open ...\lib\site-packages\multiping_init_.py
Line 129 where IF starts and delete "if ..... " to the end of the line and the line 130.

[geokal]

Hi, I can confirm the same happens in Ubuntu Linux

[geokal]

Hi, @jbrendel in my Windows terminal the output of socket.has_ipv6 is equal to true. But again the result after running the python script located in the readme section:
is the same as @cleonello. I will report back on Ubuntu cause I don't have access right now. Thanks again!!

[alexjacobs08]

Having the same issue on Windows 10 using python 3.5.2

>>> import socket
>>> socket.has_ipv6

returns True

[mametov]

Hi, I can confirm the same happens on Win 10 Python 3.6.5
socket.has_ipv6 returns True

[geostar1024]

@mametov's solution doesn't work for me under python 3.4.3 on Windows XP (nor does replacing socket.IPPROTO_ICMPV6 with the hardcoded value 58). What does work is to comment out the self._sock6... statements on lines 119 and 120 (effectively removing IPv6 from multiping).

Technically, this shouldn't be a problem, given that Microsoft says that IPPROTO_ICMPV6 should be available on Windows XP. Maybe this has something to do with this bug.

[zachbryant]

Can confirm @mametov's workaround on win10 3.6.5, but still hoping for a real solution

[jbrendel]

Hello everyone. Thank you for continuing to follow up on this. My apologies for not checking back earlier.

I have just pushed what I believe is a patch for this. In effect, it checks whether the IPPROTO_ICMPV6 attribute is present in socket or not. And if not then it uses the hardcoded value of 58.

I tested this on Ubuntu 14.04, 16.04 and on Windows 10 and it works (no surprise on the Ubuntus, since they have that value available anyway).

I'm concerned about @geostar1024 report, though, since apparently you tried hardocding 58 and it did not work for you? I really don't know what to say to that. This, as patched right now, seems to work. At least with Python3 on Windows 10.

If everyone could give the current version a go, please, and let me know? I don't want to just close this issue without getting your feedback first.

Thank you again!

[jbrendel]

If this works for everyone, I will mark this issue as closed...

[geokal]

@jbrendel thanks a lot will check tomorrow!