New: Webhook improvements

src/NzbDrone.Core/Notifications/Webhook/Webhook.cs

@@ -1,35 +1,78 @@
-
 using System.Collections.Generic;
+using System.Linq;
 using FluentValidation.Results;
 using NzbDrone.Core.Music;
 using NzbDrone.Common.Extensions;
+using NzbDrone.Core.Validation;
 
 namespace NzbDrone.Core.Notifications.Webhook
 {
     public class Webhook : NotificationBase<WebhookSettings>
     {
-        private readonly IWebhookService _service;
+        private readonly IWebhookProxy _proxy;
 
-        public Webhook(IWebhookService service)
+        public Webhook(IWebhookProxy proxy)
         {
-            _service = service;
+            _proxy = proxy;
         }
 
         public override string Link => "https://github.com/Lidarr/Lidarr/wiki/Webhook";
 
         public override void OnGrab(GrabMessage message)
         {
-            _service.OnGrab(message.Artist, message.Album, message.Quality, Settings);
+            var remoteAlbum = message.Album;
+            var quality = message.Quality;
+
+            var payload = new WebhookGrabPayload
+
+            {
+                EventType = "Grab",
+                Artist = new WebhookArtist(message.Artist),
+                Albums = remoteAlbum.Albums.ConvertAll(x => new WebhookAlbum(x)
+                {
+                    // TODO: Stop passing these parameters inside an album v3
+                    Quality = quality.Quality.Name,
+                    QualityVersion = quality.Revision.Version,
+                    ReleaseGroup = remoteAlbum.ParsedAlbumInfo.ReleaseGroup
+                }),
+                Release = new WebhookRelease(quality, remoteAlbum)
+            };
+
+            _proxy.SendWebhook(payload, Settings);
         }
 
         public override void OnDownload(DownloadMessage message)
         {
-            _service.OnDownload(message.Artist, message.TrackFile, Settings);
+            var trackFile = message.TrackFile;
+
+            var payload = new WebhookImportPayload
+
+            {
+                EventType = "Download",
+                Artist = new WebhookArtist(message.Artist),
+                Tracks = trackFile.Tracks.Value.ConvertAll(x => new WebhookTrack(x)
+                {
+                    // TODO: Stop passing these parameters inside an episode v3
+                    Quality = trackFile.Quality.Quality.Name,
+                    QualityVersion = trackFile.Quality.Revision.Version,
+                    ReleaseGroup = trackFile.ReleaseGroup
+                }),
+                TrackFile = new WebhookTrackFile(trackFile),
+                IsUpgrade = message.OldFiles.Any()
+            };
+
+            _proxy.SendWebhook(payload, Settings);
         }
 
         public override void OnRename(Artist artist)
         {
-            _service.OnRename(artist, Settings);
+            var payload = new WebhookPayload
+            {
+                EventType = "Rename",
+                Artist = new WebhookArtist(artist)
+            };
+
+            _proxy.SendWebhook(payload, Settings);
         }
 
         public override string Name => "Webhook";
@@ -38,9 +81,42 @@ namespace NzbDrone.Core.Notifications.Webhook
         {
             var failures = new List<ValidationFailure>();
 
-            failures.AddIfNotNull(_service.Test(Settings));
+            failures.AddIfNotNull(SendWebhookTest());
 
             return new ValidationResult(failures);
         }
+
+        private ValidationFailure SendWebhookTest()
+        {
+            try
+            {
+                var payload = new WebhookGrabPayload
+                    {
+                        EventType = "Test",
+                        Artist = new WebhookArtist()
+                        {
+                            Id = 1,
+                            Name = "Test Name",
+                            Path = "C:\\testpath",
+                            MBId = "aaaaa-aaa-aaaa-aaaaaa"
+                        },
+                        Albums = new List<WebhookAlbum>() {
+                            new WebhookAlbum()
+                            {
+                                Id = 123,
+                                Title = "Test title"
+                            }
+                        }
+                    };
+
+                _proxy.SendWebhook(payload, Settings);
+            }
+            catch (WebhookException ex)
+            {
+                return new NzbDroneValidationFailure("Url", ex.Message);
+            }
+
+            return null;
+        }
     }
 }