You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@plc4x.apache.org by Álvaro Del Castillo <ad...@ThingsO2.com> on 2019/12/18 05:57:22 UTC

[Request for Review][OPC-UA] Fix wrong NOT FOUND exception in OPC-UA driver

Dear all,

I ask for a reviewer to the OPC-UA driver PR:

https://github.com/apache/plc4x/pull/112

It is related to the issue:

https://issues.apache.org/jira/projects/PLC4X/issues/PLC4X-164

Right now the OPC-UA driver can not read the value of nodes from a
sensor opc-ua server. This PR corrects the issue.

Thank you!

-- Alvaro


AW: [Request for Review][OPC-UA] Fix wrong NOT FOUND exception in OPC-UA driver

Posted by "Strljic, Matthias Milan" <ma...@isw.uni-stuttgart.de>.
Hi Alvaro,

thank you for the contribution of this bugfix.
It was a known issue where I mixed up the state declarations of milo with enumerations like in the PR #75 https://github.com/apache/plc4x/pull/75 mentioned. This will lead into a refactoring of the OPC UA driver over the holidays to increase the CQ 😉

So this PR will be merged into dev and you are welcome with active contribution and testing.

Greetings Matthias

Matthias Strljic, M.Sc.

Universität Stuttgart
Institut für Steuerungstechnik der Werkzeugmaschinen und Fertigungseinrichtungen (ISW)

Seidenstraße 36
70174 Stuttgart
GERMANY

Tel: +49 711 685-84530
Fax: +49 711 685-74530

E-Mail: matthias.strljic@isw.uni-stuttgart.de
Web: http://www.isw.uni-stuttgart.de

-----Ursprüngliche Nachricht-----
Von: Álvaro Del Castillo <ad...@ThingsO2.com> 
Gesendet: Wednesday, December 18, 2019 6:57 AM
An: dev@plc4x.apache.org
Betreff: [Request for Review][OPC-UA] Fix wrong NOT FOUND exception in OPC-UA driver

Dear all,

I ask for a reviewer to the OPC-UA driver PR:

https://github.com/apache/plc4x/pull/112

It is related to the issue:

https://issues.apache.org/jira/projects/PLC4X/issues/PLC4X-164

Right now the OPC-UA driver can not read the value of nodes from a sensor opc-ua server. This PR corrects the issue.

Thank you!

-- Alvaro