apache / plc4x

PLC4X The Industrial IoT adapter
https://plc4x.apache.org/
Apache License 2.0
1.25k stars 400 forks source link

[Bug]: plc4j - EtherNet/IP - Cannot establish connection with PLC. #845

Closed crdomatica closed 4 months ago

crdomatica commented 1 year ago

What happened?

First of all, thank you for this project. It has been very useful for us.

Using the latest version of plc4j (v0.10.0), we are not being able to establish a connection to an EtherNet/IP PLC, but using the previous version of plc4j (v0.9.1) all went well with the connection to the same EtherNet/IP PLC.

Below we share the log from PLC4j.

log-plc4j-v0.10.0.log

Thank you.

Version

v0.10.0

Programming Languages

Protocols

chrisdutz commented 1 year ago

Hi,

I think this will probably be related to some major changes we merged in recently, that were supposed to increase the functionality of the EIP driver.

Seems we might be needing a little more work on that.

Would you be so kind and tell us a little bit more about the type of device you're trying to talk to? And a wireshark recording would be perfect.

We might need one or two extra round, but it's definitely not supposed to break existing implementations.

chrisdutz commented 1 year ago

And please excuse me ... you said in 0.9.0 it worked and in 0.10.0 it didn't ... well the changes I referred to were actually in 0.11.0-SNAPSHOT so they couldn't have any impact on 0.10.0 ... but still ... please provide the information I requested so we can have a look.

crdomatica commented 1 year ago

Hi,

Thank you for your response. The PLC we are trying to connect is a ControlLogix from Allen Bradley. I Am not sure on the exact model. Attached I share the captures using the two versions of PLC4j on the same device.

Wireshark Captures.zip

Thanks.

chrisdutz commented 1 year ago

Oh ... that's a fortunate coincidence ... It happens that we just recently added a PR from @hutcheb that promisses to add direct support for Allen Bradley EIP devices. While I'm having a look at the captures, could you please give the 0.11.0-SNAPSHOT version a go?

chrisdutz commented 1 year ago

Today I got my CompactLogix working with a real program and ran into issues with PLC4J EIP driver ... however I adjusted our mspec for EIP and with these changes I was successfully able to read and write to my controller.

Could you please check again with the latest version (SNAPSHOT)?

chrisdutz commented 1 year ago

Could you please confirm if this issue is solved or not?

chrisdutz commented 4 months ago

Closing this issue as we think it's resolved but the reporter simply failed to confirm