Disable metrics tracking with no headers (part of #516) (#532)

* Initial commit

* Fix build issue

* Remove testnet on metrics tracker.

cmd/trade.go

@@ -541,6 +541,7 @@ func runTradeCmd(options inputs) {
 		botConfig.TickIntervalSeconds,
 		botConfig.TradingExchange,
 		botConfig.TradingPair(),
+		*options.noHeaders, // disable metrics if the CLI specified no headers
 		isTestnet,
 	)
 	if e != nil {

support/metrics/metricsTracker.go

@@ -24,12 +24,13 @@ const (
 // and can be used to directly send events to the
 // Amplitude HTTP API.
 type MetricsTracker struct {
-	client   *http.Client
-	apiKey   string
-	userID   string
-	deviceID string
-	props    commonProps
-	start    time.Time
+	client     *http.Client
+	apiKey     string
+	userID     string
+	deviceID   string
+	props      commonProps
+	start      time.Time
+	isDisabled bool
 }
 
 // TODO DS Investigate other fields to add to this top-level event.
@@ -117,6 +118,7 @@ func MakeMetricsTracker(
 	updateTimeIntervalSeconds int32,
 	exchange string,
 	tradingPair string,
+	isDisabled bool,
 	isTestnet bool,
 ) (*MetricsTracker, error) {
 	props := commonProps{
@@ -133,12 +135,13 @@ func MakeMetricsTracker(
 	}
 
 	return &MetricsTracker{
-		client:   client,
-		apiKey:   apiKey,
-		userID:   userID,
-		deviceID: deviceID,
-		props:    props,
-		start:    start,
+		client:     client,
+		apiKey:     apiKey,
+		userID:     userID,
+		deviceID:   deviceID,
+		props:      props,
+		start:      start,
+		isDisabled: isDisabled,
 	}, nil
 }
 
@@ -173,7 +176,7 @@ func (mt *MetricsTracker) SendDeleteEvent(exit bool) error {
 }
 
 func (mt *MetricsTracker) sendEvent(eventType string, eventProps interface{}) error {
-	if mt.apiKey == "" || mt.userID == "-1" {
+	if mt.apiKey == "" || mt.userID == "-1" || mt.isDisabled {
 		log.Printf("metric - not sending event metric of type '%s' because metrics are disabled", eventType)
 		return nil
 	}