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
  • Loading branch information
kylejuliandev committed Oct 4, 2024
1 parent 37870bd commit d5d08f7
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 d5d08f7

Please sign in to comment.