Release .Net SDK v16.6.7 (#647)

* KSM-550 Stop generating UIDs that start with "-" (#646) * KSM-557 Added new and updated PAM field types (#652) * KSM-557 Added new and updated PAM field types * removed ReSharper suppressions * Updated quick test Program.cs
Keeper-Security · Sep 18, 2024 · c94b526 · c94b526
1 parent 0e0896e
commit c94b526
Show file tree

Hide file tree

Showing 9 changed files with 167 additions and 316 deletions.
diff --git a/sdk/dotNet/QuickTest/Program.cs b/sdk/dotNet/QuickTest/Program.cs
@@ -21,8 +21,8 @@ private static void Main()
         private static async Task GetSecrets()
         {
             var storage = new LocalConfigStorage("test_config_storage.json");
-            // ReSharper disable once StringLiteralTypo
-            SecretsManagerClient.InitializeStorage(storage, "US:ONE_TIME_TOKEN");
+            // Initialize Storage only once - generates config.json from one-time token
+            // SecretsManagerClient.InitializeStorage(storage, "US:ONE_TIME_TOKEN");
             var options = new SecretsManagerOptions(storage);
             // var options = new SecretsManagerOptions(storage, SecretsManagerClient.CachingPostFunction);
             var secrets = await SecretsManagerClient.GetSecrets(options);

diff --git a/sdk/dotNet/README.md b/sdk/dotNet/README.md
@@ -3,6 +3,11 @@
 
 # Change Log
 
+## 16.6.7
+
+* KSM-550 - Stop generating UIDs that start with "-"
+* KSM-557 - Added new and updated PAM field types
+
 ## 16.6.6
 
 * KSM-360 - GHA to build and release strong named assemblies

diff --git a/sdk/dotNet/SecretManagement.Keeper/src/SecretManagement.cs b/sdk/dotNet/SecretManagement.Keeper/src/SecretManagement.cs
@@ -46,8 +46,6 @@ public static async Task<KeeperResult> GetVaultConfigFromConfigString(string con
             return KeeperResult.Ok(storage.AsHashTable());
         }
 
-        [SuppressMessage("ReSharper", "UnusedAutoPropertyAccessor.Global")]
-        [SuppressMessage("ReSharper", "MemberCanBePrivate.Global")]
         public class KeeperResult
         {
             private object result;

diff --git a/sdk/dotNet/SecretsManager.Test.Core/SecretsManagerClient.Test.cs b/sdk/dotNet/SecretsManager.Test.Core/SecretsManagerClient.Test.cs
@@ -8,8 +8,6 @@ namespace SecretsManager.Test
 {
     using QueryFunction = Func<string, TransmissionKey, EncryptedPayload, Task<KeeperHttpResponse>>;
 
-    [SuppressMessage("ReSharper", "InconsistentNaming")]
-    [SuppressMessage("ReSharper", "UnusedAutoPropertyAccessor.Global")]
     public class TestResponse
     {
         public string transmissionKey { get; set; }

diff --git a/sdk/dotNet/SecretsManager/CryptoUtils.cs b/sdk/dotNet/SecretsManager/CryptoUtils.cs
@@ -80,6 +80,19 @@ public static byte[] GetRandomBytes(int length)
             return bytes;
         }
 
+        public static byte[] GetUidBytes()
+        {
+            byte dash = 0b1111_1000;
+            var bytes = new byte[] { };
+            for (int i = 0; i < 8; i++) {
+                bytes = GetRandomBytes(16);
+                if ((dash & bytes[0]) != dash) break;
+            }
+            if ((dash & bytes[0]) == dash)
+                bytes[0] = (byte)(bytes[0] & (byte)0b0111_1111);
+            return bytes;
+        }
+
         public static byte[] GenerateKeyPair()
         {
             var keyGenerator = new ECKeyPairGenerator();

diff --git a/sdk/dotNet/SecretsManager/JsonUtils.cs b/sdk/dotNet/SecretsManager/JsonUtils.cs
@@ -1,4 +1,3 @@
-using System.Text.Encodings.Web;
 using System.Text.Json;
 using System.Text.Json.Serialization;