View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0008435 | 10000-005: Information Model | Spec | public | 2022-11-11 14:46 | 2024-02-13 16:09 |
Reporter | Adrian Scholl | Assigned To | Randy Armstrong | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | assigned | Resolution | reopened | ||
Product Version | 1.04 | ||||
Target Version | 1.05.03 RC1 | Fixed in Version | 1.05.03 | ||
Summary | 0008435: 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 | ||||
Tags | No tags attached. | ||||
Commit Version | |||||
Fix Due Date | |||||
|
Paul thinks it should remain a String but what is missing is how to encode the bytestring as a string for display. |
|
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. |
|
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 |
|
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. |
|
Changed DataType of CertificateErrorEventId property from String to ByteString in the 1.05.03 draft. |
|
Agreed to Errata edited in Web meeting. |
|
1.04.12 nodeset was not updated to reflect the fix. |
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 |