Revert "fix(core): Stop unfreezing Baggage"

This reverts commit bb4cac3.

Author: adinauer

sentry/src/main/java/io/sentry/Baggage.java

@@ -221,23 +221,21 @@ public Baggage(final @NotNull Baggage baggage) {
 
   @ApiStatus.Internal
   static @NotNull Baggage copyWithOverrides(
-      final @Nullable Baggage baggage,
+      final @NotNull Baggage baggage,
       final @NotNull SentryId traceId,
       final @Nullable Double sampleRand) {
-    final @NotNull Baggage source =
-        baggage == null ? new Baggage(NoOpLogger.getInstance()) : baggage;
     final @NotNull ConcurrentHashMap<String, String> keyValues =
-        new ConcurrentHashMap<>(source.keyValues);
+        new ConcurrentHashMap<>(baggage.keyValues);
     keyValues.put(DSCKeys.TRACE_ID, traceId.toString());
 
     return new Baggage(
         keyValues,
-        source.sampleRate,
+        baggage.sampleRate,
         sampleRand,
-        source.thirdPartyHeader,
-        source.mutable,
-        source.shouldFreeze,
-        source.logger);
+        baggage.thirdPartyHeader,
+        true,
+        false,
+        baggage.logger);
   }
 
   @ApiStatus.Internal

sentry/src/main/java/io/sentry/TransactionContext.java

@@ -44,7 +44,7 @@ public static TransactionContext fromPropagationContext(
       final @NotNull TransactionContext transactionContext) {
     final @NotNull Baggage baggage =
         Baggage.copyWithOverrides(
-            transactionContext.getBaggage(),
+            propagationContext.getBaggage(),
             propagationContext.getTraceId(),
             propagationContext.getSampleRand());
 

sentry/src/test/java/io/sentry/BaggageTest.kt

@@ -440,7 +440,7 @@ class BaggageTest {
   }
 
   @Test
-  fun `copy with overrides preserves frozen state`() {
+  fun `copy with overrides creates mutable baggage`() {
     val baggage =
       Baggage.fromHeader(
         "sentry-sample_rand=0.1,sentry-trace_id=75302ac48a024bde9a3b3734a82e36c8",
@@ -450,22 +450,12 @@ class BaggageTest {
 
     val copy = Baggage.copyWithOverrides(baggage, SentryId("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"), 0.2)
 
-    assertFalse(copy.isMutable)
-    assertTrue(copy.isShouldFreeze)
-    assertEquals("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", copy.traceId)
-    assertEquals(0.2, copy.sampleRand!!, 0.0001)
-    assertEquals("75302ac48a024bde9a3b3734a82e36c8", baggage.traceId)
-    assertEquals(0.1, baggage.sampleRand!!, 0.0001)
-  }
-
-  @Test
-  fun `copy with overrides creates baggage when source is null`() {
-    val copy = Baggage.copyWithOverrides(null, SentryId("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"), 0.2)
-
     assertTrue(copy.isMutable)
     assertFalse(copy.isShouldFreeze)
     assertEquals("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", copy.traceId)
     assertEquals(0.2, copy.sampleRand!!, 0.0001)
+    assertEquals("75302ac48a024bde9a3b3734a82e36c8", baggage.traceId)
+    assertEquals(0.1, baggage.sampleRand!!, 0.0001)
   }
 
   @Test

sentry/src/test/java/io/sentry/TransactionContextTest.kt

@@ -99,7 +99,6 @@ class TransactionContextTest {
   fun `fromPropagationContextAsRoot copies non trace state`() {
     val propagationBaggage = Baggage(NoOpLogger.getInstance())
     propagationBaggage.sampleRand = 0.42
-    propagationBaggage.publicKey = "propagation-public-key"
     val propagationContext =
       PropagationContext(
         SentryId("75302ac48a024bde9a3b3734a82e36c8"),
@@ -110,7 +109,6 @@ class TransactionContextTest {
       )
     val samplingDecision = TracesSamplingDecision(true, 0.3, true, 0.4)
     val transactionContext = TransactionContext("name", "op", samplingDecision)
-    transactionContext.baggage!!.publicKey = "transaction-public-key"
     transactionContext.transactionNameSource = TransactionNameSource.ROUTE
     transactionContext.description = "description"
     transactionContext.status = SpanStatus.OK
@@ -147,7 +145,6 @@ class TransactionContextTest {
     assertEquals(0.4, context.samplingDecision!!.profileSampleRate)
     assertEquals(0.42, context.baggage!!.sampleRand)
     assertEquals(propagationContext.traceId.toString(), context.baggage!!.traceId)
-    assertEquals("transaction-public-key", context.baggage!!.publicKey)
     assertNull(context.featureFlagBuffer.featureFlags)
   }