Exception: System.ThrowHelper.ThrowInvalidOperationException_ConcurrentOperationsNotSupported about flagsmith-dotnet-client HOT 5 CLOSED

While I was not able to reproduce the issue with functional tests, or automatic tests using the SDK main functions, I was able to systematically replicate the issue accessing directly the TrackFeature method of the AnalycticsProcessor class and I put a fix in place in the above PR, which is ready for review now. I will update this thread as soon as we have the fix merged.

from flagsmith-dotnet-client.

Hi @ivastokic. Could you please provide us with more information? For example:
Are you using .Net Core or .Net Framework? Which version?
Also, from the info and context provided, it looks like a safe thread issue, however, it is hard to say the root cause or recommend a solution without more context. Can you generate a minimal project reproducing the issue and your current configuration?

from flagsmith-dotnet-client.

Hi, it is .NET Core app, but is it also hard to reproduce it from out side. The change with introducing enableAnalytics ON, on registering FlagsmithClient was introduced in October 2023, and worked well so far, even when this occurred, it fail on only one of pods, fine on others. We will turn off analytics for now.

from flagsmith-dotnet-client.

It looks like you have a thread sefety issue inside AnalyticsProcessor:

• according to your documentation FlagsmithClient is used as a sigleton
• it means that AnalyticsProcessor created inside FlagsmithClient is used as a sigleton too
• inside AnalyticsProcessor you use plain dictionary for feature usage tracking without any synchronization and periodically you call Clear for this dictionary

So you read and modify this dictionary concurrently without any protection. I made a screenshot from the decompiled code:

from flagsmith-dotnet-client.

Thank you @MaximTkachenko and sorry about the late reply! We will take a look into this issue soon.
In the mean time if you prefer you can submit a PR and We'll get that merged for you ASAP.

from flagsmith-dotnet-client.