Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an OPC UA Server #194

Merged
merged 8 commits into from
Oct 16, 2020
Merged

Add an OPC UA Server #194

merged 8 commits into from
Oct 16, 2020

Conversation

hutcheb
Copy link
Contributor

@hutcheb hutcheb commented Oct 10, 2020

No description provided.

@hutcheb hutcheb marked this pull request as draft October 10, 2020 19:03
Otherwise it will create a security dir in the working dir.
Added a with-opcua-server option.
Used the projects dependancy versions.
Split out the Plc4x comms logic into it's one files so they aren't mixed 
up in the OPC UA server logic.
Added an outline for the OPC UA Server README file
We should be able to use a base data type in opc ua as we do all the 
conversions already in Plcx.
Handling of Arrays could be better by creating a node for each item in 
the array.
@hutcheb hutcheb marked this pull request as ready for review October 12, 2020 11:59
@hutcheb
Copy link
Contributor Author

hutcheb commented Oct 12, 2020

It is definitely usable but somethings to improve on:-

  • Security isn't implemented.
  • Arrays should probably be a list of nodes in OPCUA instead of a single one.
  • Discovery doesn't seem to work.

Passwords are entered on startup if they don't exist and then saved to 
the config file. They still need to be encrypted.
Also went ot he latest stable build 5.1 instead of 5.2-snapshot for the 
opcua server.
@hutcheb hutcheb merged commit b6aa1d0 into apache:develop Oct 16, 2020
@hutcheb hutcheb deleted the Feature/opcua-server branch November 12, 2020 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant