Merge pull request #19 from carltibule/feature/RegisterNewUsersLocally
Modified register user service method to be async
This commit is contained in:
@ -27,7 +27,7 @@ namespace YABA.API.Controllers
|
||||
[ProducesResponseType(typeof(UserResponse), (int)HttpStatusCode.OK)]
|
||||
[ProducesResponseType((int)HttpStatusCode.NotFound)]
|
||||
[ProducesResponseType((int)HttpStatusCode.NoContent)]
|
||||
public IActionResult Register()
|
||||
public async Task<IActionResult> Register()
|
||||
{
|
||||
var authProviderId = this.GetAuthProviderId();
|
||||
|
||||
@ -37,7 +37,7 @@ namespace YABA.API.Controllers
|
||||
|
||||
if (isRegistered) return NoContent();
|
||||
|
||||
var registedUser = _userService.RegisterUser(authProviderId);
|
||||
var registedUser = await _userService.RegisterUser(authProviderId);
|
||||
return Ok(_mapper.Map<UserResponse>(registedUser));
|
||||
}
|
||||
}
|
||||
|
||||
@ -19,8 +19,6 @@ namespace YABA.API.Middlewares
|
||||
{
|
||||
if (httpContext.User != null && httpContext.User.Identity.IsAuthenticated)
|
||||
{
|
||||
var claims = new List<Claim>();
|
||||
|
||||
var userAuthProviderId = httpContext.User.Identity.GetAuthProviderId();
|
||||
|
||||
if (!string.IsNullOrEmpty(userAuthProviderId))
|
||||
|
||||
@ -1,11 +1,12 @@
|
||||
using YABA.Common.DTOs;
|
||||
using System.Threading.Tasks;
|
||||
using YABA.Common.DTOs;
|
||||
|
||||
namespace YABA.Service.Interfaces
|
||||
{
|
||||
public interface IUserService
|
||||
{
|
||||
public bool IsUserRegistered(string authProviderId);
|
||||
public UserDTO RegisterUser(string authProviderId);
|
||||
public int GetUserId(string authProviderId);
|
||||
bool IsUserRegistered(string authProviderId);
|
||||
Task<UserDTO> RegisterUser(string authProviderId);
|
||||
int GetUserId(string authProviderId);
|
||||
}
|
||||
}
|
||||
|
||||
@ -1,5 +1,6 @@
|
||||
using AutoMapper;
|
||||
using System.Linq;
|
||||
using System.Threading.Tasks;
|
||||
using YABA.Common.DTOs;
|
||||
using YABA.Data.Context;
|
||||
using YABA.Models;
|
||||
@ -25,7 +26,7 @@ namespace YABA.Service
|
||||
return _roContext.Users.Any(x => x.Auth0Id == authProviderId);
|
||||
}
|
||||
|
||||
public UserDTO RegisterUser(string authProviderId)
|
||||
public async Task<UserDTO> RegisterUser(string authProviderId)
|
||||
{
|
||||
if(IsUserRegistered(authProviderId))
|
||||
{
|
||||
@ -39,7 +40,7 @@ namespace YABA.Service
|
||||
};
|
||||
|
||||
var registedUser = _context.Users.Add(userToRegister);
|
||||
return _context.SaveChanges() > 0 ? _mapper.Map<UserDTO>(registedUser.Entity) : null;
|
||||
return await _context.SaveChangesAsync() > 0 ? _mapper.Map<UserDTO>(registedUser.Entity) : null;
|
||||
}
|
||||
|
||||
public int GetUserId(string authProviderId)
|
||||
|
||||
Reference in New Issue
Block a user