DefaultUnleash.isEnabled(toggleName, context) silently dropping context

[hnicke]

Describe the bug

The Unleash Interface provides a function isEnabled(toggleName, context) with a default implementation.
In the default implementation, the context is silently dropped.
This is expected behavior i guess.

However, he DefaultUnleash implementation is missing an override for this function.
This leads to the behavior where DefaultUnleash.isEnabled(toggleName, context) silenlty drops the context.
Note that DefaultUnleash.isEnabled(toggleName, context, defaultValue) does not drop the context.

SDK information (language and version)

java client sdk 7.0.0

[pransh15]

Hey @hnicke 👋
Thanks for submitting this, someone from the team will get back to you on this soon. 😄

[gastonfournier]

Hi @hnicke the DefaultUnleash implementation seems to be working as expected (check #184 where I explained why is that). The interface is confusing nevertheless, so I'll bring this to @chriswk as an enhancement.

[hnicke]

Thank you for looking into this.
My bad - you are right!
For some odd reason it didn't work for me last week.
I can confirm it is working as intended.
Sorry for the inconvenience.

[gastonfournier]

No problem, still it is something that might be worth looking at!