Re: [PR] plc4j-driver-opcua: Fix incorrectly handled GUID tags

2023-09-24 Thread via GitHub


sruehl merged PR #1099:
URL: https://github.com/apache/plc4x/pull/1099


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] plc4j-driver-opcua: Fix incorrectly handled GUID tags

2023-09-24 Thread via GitHub


sruehl commented on PR #1099:
URL: https://github.com/apache/plc4x/pull/1099#issuecomment-1732969497

   Thanks for the PR @takraj  


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] build(deps): bump slf4j.version from 2.0.7 to 2.0.9

2023-09-24 Thread via GitHub


sruehl merged PR #1092:
URL: https://github.com/apache/plc4x/pull/1092


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] build(deps): bump org.checkerframework:checker-qual from 3.37.0 to 3.38.0

2023-09-24 Thread via GitHub


sruehl merged PR #1091:
URL: https://github.com/apache/plc4x/pull/1091


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [VOTE] Apache PLC4X Build-Tools Code-Generation 1.7.0 RC1

2023-09-24 Thread Cesar Garcia
+1 (binding)

Built on Ubuntu 22.04 jammy x86_64 Linux 6.2.0-33-generic Intel Core
i7-9700K @ 8x 4.9GHz
OpenJDK Runtime Environment (build 11.0.20.1+1-post-Ubuntu-0ubuntu122.04)

[OK] Download all staged artifacts under the url specified in the release
vote email.
[OK] Verify the signature is correct.
[OK] Check if the signature references an Apache email address.
[OK] Verify the SHA512 hashes.
[OK] Unzip the archive.
[OK] Verify the existence of LICENSE, NOTICE, README, RELEASE_NOTES files
in the extracted source bundle.
[OK] Verify the content of LICENSE, NOTICE, README, RELEASE_NOTES files in
the extracted source bundle.
[OK] [RM] Verify the staged source README, RELEASE_NOTE files correspond to
those in the extracted source bundle.
[OK] [RM] Run RAT externally to ensure there are no surprises.
[OK] Search for SNAPSHOT references
[OK] Search for Copyright references, and if they are in headers, make sure
these files containing them are mentioned in the LICENSE file.
[OK] Build the project according to the information in the README.md file.
[OK] [RM] Build the project with all with-xyz profiles and tests enabled
and an empty maven local repo.







El vie, 22 sept 2023 a las 7:20, Christofer Dutz ()
escribió:

> Apache PLC4X Build-Tools Code-Generation 1.7.0 has been staged under [2]
>and it’s time to vote on accepting it for release.
>
>All Maven artifacts are available under [1]. Voting will be open for
> 72hr.
>
>A minimum of 3 binding +1 votes and more binding +1 than binding -1
>are required to pass.
>
>Repository: https://gitbox.apache.org/repos/asf/plc4x-build-tools.git
>Release tag: releases/code-generation/1.7.0
>Hash for the release tag: 43b93a139c9580325c26082e7272fc224238355f
>
>Per [3] "Before voting +1 PMC members are required to download
>the signed source code package, compile it as provided, and test
>the resulting executable on their own platform, along with also
>verifying that the package meets the requirements of the ASF policy
>on releases."
>
>You can achieve the above by following [4].
>
>[ ]  +1 accept (indicate what you validated - e.g. performed the non-RM
> items in [4])
>[ ]  -1 reject (explanation required)
>
>
>[1]
> https://repository.apache.org/content/repositories/orgapacheplc4x-1049
>[2]
> https://dist.apache.org/repos/dist/dev/plc4x/build-tools/code-generation/1.7.0/rc1/
>[3]
> https://www.apache.org/dev/release/validation.html#approving-a-release
>[4] https://plc4x.apache.org/developers/release/validation.html
>
>

-- 
*CEOS Automatización, C.A.*
*GALPON SERVICIO INDUSTRIALES Y NAVALES FA, C.A.,*
*PISO 1, OFICINA 2, AV. RAUL LEONI, SECTOR GUAMACHITO,*

*FRENTE A LA ASOCIACION DE GANADEROS,BARCELONA,EDO. ANZOATEGUI*
*Ing. César García*

*Cel: +58 414-760.98.95*

*Hotline Técnica SIEMENS: 0800 1005080*

*Email: support.aan.automat...@siemens.com
*


[PR] plc4j-driver-opcua: Adapt error handling of reads, to writes and subscriptions

2023-09-24 Thread via GitHub


takraj opened a new pull request, #1108:
URL: https://github.com/apache/plc4x/pull/1108

   This fixes `ClassCastException` coming in case of errors.
   
   Also covers an earlier draft PR: https://github.com/apache/plc4x/pull/317


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



AW: [VOTE] Apache PLC4X Build-Tools Code-Generation 1.7.0 RC1

2023-09-24 Thread Christofer Dutz
Bumping this thread to the surface after all this GitHub activity … still need 
one +1 vote …

Chris


Von: Lukas Ott 
Datum: Freitag, 22. September 2023 um 14:30
An: dev@plc4x.apache.org 
Betreff: Re: [VOTE] Apache PLC4X Build-Tools Code-Generation 1.7.0 RC1
+1 (binding)

Built on Linuxmint - LMDE 6 using openjdk 17.0.8 2023-07-18
OpenJDK Runtime Environment (build 17.0.8+7-Debian-1deb12u1)
OpenJDK 64-Bit Server VM (build 17.0.8+7-Debian-1deb12u1, mixed mode,
sharing)


Lukas

[OK] Download all staged artifacts under the url specified in the release
vote email.
[OK] Verify the signature is correct.
[OK] Check if the signature references an Apache email address.
[OK] Verify the SHA512 hashes.
[OK] Unzip the archive.
[OK] Verify the existence of LICENSE, NOTICE, README, RELEASE_NOTES files
in the extracted source bundle.
[OK] Verify the content of LICENSE, NOTICE, README, RELEASE_NOTES files in
the extracted source bundle.
[OK] [RM] Verify the staged source README, RELEASE_NOTE files correspond to
those in the extracted source bundle.
[OK] [RM] Run RAT externally to ensure there are no surprises.
[OK] Search for SNAPSHOT references
[OK] Search for Copyright references, and if they are in headers, make sure
these files containing them are mentioned in the LICENSE file.
[OK] Build the project according to the information in the README.md file.

Am Fr., 22. Sept. 2023 um 13:32 Uhr schrieb Christofer Dutz <
christofer.d...@c-ware.de>:

> +1 (binding)
>
> Built on Mac OS 13.5.2 with Apple M2 Max using Java 11.0.20 (Amazon
> Coretto Build)
>
> Chris
>
> [OK] Download all staged artifacts under the url specified in the release
> vote email.
> [OK] Verify the signature is correct.
> [OK] Check if the signature references an Apache email address.
> [OK] Verify the SHA512 hashes.
> [OK] Unzip the archive.
> [OK] Verify the existence of LICENSE, NOTICE, README, RELEASE_NOTES files
> in the extracted source bundle.
> [OK] Verify the content of LICENSE, NOTICE, README, RELEASE_NOTES files in
> the extracted source bundle.
> [OK] [RM] Verify the staged source README, RELEASE_NOTE files correspond
> to those in the extracted source bundle.
> [OK] [RM] Run RAT externally to ensure there are no surprises.
> [OK] Search for SNAPSHOT references
> [OK] Search for Copyright references, and if they are in headers, make
> sure these files containing them are mentioned in the LICENSE file.
> [OK] Build the project according to the information in the README.md file.
> [OK] [RM] Build the project with all with-xyz profiles and tests enabled
> and an empty maven local repo.
>
>
> Von: Christofer Dutz 
> Datum: Freitag, 22. September 2023 um 13:20
> An: dev@plc4x.apache.org 
> Betreff: [VOTE] Apache PLC4X Build-Tools Code-Generation 1.7.0 RC1
> Apache PLC4X Build-Tools Code-Generation 1.7.0 has been staged under [2]
>and it’s time to vote on accepting it for release.
>
>All Maven artifacts are available under [1]. Voting will be open for
> 72hr.
>
>A minimum of 3 binding +1 votes and more binding +1 than binding -1
>are required to pass.
>
>Repository: https://gitbox.apache.org/repos/asf/plc4x-build-tools.git
>Release tag: releases/code-generation/1.7.0
>Hash for the release tag: 43b93a139c9580325c26082e7272fc224238355f
>
>Per [3] "Before voting +1 PMC members are required to download
>the signed source code package, compile it as provided, and test
>the resulting executable on their own platform, along with also
>verifying that the package meets the requirements of the ASF policy
>on releases."
>
>You can achieve the above by following [4].
>
>[ ]  +1 accept (indicate what you validated - e.g. performed the non-RM
> items in [4])
>[ ]  -1 reject (explanation required)
>
>
>[1]
> https://repository.apache.org/content/repositories/orgapacheplc4x-1049
>[2]
> https://dist.apache.org/repos/dist/dev/plc4x/build-tools/code-generation/1.7.0/rc1/
>[3]
> https://www.apache.org/dev/release/validation.html#approving-a-release
>[4] https://plc4x.apache.org/developers/release/validation.html
>


[PR] plc4j-driver-opcua: Add support for PlcUsernamePasswordAuthentication

2023-09-24 Thread via GitHub


takraj opened a new pull request, #1107:
URL: https://github.com/apache/plc4x/pull/1107

   Attempts to resolve issue #1104


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[I] [Feature Request]: plc4j-driver-opcua - Implement browsing

2023-09-24 Thread via GitHub


takraj opened a new issue, #1106:
URL: https://github.com/apache/plc4x/issues/1106

   ### What would you like to happen?
   
   The OPC-UA protocol supports browsing, but the driver does not. Please 
implement it, in order to be able to recursively fetch data with the library.
   
   Example code to browse the value tree:
   ```
   DefaultPlcDriverManager driverManager = new DefaultPlcDriverManager();
   try (PlcConnection opcuaConnection = 
driverManager.getConnection("opcua:tcp://opcuaserver.com:48010")) {
   PlcBrowseRequest request = opcuaConnection.browseRequestBuilder()
   .addQuery("Folder","ns=2;s=Demo.Static")
   .build();
   
   PlcBrowseResponse response = request.execute().get();
   System.out.println(response.getValues("Folder"));
   }
   ```
   
   The above program throws the exception below:
   ```
   org.apache.plc4x.java.api.exceptions.PlcUnsupportedOperationException: The 
connection does not support browsing
at 
org.apache.plc4x.java.spi.connection.AbstractPlcConnection.browseRequestBuilder(AbstractPlcConnection.java:159)
at 
org.apache.plc4x.java.opcua.ManualOpcuaFolderNode.main(ManualOpcuaFolderNode.java:32)
   ```
   
   Example output of another software, that supports browsing:
   ```
   $ uabrowse -u "opc.tcp://opcuaserver.com:48010" -n "ns=2;s=Demo.Static"
   WARNING:asyncua.client.client:Requested session timeout to be 360ms, got 
60ms instead
   Browsing node ns=2;s=Demo.Static at opc.tcp://opcuaserver.com:48010
   
   DisplayNameNodeIdBrowseName  
  Value
   
   LocalizedText(Locale='', Text='Arrays') ns=2;s=Demo.Static.Arrays 2:Arrays   
  
   LocalizedText(Locale='', Text='Scalar') ns=2;s=Demo.Static.Scalar 2:Scalar   
  
   LocalizedText(Locale='', Text='VariableWithNumericNodeId') ns=2;i=4294967294 
2:VariableWithNumericNodeId, 12 
   LocalizedText(Locale='', Text='VariableWithGuidNodeId') 
ns=2;g=5ce9dbce-5d79-434c-9ac3-1cfba9a6e92c 2:VariableWithGuidNodeId , 0  
   LocalizedText(Locale='', Text='VariableWithByteStringNodeId') 
ns=2;b=0x61626364 2:VariableWithByteStringNodeId, 0  
   LocalizedText(Locale='', Text='Matrix') ns=2;s=Demo.Static.Matrix 2:Matrix
   ```
   
   [Wireshark capture of the above browsing 
request](https://github.com/apache/plc4x/files/12708323/opc-ua-browse-request.pcapng.gz)
   
   
   ### Programming Languages
   
   - [X] plc4j
   - [ ] plc4go
   - [ ] plc4c
   - [ ] plc4net
   
   ### Protocols
   
   - [ ] AB-Ethernet
   - [ ] ADS /AMS
   - [ ] BACnet/IP
   - [ ] CANopen
   - [ ] DeltaV
   - [ ] DF1
   - [ ] EtherNet/IP
   - [ ] Firmata
   - [ ] KNXnet/IP
   - [ ] Modbus
   - [X] OPC-UA
   - [ ] S7


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@plc4x.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org