Cleanup code and update packages (#591)

shibayan · May 13, 2023 · f885190 · f885190
1 parent 552e761
commit f885190
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 15 deletions.
diff --git a/KeyVault.Acmebot/Internal/AccountKey.cs b/KeyVault.Acmebot/Internal/AccountKey.cs
@@ -15,7 +15,7 @@ public IJwsTool GenerateSigner()
         {
             var tool = new ACMESharp.Crypto.JOSE.Impl.ESJwsTool
             {
-                HashSize = int.Parse(KeyType.Substring(2))
+                HashSize = int.Parse(KeyType[2..])
             };
             tool.Init();
             tool.Import(KeyExport);
@@ -26,7 +26,7 @@ public IJwsTool GenerateSigner()
         {
             var tool = new ACMESharp.Crypto.JOSE.Impl.RSJwsTool
             {
-                KeySize = int.Parse(KeyType.Substring(2))
+                KeySize = int.Parse(KeyType[2..])
             };
             tool.Init();
             tool.Import(KeyExport);

diff --git a/KeyVault.Acmebot/Internal/AppRoleExtensions.cs b/KeyVault.Acmebot/Internal/AppRoleExtensions.cs
@@ -11,12 +11,7 @@ internal static class AppRoleExtensions
 
     private static bool IsAppRoleRequired => bool.TryParse(Environment.GetEnvironmentVariable("Acmebot:AppRoleRequired"), out var result) && result;
 
-    private static bool IsInAppRole(this ClaimsPrincipal claimsPrincipal, string role)
-    {
-        var roles = claimsPrincipal.Claims.Where(x => x.Type == "roles").Select(x => x.Value);
-
-        return roles.Contains(role);
-    }
+    private static bool IsInAppRole(this ClaimsPrincipal claimsPrincipal, string role) => claimsPrincipal.Claims.Where(x => x.Type == "roles").Select(x => x.Value).Contains(role);
 
     public static bool HasIssueCertificateRole(this ClaimsPrincipal claimsPrincipal) => !IsAppRoleRequired || claimsPrincipal.IsInAppRole(IssueCertificateAppRole);
 

diff --git a/KeyVault.Acmebot/Internal/ApplicationVersionInitializer.cs b/KeyVault.Acmebot/Internal/ApplicationVersionInitializer.cs
@@ -16,8 +16,5 @@ public ApplicationVersionInitializer()
 
     public string ApplicationVersion { get; }
 
-    public void Initialize(ITelemetry telemetry)
-    {
-        telemetry.Context.Component.Version = ApplicationVersion;
-    }
+    public void Initialize(ITelemetry telemetry) => telemetry.Context.Component.Version = ApplicationVersion;
 }
diff --git a/KeyVault.Acmebot/KeyVault.Acmebot.csproj b/KeyVault.Acmebot/KeyVault.Acmebot.csproj
@@ -4,8 +4,8 @@
     <AzureFunctionsVersion>v4</AzureFunctionsVersion>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="AWSSDK.Route53" Version="3.7.104.61" />
-    <PackageReference Include="Azure.Identity" Version="1.8.2" />
+    <PackageReference Include="AWSSDK.Route53" Version="3.7.104.68" />
+    <PackageReference Include="Azure.Identity" Version="1.9.0" />
     <PackageReference Include="Azure.ResourceManager.Dns" Version="1.0.1" />
     <PackageReference Include="Azure.ResourceManager.PrivateDns" Version="1.0.1" />
     <PackageReference Include="Azure.Security.KeyVault.Certificates" Version="4.5.1" />

diff --git a/KeyVault.Acmebot/Providers/GoDaddyProvider.cs b/KeyVault.Acmebot/Providers/GoDaddyProvider.cs
@@ -81,7 +81,8 @@ public GoDaddyClient(string apiKey, string apiSecret)
 
         public async Task<IReadOnlyList<ZoneDomain>> ListZonesAsync()
         {
-            var limit = 100;
+            const int limit = 100;
+
             var marker = "";
             var allActiveDomains = new List<ZoneDomain>();