Merge pull request #156 from resin-io/154-restore-private-env-vars

bugfix: do not mutate original env vars object
This commit is contained in:
Kostas Lekkas 2016-06-01 14:18:27 +03:00
commit 131f7f5b22

View File

@ -36,6 +36,8 @@ exports.mixpanelTrack = (event, properties = {}) ->
message: properties.error.message message: properties.error.message
stack: properties.error.stack stack: properties.error.stack
properties = _.cloneDeep(properties)
# Don't log private env vars (e.g. api keys) # Don't log private env vars (e.g. api keys)
if properties?.app?.env? if properties?.app?.env?
try try
@ -48,7 +50,7 @@ exports.mixpanelTrack = (event, properties = {}) ->
console.log('Event:', event, JSON.stringify(properties)) console.log('Event:', event, JSON.stringify(properties))
# Mutation is bad, and it should feel bad # Mutation is bad, and it should feel bad
properties = _.assign(_.cloneDeep(properties), mixpanelProperties) properties = _.assign(properties, mixpanelProperties)
mixpanelClient.track(event, properties) mixpanelClient.track(event, properties)