diff --git a/internal/metrics/api.go b/internal/metrics/api.go index 11f76ed0..9d6ed3f5 100644 --- a/internal/metrics/api.go +++ b/internal/metrics/api.go @@ -49,22 +49,6 @@ func MakeAPIClient(ctx context.Context, baseAPIURL, apiKey string) *APIClient { } } -// PrewarmConnection sends a redundant GET request to the Datadog API to prewarm the TSL connection -func (cl *APIClient) PrewarmConnection() error { - req, err := http.NewRequest("GET", cl.makeRoute("validate"), nil) - if err != nil { - return fmt.Errorf("Couldn't create prewarming request: %v", err) - } - req = req.WithContext(cl.context) - - cl.addAPICredentials(req) - _, err = cl.httpClient.Do(req) - if err != nil { - return fmt.Errorf("Couldn't contact server for prewarm request") - } - return nil -} - // SendMetrics posts a batch metrics payload to the Datadog API func (cl *APIClient) SendMetrics(metrics []APIMetric) error { content, err := marshalAPIMetricsModel(metrics) diff --git a/internal/metrics/api_test.go b/internal/metrics/api_test.go index 178ab11c..17918849 100644 --- a/internal/metrics/api_test.go +++ b/internal/metrics/api_test.go @@ -29,22 +29,6 @@ func TestAddAPICredentials(t *testing.T) { assert.Equal(t, "http://some-api.com/endpoint?api_key=12345", req.URL.String()) } -func TestPrewarmConnection(t *testing.T) { - - called := false - server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { - called = true - assert.Equal(t, "/validate?api_key=12345", r.URL.String()) - })) - defer server.Close() - - cl := MakeAPIClient(context.Background(), server.URL, mockAPIKey) - err := cl.PrewarmConnection() - - assert.NoError(t, err) - assert.True(t, called) -} - func TestSendMetricsSuccess(t *testing.T) { called := false server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { diff --git a/internal/metrics/listener.go b/internal/metrics/listener.go index c8dab241..d9d86a39 100644 --- a/internal/metrics/listener.go +++ b/internal/metrics/listener.go @@ -47,9 +47,6 @@ func MakeListener(config Config) Listener { config.BatchInterval = defaultBatchInterval } - // Do this in the background, doesn't matter if it returns - go apiClient.PrewarmConnection() - return Listener{ apiClient, &config,