diff --git a/internal/services/applications/application_resource.go b/internal/services/applications/application_resource.go index 2a40846e7f..fbb6732702 100644 --- a/internal/services/applications/application_resource.go +++ b/internal/services/applications/application_resource.go @@ -918,6 +918,13 @@ func applicationResourceCreate(ctx context.Context, d *schema.ResourceData, meta if callerObject == nil { return tf.ErrorDiagF(errors.New("returned callerObject was nil"), "Could not retrieve calling principal object %q", callerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if callerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve calling principal object %q", callerId) + //} + callerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, callerId))) + ownersFirst20 := msgraph.Owners{*callerObject} var ownersExtra msgraph.Owners @@ -927,21 +934,25 @@ func applicationResourceCreate(ctx context.Context, d *schema.ResourceData, meta // Retrieve and set the initial owners, which can be up to 20 in total when creating the application if v, ok := d.GetOk("owners"); ok { ownerCount := 0 - for _, id := range v.(*schema.Set).List() { - if strings.EqualFold(id.(string), callerId) { + for _, ownerId := range v.(*schema.Set).List() { + if strings.EqualFold(ownerId.(string), callerId) { removeCallerOwner = false continue } - ownerObject, _, err := directoryObjectsClient.Get(ctx, id.(string), odata.Query{}) + ownerObject, _, err := directoryObjectsClient.Get(ctx, ownerId.(string), odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", id) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if ownerObject == nil { - return tf.ErrorDiagF(errors.New("ownerObject was nil"), "Could not retrieve owner principal object %q", id) - } - if ownerObject.ODataId == nil { - return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", id) + return tf.ErrorDiagF(errors.New("ownerObject was nil"), "Could not retrieve owner principal object %q", ownerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", ownerId) + //} + ownerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, ownerId))) + if ownerCount < 19 { ownersFirst20 = append(ownersFirst20, *ownerObject) } else { @@ -1077,14 +1088,21 @@ func applicationResourceUpdate(ctx context.Context, d *schema.ResourceData, meta if len(ownersToAdd) > 0 { newOwners := make(msgraph.Owners, 0) - for _, m := range ownersToAdd { - ownerObject, _, err := directoryObjectsClient.Get(ctx, m, odata.Query{}) + for _, ownerId := range ownersToAdd { + ownerObject, _, err := directoryObjectsClient.Get(ctx, ownerId, odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", m) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if ownerObject == nil { - return tf.ErrorDiagF(errors.New("returned ownerObject was nil"), "Could not retrieve owner principal object %q", m) + return tf.ErrorDiagF(errors.New("returned ownerObject was nil"), "Could not retrieve owner principal object %q", ownerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", ownerId) + //} + ownerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, ownerId))) + newOwners = append(newOwners, *ownerObject) } diff --git a/internal/services/directoryroles/directory_role_member_resource.go b/internal/services/directoryroles/directory_role_member_resource.go index 9a25feca6c..b4204cc08e 100644 --- a/internal/services/directoryroles/directory_role_member_resource.go +++ b/internal/services/directoryroles/directory_role_member_resource.go @@ -17,6 +17,7 @@ import ( "github.com/hashicorp/terraform-provider-azuread/internal/clients" "github.com/hashicorp/terraform-provider-azuread/internal/services/directoryroles/parse" "github.com/hashicorp/terraform-provider-azuread/internal/tf" + "github.com/hashicorp/terraform-provider-azuread/internal/utils" "github.com/hashicorp/terraform-provider-azuread/internal/validate" ) @@ -90,9 +91,13 @@ func directoryRoleMemberResourceCreate(ctx context.Context, d *schema.ResourceDa if memberObject == nil { return tf.ErrorDiagF(errors.New("returned memberObject was nil"), "Could not retrieve member principal object %q", id.MemberId) } - if memberObject.ODataId == nil { - return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve member principal object %q", id.MemberId) - } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if memberObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve member principal object %q", id.MemberId) + //} + memberObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, id.MemberId))) + role.Members = &msgraph.Members{*memberObject} if _, err := client.AddMembers(ctx, role); err != nil { diff --git a/internal/services/groups/group_member_resource.go b/internal/services/groups/group_member_resource.go index 7093c7b5c4..04a1d9eaec 100644 --- a/internal/services/groups/group_member_resource.go +++ b/internal/services/groups/group_member_resource.go @@ -3,6 +3,7 @@ package groups import ( "context" "errors" + "fmt" "log" "net/http" "strings" @@ -16,6 +17,7 @@ import ( "github.com/hashicorp/terraform-provider-azuread/internal/clients" "github.com/hashicorp/terraform-provider-azuread/internal/services/groups/parse" "github.com/hashicorp/terraform-provider-azuread/internal/tf" + "github.com/hashicorp/terraform-provider-azuread/internal/utils" "github.com/hashicorp/terraform-provider-azuread/internal/validate" ) @@ -95,9 +97,13 @@ func groupMemberResourceCreate(ctx context.Context, d *schema.ResourceData, meta if memberObject == nil { return tf.ErrorDiagF(errors.New("returned memberObject was nil"), "Could not retrieve member principal object %q", memberId) } - if memberObject.ODataId == nil { - return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve member principal object %q", memberId) - } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if memberObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve member principal object %q", memberId) + //} + memberObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, memberId))) + group.Members = &msgraph.Members{*memberObject} if _, err := client.AddMembers(ctx, group); err != nil { diff --git a/internal/services/groups/group_resource.go b/internal/services/groups/group_resource.go index 6092943506..f7e7978cfd 100644 --- a/internal/services/groups/group_resource.go +++ b/internal/services/groups/group_resource.go @@ -418,9 +418,13 @@ func groupResourceCreate(ctx context.Context, d *schema.ResourceData, meta inter if ownerObject.ID == nil { return nil, errors.New("ownerObject ID was nil") } - if ownerObject.ODataId == nil { - return nil, errors.New("ODataId was nil") - } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return nil, errors.New("ODataId was nil") + //} + ownerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, id))) + if ownerObject.ODataType == nil { return nil, errors.New("ownerObject ODataType was nil") } @@ -436,15 +440,12 @@ func groupResourceCreate(ctx context.Context, d *schema.ResourceData, meta inter // First look for the calling principal in the specified owners; it should always be included in the initial // owners to avoid orphaning a group when the caller doesn't have the Groups.ReadWrite.All scope. - for _, id := range owners { - ownerObject, err := getOwnerObject(ctx, id.(string)) + for _, ownerId := range owners { + ownerObject, err := getOwnerObject(ctx, ownerId.(string)) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", id) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if strings.EqualFold(*ownerObject.ID, callerId) { - if ownerObject.ODataId == nil { - return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", id) - } if ownerCount < 20 { ownersFirst20 = append(ownersFirst20, *ownerObject) } else { @@ -456,10 +457,10 @@ func groupResourceCreate(ctx context.Context, d *schema.ResourceData, meta inter // Then look for users, and finally service principals for _, t := range []odata.Type{odata.TypeUser, odata.TypeServicePrincipal} { - for _, id := range owners { - ownerObject, err := getOwnerObject(ctx, id.(string)) + for _, ownerId := range owners { + ownerObject, err := getOwnerObject(ctx, ownerId.(string)) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", id) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if *ownerObject.ODataType == t && !strings.EqualFold(*ownerObject.ID, callerId) { if ownerCount < 20 { @@ -508,17 +509,21 @@ func groupResourceCreate(ctx context.Context, d *schema.ResourceData, meta inter // Add members after the group is created members := make(msgraph.Members, 0) if v, ok := d.GetOk("members"); ok { - for _, id := range v.(*schema.Set).List() { - memberObject, _, err := directoryObjectsClient.Get(ctx, id.(string), odata.Query{}) + for _, memberId := range v.(*schema.Set).List() { + memberObject, _, err := directoryObjectsClient.Get(ctx, memberId.(string), odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve member principal object %q", id) + return tf.ErrorDiagF(err, "Could not retrieve member principal object %q", memberId) } if memberObject == nil { - return tf.ErrorDiagF(errors.New("memberObject was nil"), "Could not retrieve member principal object %q", id) - } - if memberObject.ODataId == nil { - return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve member principal object %q", id) + return tf.ErrorDiagF(errors.New("memberObject was nil"), "Could not retrieve member principal object %q", memberId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if memberObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve member principal object %q", memberId) + //} + memberObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, memberId))) + members = append(members, *memberObject) } } @@ -603,14 +608,21 @@ func groupResourceUpdate(ctx context.Context, d *schema.ResourceData, meta inter if len(membersToAdd) > 0 { newMembers := make(msgraph.Members, 0) - for _, m := range membersToAdd { - memberObject, _, err := directoryObjectsClient.Get(ctx, m, odata.Query{}) + for _, memberId := range membersToAdd { + memberObject, _, err := directoryObjectsClient.Get(ctx, memberId, odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve principal object %q", m) + return tf.ErrorDiagF(err, "Could not retrieve principal object %q", memberId) } if memberObject == nil { - return tf.ErrorDiagF(errors.New("returned memberObject was nil"), "Could not retrieve member principal object %q", m) + return tf.ErrorDiagF(errors.New("returned memberObject was nil"), "Could not retrieve member principal object %q", memberId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", memberId) + //} + memberObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, memberId))) + newMembers = append(newMembers, *memberObject) } @@ -641,14 +653,21 @@ func groupResourceUpdate(ctx context.Context, d *schema.ResourceData, meta inter if len(ownersToAdd) > 0 { newOwners := make(msgraph.Owners, 0) - for _, m := range ownersToAdd { - ownerObject, _, err := directoryObjectsClient.Get(ctx, m, odata.Query{}) + for _, ownerId := range ownersToAdd { + ownerObject, _, err := directoryObjectsClient.Get(ctx, ownerId, odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", m) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if ownerObject == nil { - return tf.ErrorDiagF(errors.New("returned ownerObject was nil"), "Could not retrieve owner principal object %q", m) + return tf.ErrorDiagF(errors.New("returned ownerObject was nil"), "Could not retrieve owner principal object %q", ownerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", ownerId) + //} + ownerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, ownerId))) + newOwners = append(newOwners, *ownerObject) } diff --git a/internal/services/serviceprincipals/service_principal_resource.go b/internal/services/serviceprincipals/service_principal_resource.go index d5bf9eb33e..ef7ae84b77 100644 --- a/internal/services/serviceprincipals/service_principal_resource.go +++ b/internal/services/serviceprincipals/service_principal_resource.go @@ -376,6 +376,13 @@ func servicePrincipalResourceCreate(ctx context.Context, d *schema.ResourceData, if callerObject == nil { return tf.ErrorDiagF(errors.New("returned callerObject was nil"), "Could not retrieve calling principal object %q", callerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if callerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve calling principal object %q", callerId) + //} + callerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, callerId))) + ownersFirst20 := msgraph.Owners{*callerObject} var ownersExtra msgraph.Owners @@ -385,21 +392,25 @@ func servicePrincipalResourceCreate(ctx context.Context, d *schema.ResourceData, // Retrieve and set the initial owners, which can be up to 20 in total when creating the service principal if v, ok := d.GetOk("owners"); ok { ownerCount := 0 - for _, id := range v.(*schema.Set).List() { - if strings.EqualFold(id.(string), callerId) { + for _, ownerId := range v.(*schema.Set).List() { + if strings.EqualFold(ownerId.(string), callerId) { removeCallerOwner = false continue } - ownerObject, _, err := directoryObjectsClient.Get(ctx, id.(string), odata.Query{}) + ownerObject, _, err := directoryObjectsClient.Get(ctx, ownerId.(string), odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", id) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if ownerObject == nil { - return tf.ErrorDiagF(errors.New("ownerObject was nil"), "Could not retrieve owner principal object %q", id) - } - if ownerObject.ODataId == nil { - return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", id) + return tf.ErrorDiagF(errors.New("ownerObject was nil"), "Could not retrieve owner principal object %q", ownerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", ownerId) + //} + ownerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, ownerId))) + if ownerCount < 19 { ownersFirst20 = append(ownersFirst20, *ownerObject) } else { @@ -485,14 +496,20 @@ func servicePrincipalResourceUpdate(ctx context.Context, d *schema.ResourceData, if len(ownersToAdd) > 0 { newOwners := make(msgraph.Owners, 0) - for _, m := range ownersToAdd { - ownerObject, _, err := directoryObjectsClient.Get(ctx, m, odata.Query{}) + for _, ownerId := range ownersToAdd { + ownerObject, _, err := directoryObjectsClient.Get(ctx, ownerId, odata.Query{}) if err != nil { - return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", m) + return tf.ErrorDiagF(err, "Could not retrieve owner principal object %q", ownerId) } if ownerObject == nil { - return tf.ErrorDiagF(errors.New("returned ownerObject was nil"), "Could not retrieve owner principal object %q", m) + return tf.ErrorDiagF(errors.New("returned ownerObject was nil"), "Could not retrieve owner principal object %q", ownerId) } + // TODO: remove this workaround for https://github.com/hashicorp/terraform-provider-azuread/issues/588 + //if ownerObject.ODataId == nil { + // return tf.ErrorDiagF(errors.New("ODataId was nil"), "Could not retrieve owner principal object %q", ownerId) + //} + ownerObject.ODataId = (*odata.Id)(utils.String(fmt.Sprintf("%s/v1.0/%s/directoryObjects/%s", + client.BaseClient.Endpoint, client.BaseClient.TenantId, ownerId))) newOwners = append(newOwners, *ownerObject) }