Merge pull request #35 from getsentry/ref/user-other-mut

ref: make user.other mutable
getsentry · May 1, 2019 · 1e6346e · 1e6346e
2 parents d61e2ee + fe2f333
commit 1e6346e
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/src/Sentry.Protocol/User.cs b/src/Sentry.Protocol/User.cs
@@ -48,19 +48,15 @@ public class User
         public string Username { get; set; }
 
         [DataMember(Name = "other", EmitDefaultValue = false)]
-        internal Dictionary<string, string> InternalOther;
+        internal IDictionary<string, string> InternalOther;
 
         /// <summary>
-        /// Additional informations about the user
+        /// Additional information about the user
         /// </summary>
-        /// <value>
-        /// Additional informations about the user
-        /// </value>       
-        public IReadOnlyDictionary<string, string> Other
+        public IDictionary<string, string> Other
         {
             get => InternalOther ?? (InternalOther = new Dictionary<string, string>());
-            set => InternalOther = value.ToDictionary(entry => entry.Key,
-                                                        entry => entry.Value);
+            set => InternalOther = value;
         }
 
         public User Clone()