Merge pull request #25 from ConductorOne/bt/fix_counts

btipling · web-flow · commit 1a3b00f5c652 · 2025-05-06T17:05:03.000-07:00
document that I made this change
diff --git a/pkg/connector/server_role.go b/pkg/connector/server_role.go
@@ -204,6 +204,7 @@ func (d *serverRolePrincipalSyncer) Grant(ctx context.Context, resource *v2.Reso
 
 	// database-role:baton_test:6:member
 	splitId := strings.Split(entitlement.Id, ":")
+	// id length is 3
 	if len(splitId) != 3 {
 		return nil, nil, fmt.Errorf("unexpected entitlement id: %s", entitlement.Id)
 	}

Original file line number	Diff line number	Diff line change
`@@ -204,6 +204,7 @@ func (d serverRolePrincipalSyncer) Grant(ctx context.Context, resource v2.Reso`
`204`	`204`
`205`	`205`	`// database-role:baton_test:6:member`
`206`	`206`	`splitId := strings.Split(entitlement.Id, ":")`
	`207`	`+ // id length is 3`
`207`	`208`	`if len(splitId) != 3 {`
`208`	`209`	`return nil, nil, fmt.Errorf("unexpected entitlement id: %s", entitlement.Id)`
`209`	`210`	`}`