Skip to content

Commit

Permalink
Address issue on older FTMs not resolving non nullable after check wi…
Browse files Browse the repository at this point in the history
…th string.IsNullOrEmpty

Signed-off-by: Kyle Julian <[email protected]>
  • Loading branch information
kylejuliandev committed Oct 9, 2024
1 parent 9be92a9 commit b4b4765
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions src/OpenFeature/LoggingHook.cs
Original file line number Diff line number Diff line change
Expand Up @@ -139,10 +139,10 @@ internal class LoggingHookContent

public LoggingHookContent(string? domain, string? providerName, string flagKey, string? defaultValue, EvaluationContext? evaluationContext = null)
{
this._domain = string.IsNullOrEmpty(domain) ? "missing" : domain;
this._providerName = string.IsNullOrEmpty(providerName) ? "missing" : providerName;
this._domain = string.IsNullOrEmpty(domain) ? "missing" : domain!;
this._providerName = string.IsNullOrEmpty(providerName) ? "missing" : providerName!;
this._flagKey = flagKey;
this._defaultValue = string.IsNullOrEmpty(defaultValue) ? "missing" : defaultValue;
this._defaultValue = string.IsNullOrEmpty(defaultValue) ? "missing" : defaultValue!;
this._evaluationContext = evaluationContext;
}

Expand All @@ -153,19 +153,19 @@ public override string ToString()
var stringBuilder = new StringBuilder();

stringBuilder.Append("Domain:");
stringBuilder.Append(_domain);
stringBuilder.Append(this._domain);
stringBuilder.Append(Environment.NewLine);

stringBuilder.Append("ProviderName:");
stringBuilder.Append(_providerName);
stringBuilder.Append(this._providerName);
stringBuilder.Append(Environment.NewLine);

stringBuilder.Append("FlagKey:");
stringBuilder.Append(_flagKey);
stringBuilder.Append(this._flagKey);
stringBuilder.Append(Environment.NewLine);

stringBuilder.Append("DefaultValue:");
stringBuilder.Append(_defaultValue);
stringBuilder.Append(this._defaultValue);
stringBuilder.Append(Environment.NewLine);

if (this._evaluationContext != null)
Expand Down

0 comments on commit b4b4765

Please sign in to comment.