View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0007883 | 10000-006: Mappings | Spec | public | 2022-03-23 12:21 | 2023-01-24 17:06 |
Reporter | Kevin Herron (Inductive Automation) | Assigned To | Randy Armstrong | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.04 | ||||
Target Version | 1.05.03 RC1 | Fixed in Version | 1.05.03 RC1 | ||
Summary | 0007883: OPC UA JSON encoding has no recommendation on treatment of null values for fields of built-in types | ||||
Description | In Part 6 while describing the OPC UA JSON encoding the spec is clear that structure fields with a null value are omitted from the JSON object, but for all of the built-in types, which are defined in the spec before the encoding for structures, no specific recommendation is given and thus it's not clear what to do. That is, given e.g. a DiagnosticInfo: var diagnosticInfo = new DiagnosticInfo( Is the correct encoding: {"SymbolicId":1,"NamespaceUri":0,"Locale":2,"LocalizedText":3,"AdditionalInfo":"foo","InnerStatusCode":null,"InnerDiagnosticInfo":null} or: {"SymbolicId":1,"NamespaceUri":0,"Locale":2,"LocalizedText":3,"AdditionalInfo":"foo"} Speaking with Randy he said the recommendation is to omit the fields. The spec should explicitly mention this, either in each "struct-like" built-in type, or as a general recommendation before they are defined. | ||||
Tags | No tags attached. | ||||
Commit Version | |||||
Fix Due Date | |||||
|
Table 38 – JSON Encoding Rules for Structures seems to already do this. |
|
Updated DiagnosticInfo to explicitly specify default values for all fields an indicate that fields are omitted if equal to the default value. |
|
Agreed to changes edited in web meeting. |
Date Modified | Username | Field | Change |
---|---|---|---|
2022-03-23 12:21 | Kevin Herron (Inductive Automation) | New Issue | |
2022-07-12 16:07 | Jim Luth | Target Version | => 1.05.03 RC1 |
2022-07-12 16:07 | Jim Luth | Assigned To | => Randy Armstrong |
2022-07-12 16:07 | Jim Luth | Status | new => assigned |
2022-12-29 15:47 | Randy Armstrong | Status | assigned => resolved |
2022-12-29 15:47 | Randy Armstrong | Resolution | open => no change required |
2022-12-29 15:47 | Randy Armstrong | Note Added: 0018373 | |
2023-01-24 17:02 | Randy Armstrong | Note Added: 0018568 | |
2023-01-24 17:02 | Randy Armstrong | Resolution | no change required => fixed |
2023-01-24 17:06 | Jim Luth | Status | resolved => closed |
2023-01-24 17:06 | Jim Luth | Fixed in Version | => 1.05.03 RC1 |
2023-01-24 17:06 | Jim Luth | Note Added: 0018571 |