Fixed a bug when updating key operations on managed HSM (Azure#16789)

BethanyZhou · web-flow · commit 6d890d15d755 · 2022-01-07T16:47:24.000+08:00
* Fixed a bug when update key operations on managed HSM

* Update ChangeLog.md
diff --git a/src/KeyVault/KeyVault/ChangeLog.md b/src/KeyVault/KeyVault/ChangeLog.md
@@ -18,6 +18,7 @@
         - Additional information about change #1
 -->
 ## Upcoming Release
+* Fixed a bug when updating key operations on managed HSM [#16774]
 * Fixed the issue when importing no-password certificate [#16742]
 
 ## Version 4.2.0
diff --git a/src/KeyVault/KeyVault/Track2Models/Track2HsmClient.cs b/src/KeyVault/KeyVault/Track2Models/Track2HsmClient.cs
@@ -309,8 +309,7 @@ private PSKeyVaultKey UpdateKey(KeyClient client, string keyName, string keyVers
 
             try
             {
-                keyBundle = client.UpdateKeyPropertiesAsync(keyProperties, keyAttributes.KeyOps?.Cast<KeyOperation>().ToList())
-                    .GetAwaiter().GetResult();
+                keyBundle = client.UpdateKeyProperties(keyProperties, keyAttributes.KeyOps?.Select(op => new KeyOperation(op)));
             }
             catch (Exception ex)
             {

Original file line number	Diff line number	Diff line change
`@@ -309,8 +309,7 @@ private PSKeyVaultKey UpdateKey(KeyClient client, string keyName, string keyVers`
`309`	`309`
`310`	`310`	`try`
`311`	`311`	`{`
`312`		`- keyBundle = client.UpdateKeyPropertiesAsync(keyProperties, keyAttributes.KeyOps?.Cast<KeyOperation>().ToList())`
`313`		`- .GetAwaiter().GetResult();`
	`312`	`+ keyBundle = client.UpdateKeyProperties(keyProperties, keyAttributes.KeyOps?.Select(op => new KeyOperation(op)));`
`314`	`313`	`}`
`315`	`314`	`catch (Exception ex)`
`316`	`315`	`{`