mycroes / Sally7

C# implementation of Siemens S7 connections with a focus on performance
MIT License
56 stars 22 forks source link

Verify requests are completed on dispose #29

Closed mycroes closed 1 year ago

mycroes commented 1 year ago

The ConcurrentRequestExecutor permits multiple active requests. When the connection is closed and multiple requests are active they might not be completed and get stuck indefinitely.

mycroes commented 1 year ago

Should be working, net461 does require the socket to be closed due to missing cancellation on send/receive calls, other than that there should be no issues.