Skip to content

Commit

Permalink
Also consider the opaque id when searching for users
Browse files Browse the repository at this point in the history
  • Loading branch information
aduffeck committed Mar 11, 2021
1 parent 565c668 commit 28063fd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/user/manager/demo/demo.go
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,9 @@ func extractClaim(u *userpb.User, claim string) (string, error) {
return "", errors.New("demo: invalid field")
}

// TODO(jfd) search Opaque? compare sub?
// TODO(jfd) compare sub?
func userContains(u *userpb.User, query string) bool {
return strings.Contains(u.Username, query) || strings.Contains(u.DisplayName, query) || strings.Contains(u.Mail, query)
return strings.Contains(u.Username, query) || strings.Contains(u.DisplayName, query) || strings.Contains(u.Mail, query) || strings.Contains(u.Id.OpaqueId, query)
}

func (m *manager) FindUsers(ctx context.Context, query string) ([]*userpb.User, error) {
Expand Down

0 comments on commit 28063fd

Please sign in to comment.