Search query must query also the upn as mail could be null

When checking for transitive membership, search query must also check the upn as user could have an empty or different email.

Author: eduardpaul

src/services/PeopleSearchService.ts

@@ -135,7 +135,7 @@ export default class SPPeopleSearchService {
 
       // Check if users need to be searched in a specific Microsoft 365 Group, Security Group (incl. nested groups) or Distribution List
       else if (groupId && typeof (groupId) === 'string') {
-        const graphUserRequestUrl = `/groups/${groupId}/transitiveMembers?$count=true&$search="displayName:${query}" OR "mail:${query}"`;
+        const graphUserRequestUrl = `/groups/${groupId}/transitiveMembers?$count=true&$search="userPrincipalName:${query}" OR "displayName:${query}" OR "mail:${query}" `;
         const graphClient = await this.context.msGraphClientFactory.getClient("3");
         const graphUserResponse = await graphClient.api(graphUserRequestUrl).header('ConsistencyLevel', 'eventual').get();