fix: use better diff time to check sync status

closes #858
cnpm · Apr 5, 2016 · faa840d · faa840d
1 parent 0329cfa
commit faa840d
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/sync/index.js b/sync/index.js
@@ -195,9 +195,11 @@ function* checkSyncStatus() {
   }
   var diff = Date.now() - lastSyncTime;
   var oneDay = 3600000 * 24;
-  if (diff > oneDay) {
-    var err = new Error('Last sync time is expired in ' + diff + ' ms, lastSyncTime: ' + new Date(lastSyncTime));
-    err.name = 'SyncExpriedError';
+  var maxTime = Math.max(oneDay, syncInterval);
+  if (diff > maxTime) {
+    var err = new Error('Last sync time is expired in ' + diff + ' ms, lastSyncTime: ' +
+      new Date(lastSyncTime) + ', maxTime: ' + maxTime + ' ms');
+    err.name = 'SyncExpiredError';
     sendMailToAdmin(err, null, new Date());
   }
 }