amino / amino-drone

Drone daemon to receive deployments from amino-deploy
2 stars 0 forks source link

Fix lastRespawn value in toJSON response #4

Closed danmactough closed 10 years ago

danmactough commented 10 years ago

Return milliseconds instead of a Date object. The Date object gets stringified into something not very useful to amino-deploy. Resolves #3.

danmactough commented 10 years ago

As an aside, there doesn't appear to be any point in having separate properties for this.started (and the corresponding uptime getter) and this.lastRespawn. If the process has respawned, this.lastRespawn is going to be the same as this.uptime (just stated differently -- [duration] ago, versus [duration]).

carlos8f commented 10 years ago

thanks, i wondered why this stopped working. crazy it's been a year already since i did this project.