diff --git a/backend/src/Squidex.Domain.Users/DefaultUserResolver.cs b/backend/src/Squidex.Domain.Users/DefaultUserResolver.cs index 4e32d317a..97357cc60 100644 --- a/backend/src/Squidex.Domain.Users/DefaultUserResolver.cs +++ b/backend/src/Squidex.Domain.Users/DefaultUserResolver.cs @@ -29,6 +29,8 @@ namespace Squidex.Domain.Users public async Task CreateUserIfNotExists(string email, bool invited) { + Guard.NotNullOrEmpty(email); + using (var scope = serviceProvider.CreateScope()) { var userFactory = scope.ServiceProvider.GetRequiredService(); @@ -58,6 +60,8 @@ namespace Squidex.Domain.Users public async Task FindByIdOrEmailAsync(string idOrEmail) { + Guard.NotNullOrEmpty(idOrEmail); + using (var scope = serviceProvider.CreateScope()) { var userFactory = scope.ServiceProvider.GetRequiredService(); @@ -76,6 +80,8 @@ namespace Squidex.Domain.Users public async Task> QueryByEmailAsync(string email) { + Guard.NotNullOrEmpty(email); + using (var scope = serviceProvider.CreateScope()) { var userManager = scope.ServiceProvider.GetRequiredService>(); @@ -88,9 +94,14 @@ namespace Squidex.Domain.Users public async Task> QueryManyAsync(string[] ids) { + Guard.NotNull(ids); + using (var scope = serviceProvider.CreateScope()) { var userManager = scope.ServiceProvider.GetRequiredService>(); + var userFactory = scope.ServiceProvider.GetRequiredService(); + + ids = ids.Where(x => userFactory.IsId(x)).ToArray(); var result = await userManager.QueryByIdsAync(ids);