Skip to content

Commit

Permalink
chore: adds condition to compare principal id
Browse files Browse the repository at this point in the history
  • Loading branch information
Wanis Fahmy committed Jan 8, 2025
1 parent b9736bf commit 4eb4cc7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion internal/provider/resource_aws_account.go
Original file line number Diff line number Diff line change
Expand Up @@ -452,8 +452,9 @@ func updateAccountAssignment(ctx context.Context, d *schema.ResourceData, ssoadm
sso := d.Get("sso").([]interface{})[0].(map[string]interface{})
instanceArn := sso["instance_arn"].(string)
oldPrincipalId := oldSSOMap["principal_id"].(string)
newPrincipalId := newSSOMap["principal_id"].(string)

if oldEmail != newEmail && oldPrincipalId != "" && instanceArn != "" {
if oldEmail != newEmail && oldPrincipalId != newPrincipalId && oldPrincipalId != "" && instanceArn != "" {

_, err := ssoadmincon.DeleteAccountAssignment(ctx, &ssoadmin.DeleteAccountAssignmentInput{
InstanceArn: &instanceArn,
Expand Down

0 comments on commit 4eb4cc7

Please sign in to comment.