Miscellaneous fixes for .NET 8 (#767)

* Use Microsoft.AspNetCore.WebUtilities

* Add NoDisplay=true to the XDG Desktop Entry

Directory.Packages.props

@@ -38,6 +38,7 @@
   </ItemGroup>
   <ItemGroup>
     <!-- Microsoft -->
+    <PackageVersion Include="Microsoft.AspNetCore.WebUtilities" Version="8.0.0-rc.2.23480.2" />
     <PackageVersion Include="Microsoft.Data.Sqlite" Version="7.0.13" />
     <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="7.0.0" />
     <PackageVersion Include="Microsoft.Extensions.Hosting" Version="7.0.1" />
@@ -100,4 +101,4 @@
     <PackageVersion Include="Splat.Microsoft.Extensions.Logging" Version="14.6.37" />
     <PackageVersion Include="Vogen" Version="3.0.20" />
   </ItemGroup>
-</Project>
+</Project>

src/Networking/NexusMods.Networking.NexusWebApi.NMA/NexusMods.Networking.NexusWebApi.NMA.csproj

@@ -6,6 +6,7 @@
     <ItemGroup>
         <PackageReference Include="System.Linq.Async" />
         <PackageReference Include="System.IdentityModel.Tokens.Jwt" />
+        <PackageReference Include="Microsoft.AspNetCore.WebUtilities" />
     </ItemGroup>
 
     <ItemGroup>

src/Networking/NexusMods.Networking.NexusWebApi.NMA/OAuth.cs

@@ -1,8 +1,8 @@
 using System.Reactive.Linq;
 using System.Security.Cryptography;
 using System.Text;
-using System.Text.Encodings.Web;
 using System.Text.Json;
+using Microsoft.AspNetCore.WebUtilities;
 using Microsoft.Extensions.Logging;
 using NexusMods.Common;
 using NexusMods.Common.OSInterop;
@@ -132,8 +132,7 @@ private IInterprocessJob CreateJob(Uri url)
 
     internal static Uri GenerateAuthorizeUrl(string challenge, string state)
     {
-        // TODO: switch to Microsoft.AspNetCore.WebUtilities when .NET 8 is available
-        var request = new Dictionary<string, string>
+        var request = new Dictionary<string, string?>
         {
             { "response_type", "code" },
             { "scope", "openid profile email" },
@@ -144,28 +143,6 @@ internal static Uri GenerateAuthorizeUrl(string challenge, string state)
             { "state", state },
         };
 
-        return new Uri($"{OAuthUrl}/authorize{CreateQueryString(request)}");
-    }
-
-    private static string CreateQueryString(Dictionary<string, string> parameters)
-    {
-        var builder = new StringBuilder();
-        var first = true;
-        foreach (var pair in parameters)
-        {
-            var (key, value) = pair;
-
-            builder.Append(first ? '?' : '&');
-            builder.Append(UrlEncoder.Default.Encode(key));
-            builder.Append('=');
-            if (!string.IsNullOrEmpty(value))
-            {
-                builder.Append(UrlEncoder.Default.Encode(value));
-            }
-
-            first = false;
-        }
-
-        return builder.ToString();
+        return new Uri(QueryHelpers.AddQueryString($"{OAuthUrl}/authorize", request));
     }
 }

src/NexusMods.Common/ProtocolRegistration/ProtocolRegistrationLinux.cs

@@ -57,6 +57,7 @@ public ProtocolRegistrationLinux(IProcessFactory processFactory, IFileSystem fil
         sb.AppendLine($"Path={workingDirectory}");
         sb.AppendLine($"Exec={commandLine}");
         sb.AppendLine($"MimeType=x-scheme-handler/{protocol}");
+        sb.AppendLine("NoDisplay=true");
 
         await desktopEntryFile.WriteAllTextAsync(sb.ToString());