terraform-azurerm-caf icon indicating copy to clipboard operation
terraform-azurerm-caf copied to clipboard

Fix logic for MSSQL Server AzureAD administrator

Open a-ferri opened this issue 2 years ago • 0 comments

Issue-id

PR Checklist


  • [ ] I have added example(s) inside the [./examples/] folder
  • [ ] I have added the example(s) to the integration test list for normal (~30 minutes) or long runner >30 minutes
  • [x] I have checked the coding conventions as per the wiki
  • [x] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

The current implementation is creating dynamic configuration based on a optional attribute (doc). This PR allow assigning AzureAD administrator to a SQL Server without enforcing usage of "Active Directory Only" users.

Does this introduce a breaking change

  • [ ] YES
  • [x] NO

Testing

a-ferri avatar Jul 19 '22 15:07 a-ferri