Skip to content

Commit 4ec3f8f

Browse files
Merge pull request #28 from StreetSupport/feature/3011-build-user-management-interface
3011 - Fixed bugs
2 parents 657dd02 + 0560093 commit 4ec3f8f

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

src/controllers/userController.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ const getUsers = asyncHandler(async (req: Request, res: Response) => {
4646
}
4747

4848
// Exclude VolunteerAdmin users from results if requesting user is not a SuperAdmin or VolunteerAdmin
49-
if (!requestingUserAuthClaims.includes(ROLES.SUPER_ADMIN) && !requestingUserAuthClaims.includes(ROLES.VOLUNTEER_ADMIN)) {
49+
if (!requestingUserAuthClaims.includes(ROLES.SUPER_ADMIN)) {
5050
conditions.push({ AuthClaims: { $ne: ROLES.VOLUNTEER_ADMIN } });
5151
}
5252

src/middleware/authMiddleware.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,11 @@ export const authenticate = asyncHandler(async (req: Request, res: Response, nex
203203
return sendUnauthorized(res, 'User not found');
204204
}
205205

206+
// Check if user is active (not blocked/deactivated)
207+
if (user.IsActive === false) {
208+
return sendUnauthorized(res, 'User account is deactivated');
209+
}
210+
206211
req.user = user;
207212
next();
208213
});

0 commit comments

Comments
 (0)