Skip to content

Commit 463f7b5

Browse files
committedFeb 19, 2025·
fix(lwm2m/aws): timestamp resolution is seconds
1 parent ef77efa commit 463f7b5

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed
 

‎lwm2m/aws/instanceMeasuresToRecord.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import {
33
TimeUnit,
44
type _Record,
55
} from '@aws-sdk/client-timestream-write'
6-
import { instanceTsAsDate } from '../instanceTs.js'
6+
import { instanceTs } from '../instanceTs.js'
77
import type { LwM2MObjectInstance } from '../LwM2MObjectInstance.js'
88
import { instanceToMeasures } from './instanceToMeasures.js'
99
import { NoHistoryMeasuresError } from './NoHistoryMeasuresError.js'
@@ -29,12 +29,12 @@ export const instanceMeasuresToRecord = ({
2929
`No measure to be stored in history for object ${ObjectID}!`,
3030
),
3131
}
32-
const instanceTs = instanceTsAsDate({
32+
const instanceTime = instanceTs({
3333
ObjectID,
3434
ObjectInstanceID,
3535
Resources,
3636
})
37-
if (instanceTs === undefined)
37+
if (instanceTime === undefined)
3838
return { error: new Error(`No timestamp found for ${ObjectID}!`) }
3939
return {
4040
record: {
@@ -55,8 +55,8 @@ export const instanceMeasuresToRecord = ({
5555
MeasureName: `${ObjectID}/${ObjectInstanceID ?? 0}`,
5656
MeasureValues: maybeMeasures.measures,
5757
MeasureValueType: MeasureValueType.MULTI,
58-
Time: instanceTs.getTime().toString(),
59-
TimeUnit: TimeUnit.MILLISECONDS,
58+
Time: instanceTime.toString(),
59+
TimeUnit: TimeUnit.SECONDS,
6060
},
6161
}
6262
}

‎lwm2m/aws/instanceToMeasure.spec.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ void describe('instanceMeasuresToRecord()', () => {
4646
Value: '45',
4747
},
4848
],
49-
Time: '1717419305000',
50-
TimeUnit: 'MILLISECONDS',
49+
Time: '1717419305',
50+
TimeUnit: 'SECONDS',
5151
})
5252
})
5353
})

0 commit comments

Comments
 (0)
Please sign in to comment.