View Issue Details

IDProjectCategoryView StatusLast Update
000843510000-005: Information ModelSpecpublic2024-02-13 16:09
ReporterAdrian Scholl Assigned ToRandy Armstrong  
PrioritynormalSeverityminorReproducibilityalways
Status assignedResolutionreopened 
Product Version1.04 
Target Version1.05.03 RC1Fixed in Version1.05.03 
Summary0008435: Datatype for CertificateErrorEventId within AuditOpenSecureChannelEventType should be ByteString instead of String
Description

Datatype for CertificateErrorEventId within AuditOpenSecureChannelEventType should be ByteString instead of String

The errata 1.04.10 adds the field CertificateErrorEventId to the event type AuditOpenSecureChannelEventType. According to the description, this field should contain an EventId of a specific event. EventIds are ByteStrings according to the UA specification. The same type should be used here instead of a String which represents just text.

The field was added with https://mantis.opcfoundation.org/view.php?id=6470

TagsNo tags attached.
Commit Version
Fix Due Date

Activities

Jim Luth

2022-11-15 17:21

administrator   ~0018151

Paul thinks it should remain a String but what is missing is how to encode the bytestring as a string for display.

Adrian Scholl

2022-11-16 07:25

reporter   ~0018157

But why should the server convert the value to be displayed. It's up to the client to display the value or, even better, display the corresponding event. If the client wants to match the EventIds the string needs to be converted back into a bytestring. In this case, there are two useless conversions.

Jim Luth

2022-12-21 14:31

administrator   ~0018332

More arguments from Adrian for changing this to a ByteString:

• Because of the missing conversion rule no one could have implemented this according to the specification
• A major advantage of OPC UA vs. classic OPC on the server side is the removed requirement for type conversions, which would be introduced by this again
• Why does the server need to bother with the conversion from ByteString to String, when its only used for display purposes

Jim Luth

2023-01-10 16:50

administrator   ~0018451

Agreed it needs to be changed to a ByteString. The assumption is this breaking change will not affect interop of shipping product. Needs new 1.04 Errata to fix the 1.04.10 Errata entry.

Jeff Harding

2023-01-11 19:34

developer   ~0018455

Changed DataType of CertificateErrorEventId property from String to ByteString in the 1.05.03 draft.
Added entry to 1.04.12 Errata.

Jim Luth

2023-04-11 16:48

administrator   ~0019163

Agreed to Errata edited in Web meeting.

Jim Luth

2024-02-13 16:08

administrator   ~0020800

1.04.12 nodeset was not updated to reflect the fix.

Issue History

Date Modified Username Field Change
2022-11-11 14:46 Adrian Scholl New Issue
2022-11-15 17:21 Jim Luth Note Added: 0018151
2022-11-15 17:22 Jim Luth Assigned To => Jeff Harding
2022-11-15 17:22 Jim Luth Status new => assigned
2022-11-15 17:22 Jim Luth Target Version => 1.05.03 RC1
2022-11-16 07:25 Adrian Scholl Note Added: 0018157
2022-12-21 14:31 Jim Luth Note Added: 0018332
2023-01-10 16:50 Jim Luth Note Added: 0018451
2023-01-11 19:34 Jeff Harding Status assigned => resolved
2023-01-11 19:34 Jeff Harding Resolution open => fixed
2023-01-11 19:34 Jeff Harding Fixed in Version => 1.05.03
2023-01-11 19:34 Jeff Harding Note Added: 0018455
2023-04-11 16:48 Jim Luth Status resolved => closed
2023-04-11 16:48 Jim Luth Note Added: 0019163
2024-02-13 16:08 Jim Luth Assigned To Jeff Harding => Randy Armstrong
2024-02-13 16:08 Jim Luth Status closed => feedback
2024-02-13 16:08 Jim Luth Resolution fixed => reopened
2024-02-13 16:08 Jim Luth Note Added: 0020800
2024-02-13 16:09 Jim Luth Status feedback => assigned