mikakaraila / node-red-contrib-opcua

A Node-RED node to communicate OPC UA. Uses node-opcua library.
Other
211 stars 196 forks source link

Some bugs after the update #238

Closed halcyon0606 closed 3 years ago

halcyon0606 commented 3 years ago

Hello Mika,

After updating the package to the latest version, I found some bugs while conducting my usual testing, these are things that worked in the previous version and are now buggy.

The First Issue

I am now unable to use the node red OPCUA client to connect to the server. The following are the options selected:

image

image

And the error message is as shown:

image

After looking at the error message, I've noticed that the usual directory for the client's self signed certificate is no longer there.

The Second Issue

The Browser UI example can no longer be used even when security mode and policy are set to none. The browser UI example flow: image The UI: image

Hope that a fix can be added soon! Thank you.

mikakaraila commented 3 years ago

Strange, can you check node-opcua version?

Certificate file should be in pem format, now it seems to use .pem.csr

Check one more time your environment!

halcyon0606 commented 3 years ago

The node-opcua version is given as 6.14.8. Is that the current version that is being used?

image

Also, the node-opcua-client file does not have the certificate folder anymore, is that also an issue?

image

mikakaraila commented 3 years ago

OK that explains why client is not anymore working.

node-opcua should be v2.28.3

Your command shows npm version. Use npm list command:

C:\Users\karaimi.node-red>npm list node-red-project@0.0.1 C:\Users\karaimi.node-red +-- async@3.2.0 +-- chalk@4.1.0 | +-- ansi-styles@4.2.1 | | +-- @types/color-name@1.1.1 | | -- color-convert@2.0.1 | |-- color-name@1.1.4 | -- supports-color@7.1.0 |-- has-flag@4.0.0 +-- circular-json@0.5.9 +-- flatted@3.1.0 +-- node-opcua@2.28.3 | +-- chalk@4.1.0 deduped | +-- node-opcua-address-space@2.28.2

halcyon0606 commented 3 years ago

Ahh I see, sorry about that, my npm list output shows that node opcua version is 2.29.0

| +-- node-opcua@2.29.0 | | +-- chalk@4.1.0 deduped | | +-- node-opcua-address-space@2.29.0 | | | +-- @types/lodash@4.14.167 | | | +-- async@3.2.0 deduped | | | +-- chalk@4.1.0 deduped | | | +-- dequeue@1.0.5

So if that's the case, do I have to regress to v2.28.3?

mikakaraila commented 3 years ago

There are breaking change in new v2.29.0

OPCUAClient & OPCUAServer : the location of the default PKI is now created in .config folder or in the %APPDATA% folder instead of inside the node_module itself.

halcyon0606 commented 3 years ago

Oh, I see, in that case do I wait for you to fix the default PKI path in a newer version or is there a way for me to fix this currently?

mikakaraila commented 3 years ago

Just install older node-opcua v2.28.3 then it should work again. I need couple of days to fix code to use new node-opcua.

mikakaraila commented 3 years ago

I made quick fix, 0.2.93 uses node-opcua v2.28.3

I need more time to do all needed changes.

mikakaraila commented 3 years ago

Now new version 0.2.94 uses node-opcua 2.29.0 let´s see if it helps now.