fixed methods error response

This commit is contained in:
Pau 2025-04-02 19:07:54 +02:00
parent 4ebc73031e
commit 9ab2f61b8c
4 changed files with 13 additions and 21 deletions

View File

@ -2,8 +2,8 @@ import { getUserListModel } from "../../models/authorization/userModel.js";
import { noResults } from "../../validators/result-validators.js"; import { noResults } from "../../validators/result-validators.js";
import mysql from "../../adapters/mysql.js"; import mysql from "../../adapters/mysql.js";
import bcrypt from "bcrypt"; import bcrypt from "bcrypt";
import { error404, errorHandler } from "../../utils/errors.js"; import { errorHandler } from "../../utils/errors.js";
import { sendResponseNotFound, sendResponseUnauthorized } from "../../utils/responses.js"; import { sendResponseUnauthorized } from "../../utils/responses.js";
import { getRolesHasPermissionsModel } from "../../models/authorization/roles_has_permissionsModel.js"; import { getRolesHasPermissionsModel } from "../../models/authorization/roles_has_permissionsModel.js";
const postLoginController = (req, res, next, config) => { const postLoginController = (req, res, next, config) => {
@ -46,7 +46,6 @@ const postLoginController = (req, res, next, config) => {
role_permissions: rolePermissions role_permissions: rolePermissions
}, },
}; };
next(result); next(result);
}); });
}); });
@ -60,25 +59,18 @@ const postLoginController = (req, res, next, config) => {
}); });
} }
const _getRolePermissions = (config, roleUuid, conn) => { const _getRolePermissions = (config, roleName, conn) => {
return getRolesHasPermissionsModel({ roleName: roleName, conn })
return getRolesHasPermissionsModel({ uuid_role: roleUuid, conn })
.then((response) => { .then((response) => {
if (noResults(response)) { if (noResults(response)) {
const err = error404(); return [] //if user has no role assigned/no permissions assigned to role, return empty array to allow log in
const error = errorHandler(err, config.environment);
return sendResponseNotFound(res, error);
} }
return response.map(({ permission }) => permission); return response
}) })
.catch((err) => { .catch((err) => {
const error = errorHandler(err, config.environment); return errorHandler(err, config.environment);
res.status(error.code).json(error);
}) })
.finally(() => {
mysql.end(conn);
});
} }
export { export {

View File

@ -57,7 +57,7 @@ const getPermissionByUuidController = (req, res, next, config) => {
}) })
.catch((err) => { .catch((err) => {
const error = errorHandler(err, config.environment) const error = errorHandler(err, config.environment)
res.status(error.code).json(error) return res.status(error.code).json(error)
}) })
.finally(() => { .finally(() => {
mysql.end(conn) mysql.end(conn)
@ -79,7 +79,7 @@ const postPermissionController = (req, res, next, config) => {
}) })
.catch((err) => { .catch((err) => {
const error = errorHandler(err, config.environment) const error = errorHandler(err, config.environment)
res.status(error.code).json(error) return res.status(error.code).json(error)
}) })
.finally(() => { .finally(() => {
mysql.end(conn) mysql.end(conn)

View File

@ -5,7 +5,7 @@ import {
countEndpointsQuery, countEndpointsQuery,
getEndpointsQuery, getEndpointsQuery,
insertEndpointsQuery, insertEndpointsQuery,
modifyEndpointsQuery, updateEndpointsQuery,
deleteEndpointsQuery, deleteEndpointsQuery,
softDeleteEndpointsQuery softDeleteEndpointsQuery
} from '../../repositories/authorization/endpointsRepository.js' } from '../../repositories/authorization/endpointsRepository.js'
@ -16,7 +16,7 @@ const getEndpointsModel = ({conn, ...rest}) => {
return mysql return mysql
.execute(getEndpointsQuery(paramsToSearch), conn, paramsToSearch) .execute(getEndpointsQuery(paramsToSearch), conn, paramsToSearch)
.then(queryResult => queryResult.map(({id, created, deleted, createdBy, deletedBy, ...resultFiltered}) => resultFiltered)) .then(queryResult => queryResult.map(({created, deleted, createdBy, deletedBy, ...resultFiltered}) => resultFiltered))
} }
const countEndpointsModel = ({conn, ...rest}) => { const countEndpointsModel = ({conn, ...rest}) => {
@ -42,7 +42,7 @@ const insertEndpointsModel = ({conn, ...params}) => {
const modifyEndpointsModel = ({conn, ...params}) => { const modifyEndpointsModel = ({conn, ...params}) => {
return mysql return mysql
.execute(modifyEndpointsQuery(params), conn, params) .execute(updateEndpointsQuery(params), conn, params)
.then(queryResult => queryResult[1].map(({id, created, deleted, createdBy, deletedBy, ...resultFiltered}) => resultFiltered)) .then(queryResult => queryResult[1].map(({id, created, deleted, createdBy, deletedBy, ...resultFiltered}) => resultFiltered))
} }

View File

@ -36,7 +36,7 @@ const insertRolesHasPermissionsModel = ({conn, ...rest}) => {
.then(results => results[1].map(({id, uuid, fk_role, fk_permission, created, deleted, createdBy, deletedBy, ...rest}) => ({...rest}))) .then(results => results[1].map(({id, uuid, fk_role, fk_permission, created, deleted, createdBy, deletedBy, ...rest}) => ({...rest})))
} }
const softDeleteRolesHasPermissionsModel = ({conn, ...rest}) => { const softDeleteRolesHasPermissionsModel = ({conn, deleted, deletedBy, ...rest}) => {
const deletedData = deleted ? dayjs.utc(deleted).format('YYYY-MM-DD HH:mm:ss') : dayjs.utc().format('YYYY-MM-DD HH:mm:ss') const deletedData = deleted ? dayjs.utc(deleted).format('YYYY-MM-DD HH:mm:ss') : dayjs.utc().format('YYYY-MM-DD HH:mm:ss')
const params = { ...rest, deleted: deletedData } const params = { ...rest, deleted: deletedData }