From ecdbefffd9c0acbbecd8178d7d2ac285715b5e7f Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Fri, 22 Sep 2017 21:49:05 -0400
Subject: move hardcoded values into config (#336)

---
 .../Controllers/ModsController.cs                  | 27 +++++++++++++++-------
 1 file changed, 19 insertions(+), 8 deletions(-)

(limited to 'src/StardewModdingAPI.Web/Controllers')

diff --git a/src/StardewModdingAPI.Web/Controllers/ModsController.cs b/src/StardewModdingAPI.Web/Controllers/ModsController.cs
index bd5ecef9..50c23b99 100644
--- a/src/StardewModdingAPI.Web/Controllers/ModsController.cs
+++ b/src/StardewModdingAPI.Web/Controllers/ModsController.cs
@@ -4,6 +4,8 @@ using System.Linq;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Caching.Memory;
+using Microsoft.Extensions.Options;
+using StardewModdingAPI.Web.Framework.ConfigModels;
 using StardewModdingAPI.Web.Framework.ModRepositories;
 using StardewModdingAPI.Web.Models;
 
@@ -18,25 +20,34 @@ namespace StardewModdingAPI.Web.Controllers
         ** Properties
         *********/
         /// <summary>The mod repositories which provide mod metadata.</summary>
-        private readonly IDictionary<string, IModRepository> Repositories =
-            new IModRepository[]
-            {
-                new NexusRepository()
-            }
-            .ToDictionary(p => p.VendorKey, StringComparer.CurrentCultureIgnoreCase);
+        private readonly IDictionary<string, IModRepository> Repositories;
 
         /// <summary>The cache in which to store mod metadata.</summary>
         private readonly IMemoryCache Cache;
 
+        /// <summary>The number of minutes update checks should be cached before refetching them.</summary>
+        private readonly int CacheMinutes;
+
 
         /*********
         ** Public methods
         *********/
         /// <summary>Construct an instance.</summary>
         /// <param name="cache">The cache in which to store mod metadata.</param>
-        public ModsController(IMemoryCache cache)
+        /// <param name="configProvider">The config settings for mod update checks.</param>
+        public ModsController(IMemoryCache cache, IOptions<ModUpdateCheckConfig> configProvider)
         {
+            ModUpdateCheckConfig config = configProvider.Value;
+
             this.Cache = cache;
+            this.CacheMinutes = config.CacheMinutes;
+
+            this.Repositories =
+                new IModRepository[]
+                {
+                    new NexusRepository(config.NexusKey, config.NexusUserAgent, config.NexusBaseUrl, config.NexusModUrlFormat)
+                }
+                .ToDictionary(p => p.VendorKey, StringComparer.CurrentCultureIgnoreCase);
         }
 
         /// <summary>Fetch version metadata for the given mods.</summary>
@@ -82,7 +93,7 @@ namespace StardewModdingAPI.Web.Controllers
                 // fetch mod info
                 result[modKey] = await this.Cache.GetOrCreateAsync($"{repository.VendorKey}:{modID}".ToLower(), async entry =>
                 {
-                    entry.AbsoluteExpiration = DateTimeOffset.UtcNow.AddHours(1);
+                    entry.AbsoluteExpiration = DateTimeOffset.UtcNow.AddMinutes(this.CacheMinutes);
                     return await repository.GetModInfoAsync(modID);
                 });
             }
-- 
cgit