Skip to content

Commit 8d28653

Browse files
committed
feat: Add validation for tracking event name in MultiProvider
Signed-off-by: André Silva <[email protected]>
1 parent 3db4b0e commit 8d28653

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

src/OpenFeature.Providers.MultiProvider/MultiProvider.cs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,12 @@ public override void Track(string trackingEventName, EvaluationContext? evaluati
121121
throw new ObjectDisposedException(nameof(MultiProvider));
122122
}
123123

124+
if (string.IsNullOrWhiteSpace(trackingEventName))
125+
{
126+
this.LogErrorTrackingEventEmptyName();
127+
return;
128+
}
129+
124130
foreach (var registeredProvider in this._registeredProviders)
125131
{
126132
var providerContext = new StrategyPerProviderContext<object>(
@@ -672,4 +678,7 @@ internal void SetStatus(ProviderStatus providerStatus)
672678

673679
[LoggerMessage(EventId = 2, Level = LogLevel.Debug, Message = "Error tracking event {TrackingEventName} with provider {ProviderName}")]
674680
private partial void LogErrorTrackingEvent(string providerName, string trackingEventName, Exception exception);
681+
682+
[LoggerMessage(EventId = 3, Level = LogLevel.Debug, Message = "Tracking event with empty name is not allowed")]
683+
private partial void LogErrorTrackingEventEmptyName();
675684
}

0 commit comments

Comments
 (0)