Fixes crash when no certificate_permissions are defined

[civascu]

Fixes the terraform crash when no certificate_permissions have been defined (e.g. when migrating an older config which did not support the certificate_permissions key)

Also fixes a bug where the Permissions.Keys was referenced as the source of the Certificates list

[tombuildsstuff]

Hey @civascu

Thanks for this PR - I've taken a look and this LGTM :)

[tombuildsstuff]

Tests pass:

$ acctests azurerm TestAccAzureRMKeyVault
=== RUN   TestAccAzureRMKeyVaultKey_importBasicEC
--- PASS: TestAccAzureRMKeyVaultKey_importBasicEC (74.86s)
=== RUN   TestAccAzureRMKeyVaultKey_importBasicRSA
--- PASS: TestAccAzureRMKeyVaultKey_importBasicRSA (76.85s)
=== RUN   TestAccAzureRMKeyVaultKey_importBasicRSAHSM
--- PASS: TestAccAzureRMKeyVaultKey_importBasicRSAHSM (82.93s)
=== RUN   TestAccAzureRMKeyVaultKey_importComplete
--- PASS: TestAccAzureRMKeyVaultKey_importComplete (78.65s)
=== RUN   TestAccAzureRMKeyVaultSecret_importBasic
--- PASS: TestAccAzureRMKeyVaultSecret_importBasic (78.65s)
=== RUN   TestAccAzureRMKeyVaultSecret_importComplete
--- PASS: TestAccAzureRMKeyVaultSecret_importComplete (76.35s)
=== RUN   TestAccAzureRMKeyVault_importBasic
--- PASS: TestAccAzureRMKeyVault_importBasic (70.74s)
=== RUN   TestAccAzureRMKeyVaultKey_basicEC
--- PASS: TestAccAzureRMKeyVaultKey_basicEC (76.65s)
=== RUN   TestAccAzureRMKeyVaultKey_basicRSA
--- PASS: TestAccAzureRMKeyVaultKey_basicRSA (78.75s)
=== RUN   TestAccAzureRMKeyVaultKey_basicRSAHSM
--- PASS: TestAccAzureRMKeyVaultKey_basicRSAHSM (81.41s)
=== RUN   TestAccAzureRMKeyVaultKey_complete
--- PASS: TestAccAzureRMKeyVaultKey_complete (77.42s)
=== RUN   TestAccAzureRMKeyVaultKey_update
--- PASS: TestAccAzureRMKeyVaultKey_update (91.55s)
=== RUN   TestAccAzureRMKeyVaultChild_validateName
--- PASS: TestAccAzureRMKeyVaultChild_validateName (0.00s)
=== RUN   TestAccAzureRMKeyVaultSecret_parseID
--- PASS: TestAccAzureRMKeyVaultSecret_parseID (0.00s)
=== RUN   TestAccAzureRMKeyVaultSecret_basic
--- PASS: TestAccAzureRMKeyVaultSecret_basic (76.80s)
=== RUN   TestAccAzureRMKeyVaultSecret_complete
--- PASS: TestAccAzureRMKeyVaultSecret_complete (78.34s)
=== RUN   TestAccAzureRMKeyVaultSecret_update
--- PASS: TestAccAzureRMKeyVaultSecret_update (89.70s)
=== RUN   TestAccAzureRMKeyVault_name
--- PASS: TestAccAzureRMKeyVault_name (0.00s)
=== RUN   TestAccAzureRMKeyVault_basic
--- PASS: TestAccAzureRMKeyVault_basic (70.96s)
=== RUN   TestAccAzureRMKeyVault_complete
--- PASS: TestAccAzureRMKeyVault_complete (74.22s)
=== RUN   TestAccAzureRMKeyVault_update
--- PASS: TestAccAzureRMKeyVault_update (83.38s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	1418.245s

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!