Skip to content

Commit 8dbe370

Browse files
committed
serialize seq_id in TelemetryRequestBody
1 parent abe734a commit 8dbe370

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

telemetry/src/main/java/datadog/telemetry/TelemetryRequestBody.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -230,6 +230,8 @@ public void writeConfiguration(ConfigSetting configSetting) throws IOException {
230230
bodyWriter.name("value").value(configSetting.stringValue());
231231
bodyWriter.setSerializeNulls(false);
232232
bodyWriter.name("origin").value(configSetting.origin.value);
233+
bodyWriter.name("seq_id").value(configSetting.seqId);
234+
// bodyWriter.setSerializeNulls(false); ?
233235
bodyWriter.endObject();
234236
}
235237

telemetry/src/test/groovy/datadog/telemetry/TelemetryRequestBodySpecification.groovy

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -79,12 +79,12 @@ class TelemetryRequestBodySpecification extends DDSpecification {
7979

8080
then:
8181
drainToString(req) == ',"configuration":[' +
82-
'{"name":"string","value":"bar","origin":"remote_config"},' +
83-
'{"name":"int","value":"2342","origin":"default"},' +
84-
'{"name":"double","value":"123.456","origin":"env_var"},' +
85-
'{"name":"map","value":"key1:value1,key2:432.32,key3:324","origin":"jvm_prop"},' +
86-
'{"name":"list","value":"1,2,3","origin":"default"},' +
87-
'{"name":"null","value":null,"origin":"default"}]'
82+
'{"name":"string","value":"bar","origin":"remote_config","seq_id":0},' +
83+
'{"name":"int","value":"2342","origin":"default","seq_id":0},' +
84+
'{"name":"double","value":"123.456","origin":"env_var","seq_id":0},' +
85+
'{"name":"map","value":"key1:value1,key2:432.32,key3:324","origin":"jvm_prop","seq_id":0},' +
86+
'{"name":"list","value":"1,2,3","origin":"default","seq_id":0},' +
87+
'{"name":"null","value":null,"origin":"default","seq_id":0}]'
8888
}
8989

9090
def 'use snake_case for setting keys'() {

0 commit comments

Comments
 (0)