get_connected returns true if the connection is lost

[EnricGarrigo]

I tested with Python 3.6 and Windows 7 64 bit.
I have a 'While True' loop for test the connection continuously. If when the program starts the connection is OK, 'get_connected' returns 'true' and the loop runs. If the connection is broken during execution, 'get_connected' still returns 'true' and db_write crashes.

Does anyone know who avoid this behaviour?

Thanks.

[Steven1408]

I can confirm, had to use If str(plc.get_cpu_state()) == "S7CpuStatusRun":
Works like charm but feels ghetto.

get_Connected() returns False when you run plc.disconnect() but not when disconnected from the Network, still writes True.

Python 2.7, Debian, snap7-1.2.1 and the wrapper.

[gijzelaerr]

since python-snap7 is a thin wrapper around snap7, this is actually a bug in the snap7 lib. So the long-term solution is to report it upstream. The only problem is that last time I checked the project is not really maintained anymore, so that fix might never happen.

I could add @N3mesisXS's suggestion to python-snap7, but I'm not sure if that works correctly for all other users. So @EnricGarrigo, for now, I suggest adding this check to your code and see if that solves the problem.

[spreeker]

davide is doing his best to maintain it right? I think he will fix it if you report it :)

…

On Thu, Aug 2, 2018 at 10:26 AM, Gijs Molenaar ***@***.***> wrote: since python-snap7 is a thin wrapper around snap7, this is actually a bug in the snap7 lib. So the long-term solution is to report it upstream. The only problem is that last time I checked the project is not really maintained anymore, so that fix might never happen. I could add @N3mesisXS <https://github.com/N3mesisXS>'s suggestion to python-snap7, but I'm not sure if that works correctly for all other users. So @EnricGarrigo <https://github.com/EnricGarrigo>, for now, I suggest adding this check to your code and see if that solves the problem. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#111 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADpMbh3HKdMu6I5wUGoMD5v9-WjAeZgks5uMrfBgaJpZM4VMT8l> .

[gijzelaerr]

there have been no commits to the repo for more than a year...

@EnricGarrigo if you could report the issue upstream and link back here that would be useful.

[spreeker]

Luckily PLC's are made to run for 25 years or so. :). Interfaces do not change much. They have improved encryption on the newer models which are slowly very slowly being deployed..

…

On Thu, Aug 2, 2018, 12:11 Gijs Molenaar ***@***.***> wrote: there have been no commits to the repo for more than a year... @EnricGarrigo <https://github.com/EnricGarrigo> if you could report the issue upstream and link back here that would be useful. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#111 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADpMZ8V44BpwT-1wLyCyTaAvQmXK4C_ks5uMtBFgaJpZM4VMT8l> .

[DasLukas]

Hey everyone. Something new here? I need to check if the Logo reachable.

I wonder where to place @N3mesisXS workaround?

[MountainKing91]

My workaround for this situation is to:

1. in the read loop I check the connection status before reading, if not connected I try to connect
2. catch the read/write exception and disconnect the client. At that point get_connected will return false, so when checking the connection in the read loop (1) i will know I have to reconnect
   This is also how Davide does in some examples (not in python).