Switch to epoch in milliseconds for metric timestamp

Fixes #147
This commit is contained in:
Les Orchard 2018-08-20 12:53:08 -04:00
Родитель 0db07f4f63
Коммит 500927ec65
2 изменённых файлов: 4 добавлений и 1 удалений

Просмотреть файл

@ -39,6 +39,9 @@ describe("lib/metrics", () => {
await subject(Object.assign({ ignored: "extra" }, params));
const body = mocks.requestPost.args[0][0].body;
expect(body).to.include.key("timestamp");
// Hacky test to assert that the timestamp is roughly equivalent to the
// current time in milliseconds - e.g. not an ISO8601 string or other
expect(Date.now() - parseInt(body.timestamp) < 1000).to.be.true;
delete body.timestamp;
expect(body).to.deep.equal(
Object.assign(

Просмотреть файл

@ -23,7 +23,7 @@ const Metrics = (module.exports = {
body: Object.assign(
{
topic: "watchdog-proxy",
timestamp: new Date().toISOString()
timestamp: Date.now()
},
data
)