Fix: braces from tag keys on DefaultSentryScopeStateProcessor.

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Fixes
 
+- Removed braces from tag keys on DefaultSentryScopeStateProcessor ([#1183](https://github.com/getsentry/sentry-dotnet/pull/1183))
 - Add fallback to Scope Stack from AspNet ([#1180](https://github.com/getsentry/sentry-dotnet/pull/1180))
 
 ## 3.9.0

src/Sentry/DefaultSentryScopeStateProcessor.cs

@@ -11,6 +11,8 @@ namespace Sentry
     /// </summary>
     public class DefaultSentryScopeStateProcessor : ISentryScopeStateProcessor
     {
+        private char[] _trimFilter = new[] { '{', '}' };
+
         /// <summary>
         /// Applies state onto a scope.
         /// </summary>
@@ -31,11 +33,9 @@ public void Apply(Scope scope, object state)
                         scope.SetTags(keyValStringObject
                             .Where(kv => !string.IsNullOrEmpty(kv.Value as string))
                             .Select(k => new KeyValuePair<string, string>(
-                                k.Key,
+                                k.Key.Trim(_trimFilter),
                                 // TODO: Candidate for serialization instead. Likely Contexts is a better fit.
                                 k.Value.ToString()!)));
-
-
                         break;
                     }
 #if !NET461

test/Sentry.Tests/DefaultSentryScopeStateProcessorTests.cs

@@ -0,0 +1,33 @@
+using System.Collections.Generic;
+using System.Linq;
+using Xunit;
+
+namespace Sentry.Tests
+{
+    public class DefaultSentryScopeStateProcessorTests
+    {
+        [Theory]
+        [InlineData("a", "a")]
+        [InlineData("{}", "")]
+        [InlineData("{OriginalFormat}", "OriginalFormat")]
+        [InlineData("OriginalFormat", "OriginalFormat")]
+        public void Apply_KeyValuePairObjectWithBraces_TagAddedWithoutBraces(string key, string expectedKey)
+        {
+            // Arrange
+            var expectedValue = "some string";
+            var list = new List<KeyValuePair<string, object>>
+            {
+                new(key, expectedValue)
+            };
+            var scopeStateProcessor = new DefaultSentryScopeStateProcessor();
+            var scope = new Scope();
+
+            // Act
+            scopeStateProcessor.Apply(scope, list);
+
+            // Assert
+            Assert.Equal(expectedKey, scope.Tags.First().Key);
+            Assert.Equal(expectedValue, scope.Tags.First().Value);
+        }
+    }
+}