dont provide default time prop

Author: jaredmixpanel

lib/mixpanel-node.js

@@ -234,8 +234,10 @@ var create_client = function(token, config) {
             if (batch.length > 0) {
                 batch = batch.map(function (event) {
                     var properties = event.properties;
-
-                    event.properties.time = ensure_timestamp(event.properties.time || Date.now());
+                    if (endpoint === '/import' || event.properties.time) {
+                        // usually there will be a time property, but not required for `/track` endpoint
+                        event.properties.time = ensure_timestamp(event.properties.time);
+                    }
                     event.properties.token = event.properties.token || metrics.token;
                     return event;
                 });
@@ -291,7 +293,10 @@ var create_client = function(token, config) {
             properties = {};
         }
 
-        properties.time = ensure_timestamp(properties.time || Date.now());
+        // time is optional for `track`
+        if (properties.time) {
+            properties.time = ensure_timestamp(properties.time);
+        }
 
         metrics.send_event_request("/track", event, properties, callback);
     };

lib/profile_helpers.js

@@ -18,8 +18,6 @@ function merge_modifiers(data, modifiers) {
         }
         if (modifiers.hasOwnProperty("$time")) {
             data.$time = ensure_timestamp(modifiers.$time);
-        } else {
-            data.$time = Date.now();
         }
         if (modifiers.hasOwnProperty("$latitude") && modifiers.hasOwnProperty('$longitude')) {
             data.$latitude = modifiers.$latitude;

test/import.js

@@ -82,7 +82,7 @@ exports.import = {
                 properties: {
                     key1: 'val1',
                     token: 'token',
-                    time: Math.floor(mock_now_time)
+                    time: mock_now_time
                 }
             };