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.

Sign up for GitHub

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

Jump to bottom

Opcaua server updates #196

Merged
merged 7 commits into from
Oct 24, 2020
Merged

Opcaua server updates #196

merged 7 commits into from
Oct 24, 2020

Conversation

hutcheb
Copy link
Contributor

@hutcheb hutcheb commented Oct 21, 2020

  • Setup user password file for OPC UA Server
  • Add command line switches to handle creating users/passwords and self signed certificates.
  • OPC UA server encryption now works.
  • Bumped Milo to 0.52 for client and server
  • getNumberOfElements method to PlcField interface.
  • OPC UA server, arrays are handled now.
  • df1and code-gen in the sandbox seemed to have an issue with intermittent failures when building. I think it has to do with an external dependency for slf4j.
  • When parsing strings in PlcValue classes, leading spaces would cause issues.

I'll leave it open for comment for a few days before merging.

@hutcheb
Created password config file
11d7c4e 
This will store all the hased passwords for each user instead of in the 
main config file.
@hutcheb
Hashed passwords and additional command line switches.
6ee584a
@hutcheb
Updated documentation
63db2d0
@hutcheb
Fixed issue with password salt being recreated on every startup
fcb2063
@hutcheb
Bumped Milo 0.52, Add getNumberOfElements, OPC UA Array handling
e738424
@hutcheb
OPC UA Specific Changes
79d0047 
PlcValue add trim when parsing strings. Some logger pom changes for 
sandbox projects, hopefully it will fix the intermitant build failures. 
OPC UA server array handling.
@ottlukas ottlukas requested a review from nalim2 October 21, 2020 05:46
nalim2
nalim2 reviewed Oct 23, 2020
View reviewed changes
Copy link
Contributor

@nalim2 nalim2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I just inspected the code and will test it out in the afternoon.

@hutcheb hutcheb merged commit 6918c72 into apache:develop Oct 24, 2020
@hutcheb hutcheb deleted the opcaua-server branch November 12, 2020 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nalim2 nalim2 nalim2 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@hutcheb @nalim2