Add NotificationUsername & ClientName to authentication failure (#284)

jellyfin · Sep 9, 2024 · 1927ff2 · 1927ff2
1 parent 4a50fa2
commit 1927ff2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Jellyfin.Plugin.Webhook/Notifiers/AuthenticationFailureNotifier.cs b/Jellyfin.Plugin.Webhook/Notifiers/AuthenticationFailureNotifier.cs
@@ -46,6 +46,8 @@ public async Task OnEvent(AuthenticationRequestEventArgs eventArgs)
         dataObject[nameof(eventArgs.DeviceId)] = eventArgs.DeviceId ?? string.Empty;
         dataObject[nameof(eventArgs.DeviceName)] = eventArgs.DeviceName ?? string.Empty;
         dataObject[nameof(eventArgs.RemoteEndPoint)] = eventArgs.RemoteEndPoint ?? string.Empty;
+        dataObject["NotificationUsername"] = eventArgs.Username ?? string.Empty;
+        dataObject["ClientName"] = eventArgs.App ?? string.Empty;
 
         await _webhookSender.SendNotification(NotificationType.AuthenticationFailure, dataObject)
             .ConfigureAwait(false);

diff --git a/README.md b/README.md
@@ -85,8 +85,6 @@ See [Templates](Jellyfin.Plugin.Webhook/Templates) for sample templates.
         - Playback client name
     - Client
         - Playback client name
-    - App
-        - client name in Authentication Failure
     - RemoteEndPoint 
         - IP Address of client