aboutsummaryrefslogtreecommitdiff
path: root/src/DevHive.Services
diff options
context:
space:
mode:
Diffstat (limited to 'src/DevHive.Services')
-rw-r--r--src/DevHive.Services/Configurations/Mapping/RoleMapings.cs1
-rw-r--r--src/DevHive.Services/Interfaces/IRoleService.cs2
-rw-r--r--src/DevHive.Services/Models/Identity/Role/CreateRoleServiceModel.cs4
-rw-r--r--src/DevHive.Services/Services/RoleService.cs3
-rw-r--r--src/DevHive.Services/Services/UserService.cs2
5 files changed, 5 insertions, 7 deletions
diff --git a/src/DevHive.Services/Configurations/Mapping/RoleMapings.cs b/src/DevHive.Services/Configurations/Mapping/RoleMapings.cs
index d6c8511..5f9452f 100644
--- a/src/DevHive.Services/Configurations/Mapping/RoleMapings.cs
+++ b/src/DevHive.Services/Configurations/Mapping/RoleMapings.cs
@@ -8,6 +8,7 @@ namespace DevHive.Services.Configurations.Mapping
{
public RoleMappings()
{
+ CreateMap<CreateRoleServiceModel, Role>();
CreateMap<RoleServiceModel, Role>();
CreateMap<UpdateRoleServiceModel, Role>();
diff --git a/src/DevHive.Services/Interfaces/IRoleService.cs b/src/DevHive.Services/Interfaces/IRoleService.cs
index fd661be..3a498d2 100644
--- a/src/DevHive.Services/Interfaces/IRoleService.cs
+++ b/src/DevHive.Services/Interfaces/IRoleService.cs
@@ -6,7 +6,7 @@ namespace DevHive.Services.Interfaces
{
public interface IRoleService
{
- Task<Guid> CreateRole(RoleServiceModel roleServiceModel);
+ Task<Guid> CreateRole(CreateRoleServiceModel roleServiceModel);
Task<RoleServiceModel> GetRoleById(Guid id);
diff --git a/src/DevHive.Services/Models/Identity/Role/CreateRoleServiceModel.cs b/src/DevHive.Services/Models/Identity/Role/CreateRoleServiceModel.cs
index 53bea9e..3bed3fd 100644
--- a/src/DevHive.Services/Models/Identity/Role/CreateRoleServiceModel.cs
+++ b/src/DevHive.Services/Models/Identity/Role/CreateRoleServiceModel.cs
@@ -5,10 +5,6 @@ namespace DevHive.Services.Models.Identity.Role
{
public class CreateRoleServiceModel
{
- [NotNull]
- [Required]
- [MinLength(3)]
- [MaxLength(50)]
public string Name { get; set; }
}
}
diff --git a/src/DevHive.Services/Services/RoleService.cs b/src/DevHive.Services/Services/RoleService.cs
index 896946d..91a8c73 100644
--- a/src/DevHive.Services/Services/RoleService.cs
+++ b/src/DevHive.Services/Services/RoleService.cs
@@ -20,12 +20,11 @@ namespace DevHive.Services.Services
this._roleMapper = mapper;
}
- public async Task<Guid> CreateRole(RoleServiceModel roleServiceModel)
+ public async Task<Guid> CreateRole(CreateRoleServiceModel roleServiceModel)
{
if (await this._roleRepository.DoesNameExist(roleServiceModel.Name))
throw new ArgumentException("Role already exists!");
-
Role role = this._roleMapper.Map<Role>(roleServiceModel);
bool success = await this._roleRepository.AddAsync(role);
diff --git a/src/DevHive.Services/Services/UserService.cs b/src/DevHive.Services/Services/UserService.cs
index 533f422..cf33644 100644
--- a/src/DevHive.Services/Services/UserService.cs
+++ b/src/DevHive.Services/Services/UserService.cs
@@ -194,6 +194,8 @@ namespace DevHive.Services.Services
return false;
/* Check roles */
+ if(jwtRoleNames.Contains(Role.AdminRole))
+ return true;
// Check if jwt contains all user roles (if it doesn't, jwt is either old or tampered with)
foreach (var role in user.Roles)