Skip to content
Snippets Groups Projects
Commit f859a92d authored by Benedikt Heinrichs's avatar Benedikt Heinrichs
Browse files

Merge branch 'Issue/2127-userEmailsDisplayed' into 'dev'

Update: Exposing user email addresses

See merge request !62
parents 07bb0713 3cc8367f
No related branches found
No related tags found
2 merge requests!63Release: Sprint/2022 15 :robot:,!62Update: Exposing user email addresses
......@@ -93,8 +93,7 @@ namespace Coscine.Api.User.Controllers
(dbUser.DisplayName.ToLower().Contains(lowerQueryString)
|| dbUser.EmailAddress.ToLower().Equals(lowerQueryString)))
.Take(10)
// Email set to null so that not every email address gets exposed
.Select((dbUser) => new UserObject(dbUser.Id, dbUser.DisplayName, dbUser.Givenname, dbUser.Surname, null,
.Select((dbUser) => new UserObject(dbUser.Id, dbUser.DisplayName, dbUser.Givenname, dbUser.Surname, dbUser.EmailAddress,
((projectRoles.Any((projectRole) => projectRole.UserId == dbUser.Id)))) // return whether the user has any role in the project
));
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment