bdSeq Timestamp is null

[MRIIOT]

Ignition complains about bdSeq having a null timestamp. What is the best place to enforce timestamp on bdSeq or all metrics?

Relevant Metric with null timestamp.

Metric [name=bdSeq, alias=null, timestamp=null, dataType=Int64, isHistorical=null, isTransient=null, metaData=null, properties=null, value=0, isNull=false]

Full Ignition Error.

Failed to handle payload on topic: spBv1.0/dime/DDATA/dime/dime with payload=SparkplugBPayload [timestamp=1733620068755, metrics=[Metric [name=eipSource1/Execution, alias=null, timestamp=1733620068755, dataType=String, isHistorical=null, isTransient=null, metaData=null, properties=null, value=Active, isNull=false], Metric [name=eipSource1/SYSTEM/ExecutionDuration, alias=null, timestamp=1733620068755, dataType=Double, isHistorical=null, isTransient=null, metaData=null, properties=null, value=544.0, isNull=false], Metric [name=bdSeq, alias=null, timestamp=null, dataType=Int64, isHistorical=null, isTransient=null, metaData=null, properties=null, value=0, isNull=false]], seq=207, uuid=, body=[]]

com.cirruslink.common.util.exception.CirrusException: INVALID_ARGUMENT

at com.cirruslink.mqtt.engine.gateway.sparkplug.SparkplugPayloadHandler.handlePayload(SparkplugPayloadHandler.java:234)

at com.cirruslink.mqtt.engine.gateway.sparkplug.SparkplugMessageRunnable.run(SparkplugMessageRunnable.java:72)

at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)

at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)

at java.base/java.lang.Thread.run(Unknown Source)

Caused by: java.lang.NullPointerException: Cannot invoke "java.util.Date.getTime()" because the return value of "org.eclipse.tahu.message.model.Metric.getTimestamp()" is null

at com.cirruslink.mqtt.engine.gateway.sparkplug.SparkplugBPayloadHandler.handleDeviceData(SparkplugBPayloadHandler.java:1862)

at com.cirruslink.mqtt.engine.gateway.sparkplug.SparkplugPayloadHandler.handlePayload(SparkplugPayloadHandler.java:220)

Here?

SparkplugNet/src/SparkplugNet/Core/Messages/SparkplugMessageGenerator.cs

Line 536 in 03658b2

// Add a BDSEQ metric.

[thomassorensen2650]

They added a requirement for all metrics to have an add timestamp to be compliant in version 3 of the Sparkplug specification.

https://github.com/eclipse-sparkplug/sparkplug/blob/e45b61002a2a0cacb4f11fa4be88670435ccfcb0/specification/src/main/asciidoc/chapters/Sparkplug_6_Payloads.adoc?plain=1#L475C1-L476C97

Would it be OK to create a PR that adds the message timestamp to all metrics if SparkplugSpecificationVersion.Version30 is a timestamp is not provided?.