fix: Concurrency issue with analytics processor data

Flagsmith / flagsmith-dotnet-client

.NET Standard Client for Flagsmith. Ship features with confidence using feature flags and remote config. Host yourself or use our hosted version at https://www.flagsmith.com/

https://www.flagsmith.com/

BSD 3-Clause "New" or "Revised" License

19 stars 15 forks source link

fix: Concurrency issue with analytics processor data #93

Closed novakzaballa closed 6 months ago

novakzaballa commented 6 months ago

This PR fixes a potential concurrency issue with the dictionary AnalyticsData used in the AnalyticsProcessor class. One test reproducing the issue has been added, and the mentioned dictionary has been turned into a `ConcurrentDictionary' which implements appopiate locking logic for concurrent access.

Please refer to this issue https://github.com/Flagsmith/flagsmith-dotnet-client/issues/88 for more context.