Skip to content

Commit

Permalink
Fix comments: azuread_authentication_only instead of `azuread_only_…
Browse files Browse the repository at this point in the history
…authentication`
  • Loading branch information
Aris van Ommeren committed Oct 15, 2021
1 parent 2cb7c1a commit 6870e6b
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions internal/services/mssql/mssql_server_resource.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ func resourceMsSqlServer() *pluginsdk.Resource {
ValidateFunc: validation.IsUUID,
},

"azuread_only_authentication": {
"azuread_authentication_only": {
Type: pluginsdk.TypeBool,
Optional: true,
Computed: true,
Expand Down Expand Up @@ -533,7 +533,7 @@ func expandMsSqlServerAADOnlyAuthentictions(input []interface{}) bool {
return false
}
admin := input[0].(map[string]interface{})
if v, ok := admin["azuread_only_authentication"]; ok && v != nil {
if v, ok := admin["azuread_authentication_only"]; ok && v != nil {
return v.(bool)
}
return false
Expand Down Expand Up @@ -587,7 +587,7 @@ func flatternMsSqlServerAdministrator(admin sql.ServerAzureADAdministrator) []in
"login_username": login,
"object_id": sid,
"tenant_id": tid,
"azuread_only_authentication": aadOnlyAuthentictionsEnabled,
"azuread_authentication_only": aadOnlyAuthentictionsEnabled,
},
}
}
Expand Down
2 changes: 1 addition & 1 deletion internal/services/mssql/mssql_server_resource_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -620,7 +620,7 @@ resource "azurerm_mssql_server" "test" {
azuread_administrator {
login_username = "AzureAD Admin2"
object_id = data.azuread_service_principal.test.id
azuread_only_authentication = true
azuread_authentication_only = true
}
}
`, data.RandomInteger, data.Locations.Primary, os.Getenv("ARM_CLIENT_ID"))
Expand Down
2 changes: 1 addition & 1 deletion website/docs/r/mssql_server.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ An `azuread_administrator` block supports the following:

* `tenant_id` - (Optional) The tenant id of the Azure AD Administrator of this SQL Server.

* `azuread_only_authentication` - (Optional) Specifies whether only AD Users and administrators (like `azuread_administrator.0.login_username`) can be used to login or also local database users (like `administrator_login`).
* `azuread_authentication_only` - (Optional) Specifies whether only AD Users and administrators (like `azuread_administrator.0.login_username`) can be used to login or also local database users (like `administrator_login`).

---

Expand Down

0 comments on commit 6870e6b

Please sign in to comment.