[exporter/datadog] Remove Config.OnlyMetadata and APIConfig.GetCensoredKey

CHANGELOG.md

@@ -9,6 +9,9 @@
 - `filelogreceiver`, `journaldreceiver`, `syslogreceiver`, `tcplogreceiver`, `udplogreceiver`:
   - Updated data model to align with stable logs data model, which includes various breaking changes. (#8835)
     - A detailed [Upgrade Guide](https://github.com/open-telemetry/opentelemetry-log-collection/releases/tag/v0.28.0) is available in the log-collection v0.28.0 release notes. 
+- `datadogexporter`: Remove `OnlyMetadata` method from `Config` struct (#8980)
+- `datadogexporter`: Remove `GetCensoredKey` method from `APIConfig` struct (#8980)
+
 ### 🧰 Bug fixes 🧰
 
 - `hostmetricsreceiver`: Use cpu times for time delta in cpu.utilization calculation (#8856)

exporter/datadogexporter/config/config.go

@@ -20,7 +20,6 @@ import (
 	"fmt"
 	"regexp"
 	"strings"
-	"sync"
 
 	"go.opentelemetry.io/collector/config"
 	"go.opentelemetry.io/collector/config/confignet"
@@ -61,15 +60,6 @@ type APIConfig struct {
 	Site string `mapstructure:"site"`
 }
 
-// GetCensoredKey returns the API key censored for logging purposes.
-// Deprecated: [v0.48.0] Will be removed in v0.49.0.
-func (api *APIConfig) GetCensoredKey() string {
-	if len(api.Key) <= 5 {
-		return api.Key
-	}
-	return strings.Repeat("*", len(api.Key)-5) + api.Key[len(api.Key)-5:]
-}
-
 // MetricsConfig defines the metrics exporter specific configuration options
 type MetricsConfig struct {
 	// Quantiles states whether to report quantiles from summary metrics.
@@ -311,20 +301,10 @@ type Config struct {
 	// Disable this in the Collector if you are using an agent-collector setup.
 	UseResourceMetadata bool `mapstructure:"use_resource_metadata"`
 
-	// onceMetadata ensures only one exporter (metrics/traces) sends host metadata
-	onceMetadata sync.Once
-
 	// warnings stores non-fatal configuration errors.
 	warnings []error
 }
 
-// OnceMetadata gets a sync.Once instance used for initializing the host metadata.
-// Deprecated: [v0.48.0] do not use, will be removed on v0.49.0.
-// TODO (#8373): Remove this method.
-func (c *Config) OnceMetadata() *sync.Once {
-	return &c.onceMetadata
-}
-
 // Sanitize tries to sanitize a given configuration
 func (c *Config) Sanitize(logger *zap.Logger) error {
 	if c.TagsConfig.Env == "" {

exporter/datadogexporter/config/config_test.go

@@ -161,18 +161,6 @@ func TestInvalidHostname(t *testing.T) {
 	require.Error(t, err)
 }
 
-func TestCensorAPIKey(t *testing.T) {
-	cfg := APIConfig{
-		Key: "ddog_32_characters_long_api_key1",
-	}
-
-	assert.Equal(
-		t,
-		"***************************_key1",
-		cfg.GetCensoredKey(),
-	)
-}
-
 func TestIgnoreResourcesValidation(t *testing.T) {
 	validCfg := Config{Traces: TracesConfig{IgnoreResources: []string{"[123]"}}}
 	invalidCfg := Config{Traces: TracesConfig{IgnoreResources: []string{"[123"}}}

exporter/datadogexporter/factory.go

@@ -141,9 +141,6 @@ func (f *factory) createMetricsExporter(
 				go metadata.Pusher(ctx, set, newMetadataConfigfromConfig(cfg), attrs)
 			})
 
-			// Consume configuration sync.Once to preserve behavior.
-			// TODO (#8373): Remove this call.
-			cfg.OnceMetadata().Do(func() {})
 			return nil
 		}
 	} else {
@@ -204,9 +201,6 @@ func (f *factory) createTracesExporter(
 				go metadata.Pusher(ctx, set, newMetadataConfigfromConfig(cfg), attrs)
 			})
 
-			// Use configuration sync.Once to do nothing to preserve behavior.
-			// TODO (#8373): Remove this call.
-			cfg.OnceMetadata().Do(func() {})
 			return nil
 		}
 	} else {

exporter/datadogexporter/metrics_exporter.go

@@ -169,10 +169,6 @@ func (exp *metricsExporter) PushMetricsData(ctx context.Context, md pdata.Metric
 			}
 			go metadata.Pusher(exp.ctx, exp.params, newMetadataConfigfromConfig(exp.cfg), attrs)
 		})
-
-		// Consume configuration's sync.Once to preserve behavior.
-		// TODO (#8373): Remove this function call.
-		exp.cfg.OnceMetadata().Do(func() {})
 	}
 
 	consumer := metrics.NewConsumer()

exporter/datadogexporter/traces_exporter.go

@@ -124,10 +124,6 @@ func (exp *traceExporter) pushTraceData(
 			}
 			go metadata.Pusher(exp.ctx, exp.params, newMetadataConfigfromConfig(exp.cfg), attrs)
 		})
-
-		// Consume configuration's sync.Once to preserve behavior.
-		// TODO (#8373): Remove this function call.
-		exp.cfg.OnceMetadata().Do(func() {})
 	}
 
 	// convert traces to datadog traces and group trace payloads by env