Modbus TCP does not support hostname resolution

Azure / iot-edge-modbus

Modbus protocol module for use with the Azure IoT Edge

Other

90 stars 71 forks source link

Modbus TCP does not support hostname resolution #65

Closed 4nandP closed 3 years ago

4nandP commented 5 years ago

It would be convenient if we could use syntax such as tcp:mycontainer instead of 10.0.71.1 for the SlaveConnection setting and have the iot-edge-modbus resolve the container to IP.

PR to follow.

simonachmueller commented 2 years ago

I can see PR is not accepted, and I don't see it's working, it still try to parse the SlaveConnection as IP Address (see ConnectSlaveAsync() in ModbusTCPSlaveSession.cs I would say it needs to be reopened.