Merge pull request #168 from mesosphere/analytics_identify

only identify a non-anonymous user
dcos · May 13, 2015 · c93ac0a · c93ac0a
2 parents 1a71460 + e59c39c
commit c93ac0a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/cli/dcoscli/analytics.py b/cli/dcoscli/analytics.py
@@ -85,10 +85,7 @@ def segment_identify(conf):
 
     if 'core.email' in conf:
         data = {'userId': conf.get('core.email')}
-    else:
-        data = {'anonymousId': session_id}
-
-    _segment_request('identify', data)
+        _segment_request('identify', data)
 
 
 def _segment_request(path, data):

diff --git a/cli/dcoscli/config/main.py b/cli/dcoscli/config/main.py
@@ -34,7 +34,7 @@
 from dcos import (cmds, config, constants, emitting, http, jsonitem,
                   subcommand, util)
 from dcos.errors import DCOSException
-from dcoscli.analytics import segment_identify
+from dcoscli import analytics
 
 emitter = emitting.FlatEmitter()
 
@@ -162,7 +162,7 @@ def _set(name, value):
 
     if (name == 'core.reporting' and python_value is True) or \
        (name == 'core.email'):
-        segment_identify(toml_config)
+        analytics.segment_identify(toml_config)
 
     _save_config_file(config_path, toml_config)