From 86bb93515c85283767df2a1a5588c5580250150a Mon Sep 17 00:00:00 2001
From: David <daullmer@gmail.com>
Date: Wed, 12 Aug 2020 14:55:55 +0200
Subject: [PATCH] Migrate to Asp.Net Core Web Api

---
 ...s => PushbulletNotificationsController.cs} | 48 ++++++++-----------
 Pushbullet/Api/TestNotification.cs            | 23 ---------
 Pushbullet/Pushbullet.csproj                  |  1 +
 3 files changed, 22 insertions(+), 50 deletions(-)
 rename Pushbullet/Api/{ServerApiEntryPoints.cs => PushbulletNotificationsController.cs} (60%)
 delete mode 100644 Pushbullet/Api/TestNotification.cs

diff --git a/Pushbullet/Api/ServerApiEntryPoints.cs b/Pushbullet/Api/PushbulletNotificationsController.cs
similarity index 60%
rename from Pushbullet/Api/ServerApiEntryPoints.cs
rename to Pushbullet/Api/PushbulletNotificationsController.cs
index c2a86fd..a7c3dc0 100644
--- a/Pushbullet/Api/ServerApiEntryPoints.cs
+++ b/Pushbullet/Api/PushbulletNotificationsController.cs
@@ -1,58 +1,50 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Net.Mime;
 using System.Threading.Tasks;
 using MediaBrowser.Common.Net;
 using MediaBrowser.Model.Serialization;
-using MediaBrowser.Model.Services;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc;
 using Pushbullet.Configuration;
 
 namespace Pushbullet.Api
 {
     /// <summary>
-    /// API endpoints.
+    /// Pushbullet notifications controller.
     /// </summary>
-    public class ServerApiEntryPoints : IService
+    [ApiController]
+    [Route("Notification/Pushbullet")]
+    [Produces(MediaTypeNames.Application.Json)]
+    public class PushbulletNotificationsController : ControllerBase
     {
         private readonly IHttpClient _httpClient;
         private readonly IJsonSerializer _jsonSerializer;
 
         /// <summary>
-        /// Initializes a new instance of the <see cref="ServerApiEntryPoints"/> class.
+        /// Initializes a new instance of the <see cref="PushbulletNotificationsController"/> class.
         /// </summary>
         /// <param name="jsonSerializer">Instance of the <see cref="IJsonSerializer"/> interface.</param>
         /// <param name="httpClient">Instance of the <see cref="IHttpClient"/> interface.</param>
-        public ServerApiEntryPoints(IJsonSerializer jsonSerializer, IHttpClient httpClient)
+        public PushbulletNotificationsController(IJsonSerializer jsonSerializer, IHttpClient httpClient)
         {
             _jsonSerializer = jsonSerializer;
             _httpClient = httpClient;
         }
 
-        private static PushbulletOptions GetOptions(string userId)
-        {
-            return Plugin.Instance!.Configuration.GetOptions()
-                .FirstOrDefault(i => string.Equals(i.UserId, userId, StringComparison.OrdinalIgnoreCase));
-        }
-
         /// <summary>
-        /// Send test notification.
+        /// Send Pushbullet test notification.
         /// </summary>
-        /// <param name="request">Request to send.</param>
-        public void Post(TestNotification request)
+        /// <param name="userId">The user id of the Jellyfin user.</param>
+        /// <response code="204">Test notification successfully sent.</response>
+        /// <returns>A <see cref="NoContentResult"/> indicating success.</returns>
+        [HttpPost("Test/{userId}")]
+        [ProducesResponseType(StatusCodes.Status204NoContent)]
+        public async Task<ActionResult> PostAsync([FromRoute] string userId)
         {
-            PostAsync(request)
-                .GetAwaiter()
-                .GetResult();
-        }
-
-        /// <summary>
-        /// Send test notification.
-        /// </summary>
-        /// <param name="request">Request to send.</param>
-        /// <returns>A <see cref="Task"/>.</returns>
-        public async Task PostAsync(TestNotification request)
-        {
-            var options = GetOptions(request.UserId!);
+            var options = Plugin.Instance!.Configuration.GetOptions()
+                .FirstOrDefault(i => string.Equals(i.UserId, userId, StringComparison.OrdinalIgnoreCase));
 
             var parameters = new Dictionary<string, string>
             {
@@ -71,6 +63,8 @@ public async Task PostAsync(TestNotification request)
             };
 
             await _httpClient.Post(requestOptions).ConfigureAwait(false);
+
+            return NoContent();
         }
     }
 }
\ No newline at end of file
diff --git a/Pushbullet/Api/TestNotification.cs b/Pushbullet/Api/TestNotification.cs
deleted file mode 100644
index fa8594e..0000000
--- a/Pushbullet/Api/TestNotification.cs
+++ /dev/null
@@ -1,23 +0,0 @@
-using MediaBrowser.Model.Services;
-
-namespace Pushbullet.Api
-{
-    /// <summary>
-    /// Test notification request.
-    /// </summary>
-    [Route("/Notification/Pushbullet/Test/{UserId}", "POST", Summary = "Tests Pushbullet")]
-    public class TestNotification : IReturnVoid
-    {
-        /// <summary>
-        /// Gets or sets user Id to test.
-        /// </summary>
-        [ApiMember(
-            Name = "UserId",
-            Description = "User Id",
-            IsRequired = true,
-            DataType = "string",
-            ParameterType = "path",
-            Verb = "GET")]
-        public string? UserId { get; set; }
-    }
-}
\ No newline at end of file
diff --git a/Pushbullet/Pushbullet.csproj b/Pushbullet/Pushbullet.csproj
index 30baa5f..b5ec413 100644
--- a/Pushbullet/Pushbullet.csproj
+++ b/Pushbullet/Pushbullet.csproj
@@ -23,6 +23,7 @@
   </ItemGroup>
   <ItemGroup>
     <PackageReference Include="Jellyfin.Controller" Version="10.*-*" />
+    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="2.2.0" />
   </ItemGroup>
 
   <PropertyGroup Condition=" '$(Configuration)' == 'Debug' ">