frangoteam / FUXA

Web-based Process Visualization (SCADA/HMI/Dashboard) software
https://frangoteam.org
MIT License
2.59k stars 769 forks source link

[BUG] Missing library SiemensS7 #1268

Open martinjug opened 1 month ago

martinjug commented 1 month ago

Describe the bug Missing library SiemensS7.

Screenshots image

Environment

mauroalexandre commented 1 month ago

I opened a discussion for voting on this topic a few days ago. Please consider participating.

https://github.com/frangoteam/FUXA/discussions/1265

martinjug commented 1 month ago

It was included by default in Docker. Library is included in version frangoteam/fuxa:1.1.19 and works without issue. I think something got broken in latest version.

MatthewReed303 commented 1 month ago

@martinjug it's not in the packages for the latest https://github.com/frangoteam/FUXA/blob/master/server/package.json maybe @unocelli will add it so it can be installed separately like nodepccc I personally use OPC-UA with Siemens anyway, have for both s7-1200 and s7-1500

martinjug commented 1 month ago

@MatthewReed303 The library worked perfectly, so I don't understand why it was excluded. I'm currently using it to connect to Siemens LOGO, and there are still many machines/applications using S7 PLCs that don't have built-in OPCUA Server.

MatthewReed303 commented 1 month ago

@martinjug you could just fork Fuxa and add it back to the package list and do a build of your fork, remember to change the git clone in the docker file to your fork. If it works with no issues then create a pull request to add it back in or any fixes etc

unocelli commented 1 month ago

Hi, the package have compatibility issue with the nodejs version 16/18, I propose to create a docker version (tagged s7) with node-snap7

martinjug commented 1 month ago

@unocelli that would be great if there is S7 docker version based on node version 14.

unocelli commented 3 weeks ago

Hi, you will find the the docker version with installed node-snap7 library in frangoteam/fuxa:snap7

martinjug commented 2 weeks ago

Thanks @unocelli. I have tested it and it works.