Fixing bug with dropping baggage when TracePropagationBehaviorExtract=IGNORE (#9037)

* Fixing issues with dropping baggage when TracePropagationBehaviorExtract = IGNORE

* updating calls to BaggagePropagator from unit tests

* fixing call to base constructor

* updating to handle behaviorExtract=IGNORE in CoreTracer rather than BaggagePropagator

Author: mhlidd

dd-trace-core/src/main/java/datadog/trace/core/CoreTracer.java

@@ -4,7 +4,7 @@
 import static datadog.trace.api.DDTags.DJM_ENABLED;
 import static datadog.trace.api.DDTags.DSM_ENABLED;
 import static datadog.trace.api.DDTags.PROFILING_CONTEXT_ENGINE;
-import static datadog.trace.api.TracePropagationBehaviorExtract.RESTART;
+import static datadog.trace.api.TracePropagationBehaviorExtract.IGNORE;
 import static datadog.trace.bootstrap.instrumentation.api.AgentPropagation.BAGGAGE_CONCERN;
 import static datadog.trace.bootstrap.instrumentation.api.AgentPropagation.DSM_CONCERN;
 import static datadog.trace.bootstrap.instrumentation.api.AgentPropagation.TRACING_CONCERN;
@@ -716,7 +716,8 @@ private CoreTracer(
     if (config.isDataStreamsEnabled()) {
       Propagators.register(DSM_CONCERN, this.dataStreamsMonitoring.propagator());
     }
-    if (config.isBaggagePropagationEnabled()) {
+    if (config.isBaggagePropagationEnabled()
+        && config.getTracePropagationBehaviorExtract() != IGNORE) {
       Propagators.register(BAGGAGE_CONCERN, new BaggagePropagator(config));
     }
 

dd-trace-core/src/main/java/datadog/trace/core/baggage/BaggagePropagator.java

@@ -32,7 +32,7 @@ public class BaggagePropagator implements Propagator {
   public BaggagePropagator(Config config) {
     this(
         config.isBaggageInject(),
-        config.isBaggageInject(),
+        config.isBaggageExtract(),
         config.getTraceBaggageMaxItems(),
         config.getTraceBaggageMaxBytes());
   }