getFlags removes identity identifier in evaluation context

[jhoermann]

In the current flagsmith version 7.0.1 the identity identifier gets reset within the getFlags backend call:

1. log is during init -> the identifier is set
2. log is after getFlags / init backend call -> identifier is lost in evaluation context

This results in setContext to fail because the identifier is missing:

[matthewelwell]

Hi @jhoermann, thanks for raising this. I think I understand the issue that you're describing, but it would be very helpful for us if you are able to share a minimal example to reproduce it?

[jhoermann]

Yes here: https://stackblitz.com/edit/flagsmith-bug-setcontext?file=main.js. As you can see the identifier is being stripped from the context @matthewelwell.

[matthewelwell]

Great, thanks for sharing that @jhoermann - we'll take a look into this and get it fixed up. Alternatively, if you want to submit a PR, we'd gladly accept it.

[jhoermann]

I opened up a PR - have a look @matthewelwell: #263. Still need to verify that this solves the issue and need to add a test for it though.

[khvn26]

Hello @jhoermann — we have identified an error in the backend that caused this issue and deployed the fix. We've observed correct behaviour in the reproduction StackBlitz you've provided. Please confirm whether the fix is working for you.

[jhoermann]

Yes that resolves the issue as the identifier now gets returned by the backend endpoint @khvn26 @matthewelwell.

[Roma-Potapov]

Hello, I have same problem, but in my company we use self-hosted flagsmith, and we can't just update to the newest version, so can we reopen PR of @jhoermann to fix this problem on the client side

[kyle-ssg]

This is now released under version 8.0.1 🚀