From e99a81c34a98588090ab08159ef91d367b6c0cb1 Mon Sep 17 00:00:00 2001
From: trawzified <55751269+tr4wzified@users.noreply.github.com>
Date: Thu, 30 May 2024 22:42:38 +0200
Subject: [PATCH] Fix accidental deadlock on NexusApi.Validate(), add extra
 validation logging

---
 Wabbajack.Downloaders.Dispatcher/DownloadDispatcher.cs | 3 ++-
 Wabbajack.Downloaders.Nexus/NexusDownloader.cs         | 3 ++-
 Wabbajack.Networking.NexusApi/NexusApi.cs              | 5 +++--
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/Wabbajack.Downloaders.Dispatcher/DownloadDispatcher.cs b/Wabbajack.Downloaders.Dispatcher/DownloadDispatcher.cs
index c01eb31e..5511a58b 100644
--- a/Wabbajack.Downloaders.Dispatcher/DownloadDispatcher.cs
+++ b/Wabbajack.Downloaders.Dispatcher/DownloadDispatcher.cs
@@ -148,8 +148,9 @@ public class DownloadDispatcher
 
             return result;
         }
-        catch (HttpException)
+        catch (HttpException ex)
         {
+            _logger.LogError($"Failed verifying {a.State.PrimaryKeyString}: {ex}");
             await _verificationCache.Put(a.State, false);
             return false;
         }
diff --git a/Wabbajack.Downloaders.Nexus/NexusDownloader.cs b/Wabbajack.Downloaders.Nexus/NexusDownloader.cs
index 74a86805..1168cb5c 100644
--- a/Wabbajack.Downloaders.Nexus/NexusDownloader.cs
+++ b/Wabbajack.Downloaders.Nexus/NexusDownloader.cs
@@ -217,8 +217,9 @@ public class NexusDownloader : ADownloader<Nexus>, IUrlDownloader
             
             return fileInfo.info.FileId == state.FileID;
         }
-        catch (HttpException)
+        catch (HttpException ex)
         {
+            _logger.LogError($"HttpException: {ex} on {archive.Name}");
             return false;
         }
     }
diff --git a/Wabbajack.Networking.NexusApi/NexusApi.cs b/Wabbajack.Networking.NexusApi/NexusApi.cs
index f75974be..77d56277 100644
--- a/Wabbajack.Networking.NexusApi/NexusApi.cs
+++ b/Wabbajack.Networking.NexusApi/NexusApi.cs
@@ -51,7 +51,6 @@ public class NexusApi
     public virtual async Task<(ValidateInfo info, ResponseMetadata header)> Validate(
         CancellationToken token = default)
     {
-        using var _ = await _authLock.WaitAsync();
         var (isApi, code) = await GetAuthInfo();
 
         if (isApi)
@@ -189,7 +188,6 @@ public class NexusApi
     protected virtual async ValueTask<HttpRequestMessage> GenerateMessage(HttpMethod method, string uri,
         params object?[] parameters)
     {
-        using var _ = await _authLock.WaitAsync();
         var msg = new HttpRequestMessage();
         msg.Method = method;
 
@@ -233,6 +231,7 @@ public class NexusApi
 
     private async ValueTask<(bool IsApiKey, string code)> GetAuthInfo()
     {
+        using var _ = await _authLock.WaitAsync();
         if (AuthInfo.HaveToken())
         {
             var info = await AuthInfo.Get();
@@ -273,6 +272,8 @@ public class NexusApi
         var response = await _client.PostAsync($"https://users.nexusmods.com/oauth/token", content, cancel);
         var responseString = await response.Content.ReadAsStringAsync(cancel);
         var newJwt = JsonSerializer.Deserialize<JwtTokenReply>(responseString);
+        if (newJwt != null) 
+            newJwt.ReceivedAt = DateTime.UtcNow.ToFileTimeUtc();
         
         state.OAuth = newJwt;
         await AuthInfo.SetToken(state);