Refactor storage providers and update BackupController

Refactored the storage provider interfaces and implementations:
- Added a new using directive for `Octockup.Server.Extensions` in `BackupController.cs`.
- Modified `GetProviders` in `BackupController.cs` to use `GetParametersKeys`.
- Updated `AddDbContext` in `ServiceCollectionExtensions.cs` to use `filename`.
- Added `ScpProvider` in `Program.cs`.
- Changed `FtpProvider` and `ScpProvider` to implement `IStorageProvider<BaseStorageParameters>`.
- Removed `Parameters` from `IStorageProvider` and added it to `IStorageProvider<TParams>`.
- Added `GetParametersKeys` extension method in `StorageProviderExtensions.cs`.
- Introduced `BaseStorageParameters` class to encapsulate common storage parameters.

Sources/Octockup.Server/Controllers/BackupController.cs

@@ -6,6 +6,7 @@
 using Gridify.EntityFramework;
 using Microsoft.AspNetCore.Mvc;
 using Octockup.Server.Database;
+using Octockup.Server.Extensions;
 using Octockup.Server.Models.Dto;
 using Octockup.Server.Providers.Storage;
 using Microsoft.AspNetCore.Authorization;
@@ -45,7 +46,7 @@ public IEnumerable<StorageProviderInfo> GetProviders()
             return _storageProviders.Select(p => new StorageProviderInfo()
             {
                 Name = p.Name,
-                Parameters = p.Parameters
+                Parameters = p.GetParametersKeys()
             });
         }
     }

Sources/Octockup.Server/Extensions/ServiceCollectionExtensions.cs

@@ -21,7 +21,7 @@ public static IServiceCollection AddDbContext<TContext>(this IServiceCollection
             string filename = FileSystemHelpers.GetFilePath(databaseFile);
             return services
                 .AddDbContext<TContext>(options => options
-                .UseSqlite("Data Source=" + databaseFile)
+                .UseSqlite("Data Source=" + filename)
                 .UseLazyLoadingProxies());
         }
 

Sources/Octockup.Server/Extensions/StorageProviderExtensions.cs

@@ -0,0 +1,19 @@
+using Octockup.Server.Providers.Storage;
+
+namespace Octockup.Server.Extensions
+{
+    public static class StorageProviderExtensions
+    {
+        public static IEnumerable<string> GetParametersKeys(this IStorageProvider storageProvider)
+        {
+            // storageProvider is IStorageProvider but actually is IStorageProvider<>, get generic type
+            var type = storageProvider
+                .GetType()
+                .GetInterfaces()
+                .First(i => i.IsGenericType && i.GetGenericTypeDefinition() == typeof(IStorageProvider<>))
+                .GetGenericArguments()
+                .First();
+            return type.GetProperties().Select(p => p.Name);
+        }
+    }
+}

Sources/Octockup.Server/Program.cs

@@ -18,6 +18,7 @@ public static void Main(string[] args)
             builder.Services.AddControllers();
             builder.Services
                 .AddStorageProvider<FtpProvider>()
+                .AddStorageProvider<ScpProvider>()
                 .AddHttpContextAccessor()
                 .AddValidatorsFromAssemblyContaining<Program>()
                 .AddFluentValidationAutoValidation()

Sources/Octockup.Server/Providers/Storage/BaseStorageParameters.cs

@@ -0,0 +1,11 @@
+namespace Octockup.Server.Providers.Storage
+{
+    public class BaseStorageParameters
+    {
+        public string RemoteHost { get; set; } = string.Empty;
+        public int RemotePort { get; set; }
+        public string Username { get; set; } = string.Empty;
+        public string Password { get; set; } = string.Empty;
+        public string RemotePath { get; set; } = string.Empty;
+    }
+}

Sources/Octockup.Server/Providers/Storage/FtpProvider.cs

@@ -2,19 +2,14 @@
 
 namespace Octockup.Server.Providers.Storage
 {
-    public class FtpProvider : IStorageProvider
+    public class FtpProvider : IStorageProvider<BaseStorageParameters>
     {
         public string Name => "FTP";
-
-        public IEnumerable<string> Parameters =>
-        [
-            "RemoteHost", "RemotePort", "RemotePath",
-            "Username", "Password"
-        ];
+        public BaseStorageParameters Parameters { get; set; } = null!;
 
         public IEnumerable<RemoteFileInfo> GetAllFiles()
         {
             throw new NotImplementedException();
         }
     }
-}
+}

Sources/Octockup.Server/Providers/Storage/IStorageProvider.cs

@@ -2,10 +2,14 @@
 
 namespace Octockup.Server.Providers.Storage
 {
+    public interface IStorageProvider<TParams> : IStorageProvider where TParams : class
+    {
+        TParams Parameters { get; set; }
+    }
+
     public interface IStorageProvider
     {
         string Name { get; }
-        IEnumerable<string> Parameters { get; }
         IEnumerable<RemoteFileInfo> GetAllFiles();
     }
 }

Sources/Octockup.Server/Providers/Storage/ScpProvider.cs

@@ -2,11 +2,10 @@
 
 namespace Octockup.Server.Providers.Storage
 {
-    public class ScpProvider : IStorageProvider
+    public class ScpProvider : IStorageProvider<BaseStorageParameters>
     {
-        public string Name => throw new NotImplementedException();
-
-        public IEnumerable<string> Parameters => throw new NotImplementedException();
+        public string Name => "SCP";
+        public BaseStorageParameters Parameters { get; set; } = null!;
 
         public IEnumerable<RemoteFileInfo> GetAllFiles()
         {