Skip to content

Commit

Permalink
Fix Flagsmith provider no key exception
Browse files Browse the repository at this point in the history
Signed-off-by: Vladimir Petrusevici <[email protected]>
  • Loading branch information
Vladimir Petrusevici authored and Vladimir Petrusevici committed Nov 8, 2023
1 parent 49efdde commit b6a2732
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -65,8 +65,13 @@ public FlagsmithProvider(IFlagsmithProviderConfiguration providerOptions, IFlags

private Task<IFlags> GetFlags(EvaluationContext ctx)
{
var key = ctx?.GetValue(Configuration.TargetingKey)?.AsString;
return string.IsNullOrEmpty(key)
string key = null;
if(ctx != null && ctx.TryGetValue(Configuration.TargetingKey, out var value))
{
key = value?.AsString;
}

return string.IsNullOrEmpty(key)
? _flagsmithClient.GetEnvironmentFlags()
: _flagsmithClient.GetIdentityFlags(key, ctx.AsDictionary().Select(x => new Trait(x.Key, x.Value.AsObject) as ITrait).ToList());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<PropertyGroup>
<TargetFrameworks>netstandard20</TargetFrameworks>
<PackageId>OpenFeature.Contrib.Providers.Flagsmith</PackageId>
<VersionNumber>0.1.0</VersionNumber>
<VersionNumber>0.1.1</VersionNumber>
<!--x-release-please-version -->
<Version>$(VersionNumber)</Version>
<AssemblyVersion>$(VersionNumber)</AssemblyVersion>
Expand Down

0 comments on commit b6a2732

Please sign in to comment.