View Issue Details

IDProjectCategoryView StatusLast Update
000919310000-014: PubSubSpecpublic2023-10-24 15:12
ReporterJim Luth Assigned ToMatthias Damm  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.05.01 
Target Version1.05.03Fixed in Version1.05.03 
Summary0009193: StructureDefinition.DefaultEncodingId is not useful as described.
Description

In retrospect the field should have been an key-value array. Since that is not an option we need a compromise.

The proposal is this:

Part 6 is updated to explicitly data that JSON or XML encoded ExtensionObjects may use the DataType or the DataEncoding Id.
The DataType can only be used if the ExtensionObjects is being encoded as part of a larger JSON or XML document.
If JSON is being encoded into a binary stream then the DataEncodingId shall be used.

Part 3 is updated to say DefaultEncodingId shall always contain the BinaryEncodingId with a note that the DataType id is used for JSON and XML encoding.

TagsNo tags attached.
Commit Version1.05.03
Fix Due Date2023-11-01

Relationships

related to 0008179 closedRandy Armstrong 10000-006: Mappings StructureDefinition.DefaultEncodingId is not useful as described. 

Activities

Randy Armstrong

2023-10-10 15:28

administrator   ~0020112

Updated 5.4.2.16 to require the use of the DataType NodeId when ExtensionObjects are encoded within a JSON document.

Jim Luth

2023-10-10 15:28

administrator   ~0020113

Agreed to changes in web meeting,

Matthias Damm

2023-10-10 15:28

developer   ~0020114

From my colleague Frank:
It is not clear what the condition "If the ExtensionObject is being serialized as part of a JSON document [...]" means.

The binary encoding still uses the EncodingId for JSON bodys, it would be possible to also use the TypeId by adding a
value for JSON bodys to the Encoding byte.

Matthias:
Changing the TypeId to DataType also in binary ExtensionObject would solve also the discussion we had for related issue 0009073 and would eliminate the need for the JSON DataTypeEncoding nodes.

Randy Armstrong

2023-10-10 15:28

administrator   ~0020115

Changed text to:
When serializing a PubSub Message (see XXX), the NodeId of the DataType Node is used instead.

Jim Luth

2023-10-10 15:28

administrator   ~0020116

Agreed to changes in F2F.

Jim Luth

2023-10-10 15:28

administrator   ~0020117

Last edited: 2023-10-10 15:29

We reverted Part 6 to allow either the DataType Encoding Node or a DataType Node. Part 14 needs to clarify "When serializing a PubSub Message (see XXX), only the NodeId of the DataType Node is used , never the DataType Encoding node."

Matthias Damm

2023-10-17 14:17

developer   ~0020200

7.2.5 JSON message mapping
7.2.5.1 General

Added following text:
The JSON message mapping uses the OPC UA JSON encoding defined in OPC 10000-6. If an ExtensionObject is encoded, the TypeId shall be the DataType NodeId of the contained structure.

Jim Luth

2023-10-24 15:12

administrator   ~0020223

Agreed to changes in web meeting.

Issue History

Date Modified Username Field Change
2023-10-10 15:28 Jim Luth New Issue
2023-10-10 15:28 Jim Luth Status new => assigned
2023-10-10 15:28 Jim Luth Assigned To => Matthias Damm
2023-10-10 15:28 Jim Luth Issue generated from: 0008179
2023-10-10 15:28 Jim Luth Note Added: 0020112
2023-10-10 15:28 Jim Luth Note Added: 0020113
2023-10-10 15:28 Jim Luth Note Added: 0020114
2023-10-10 15:28 Jim Luth Note Added: 0020115
2023-10-10 15:28 Jim Luth Note Added: 0020116
2023-10-10 15:28 Jim Luth Note Added: 0020117
2023-10-10 15:28 Jim Luth Relationship added related to 0008179
2023-10-10 15:28 Jim Luth Project 10000-006: Mappings => 10000-014: PubSub
2023-10-10 15:29 Jim Luth Note Edited: 0020117
2023-10-10 15:29 Jim Luth Commit Version => 1.05.03
2023-10-10 15:29 Jim Luth Fix Due Date => 2023-11-01
2023-10-17 14:17 Matthias Damm Status assigned => resolved
2023-10-17 14:17 Matthias Damm Resolution open => fixed
2023-10-17 14:17 Matthias Damm Fixed in Version => 1.05.03
2023-10-17 14:17 Matthias Damm Note Added: 0020200
2023-10-24 15:12 Jim Luth Status resolved => closed
2023-10-24 15:12 Jim Luth Note Added: 0020223