fix bug to delete secrets when removing edge device

CGI-FR · Jun 5, 2023 · 02e6dd4 · 02e6dd4
1 parent c98a70e
commit 02e6dd4
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/AzureIoTHub.Portal.Infrastructure/Services/AwsExternalDeviceService.cs b/src/AzureIoTHub.Portal.Infrastructure/Services/AwsExternalDeviceService.cs
@@ -447,6 +447,7 @@ public async Task RemoveDeviceCredentials(IoTEdgeDevice device)
             while (true);
         }
 
+
         private async Task RemoveGreengrassCertificateFromPrincipal(IoTEdgeDevice device, string principalId)
         {
             _ = await this.amazonIoTClient.DetachPolicyAsync(new DetachPolicyRequest
@@ -465,19 +466,19 @@ private async Task RemoveGreengrassCertificateFromPrincipal(IoTEdgeDevice device
             _ = await this.amazonSecretsManager.DeleteSecretAsync(new DeleteSecretRequest
             {
                 ForceDeleteWithoutRecovery = true,
-                SecretId = device.DeviceId + PublicKeyKey,
+                SecretId = device.DeviceName + PublicKeyKey,
             });
 
             _ = await this.amazonSecretsManager.DeleteSecretAsync(new DeleteSecretRequest
             {
                 ForceDeleteWithoutRecovery = true,
-                SecretId = device.DeviceId + PrivateKeyKey,
+                SecretId = device.DeviceName + PrivateKeyKey,
             });
 
             _ = await this.amazonSecretsManager.DeleteSecretAsync(new DeleteSecretRequest
             {
                 ForceDeleteWithoutRecovery = true,
-                SecretId = device.DeviceId + CertificateKey,
+                SecretId = device.DeviceName + CertificateKey,
             });
 
             var awsPricipalCertRegex = new Regex("/arn:aws:iot:([a-z0-9-]*):(\\d*):cert\\/([0-9a-fA-F]*)/gm");