fix: remove private attributes from user object when bootstrap client…

… sdk (#156) as titled
statsig-io · Aug 6, 2024 · 3a2936d · 3a2936d
1 parent fa9f0c5
commit 3a2936d
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/dotnet-statsig/src/Statsig/Server/Evaluation/Evaluator.cs b/dotnet-statsig/src/Statsig/Server/Evaluation/Evaluator.cs
@@ -300,6 +300,8 @@ internal EvaluationDetails GetEvaluationDetails(string? reason = null)
                 layerConfigs.Add(hashedName, entry);
             }
 
+            var userWithoutPrivateAttributes = user.ToMapForLogging();
+
             var result = new Dictionary<string, Object>
             {
                 ["feature_gates"] = gates,
@@ -314,7 +316,7 @@ internal EvaluationDetails GetEvaluationDetails(string? reason = null)
                     ["sdkType"] = _sdkDetails.SDKType,
                     ["sdkVersion"] = _sdkDetails.SDKVersion,
                 },
-                ["user"] = user,
+                ["user"] = userWithoutPrivateAttributes,
             };
             return result;
         }

diff --git a/dotnet-statsig/src/Statsig/StatsigUser.cs b/dotnet-statsig/src/Statsig/StatsigUser.cs
@@ -247,6 +247,24 @@ internal StatsigUser GetCopyForLogging()
             return copy;
         }
 
+        internal Dictionary<string, object> ToMapForLogging()
+        {
+            return new Dictionary<string, object>
+            {
+                { "userID", UserID ?? string.Empty },
+                { "userAgent", UserAgent ?? string.Empty },
+                { "appVersion", AppVersion ?? string.Empty },
+                { "country", Country ?? string.Empty},
+                { "custom", CustomProperties ?? new Dictionary<string, object>() },
+                { "customIDs", CustomIDs },
+                { "email", Email ?? string.Empty },
+                { "ip", IPAddress ?? string.Empty },
+                { "locale", Locale ?? string.Empty },
+                { "statsigEnvironment", StatsigEnvironment },
+                // Do NOT add private attributes here
+            };
+        }
+
         internal Dictionary<string, string> GetParsedUserAgent()
         {
             if (_parsedUserAgent == null)