feat(security): improve app security

Vasitos · Oct 16, 2023 · 3019765 · 3019765
1 parent 559328e
commit 3019765
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/database/repositories/serverProperties/index.js b/src/database/repositories/serverProperties/index.js
@@ -2,15 +2,15 @@ const ServerProperties = require('../../schemas/serverProperties');
 const ServerPropertiesCategory = require('../../schemas/serverPropertiesCategory');
 
 async function findCategoryByKey(key) {
-    return ServerPropertiesCategory.findOne({ key });
+    return await ServerPropertiesCategory.findOne({ key });
 }
 
 async function findPropertyByKeyAndCategory(key, category) {
-    return ServerProperties.findOne({ key, category });
+    return await ServerProperties.findOne({ key, category });
 }
 
 async function findPropertyByKey(key) {
-    return ServerProperties.findOne({ key });
+    return await ServerProperties.findOne({ key });
 }
 
 
@@ -157,5 +157,6 @@ module.exports = {
     findCategoryByKey,
     findPropertyByKeyAndCategory,
     createProperty,
-    getCategoryById
+    getCategoryById,
+    findPropertyByKey
 };
diff --git a/src/middlewares/checkUniqueKeyServerPropertiesCategory.js b/src/middlewares/checkUniqueKeyServerPropertiesCategory.js
@@ -1,8 +1,7 @@
-const ServerPropertiesCategory = require('../database/schemas/serverPropertiesCategory');
-
+const repository = require("../database/repositories/serverProperties/index")
 const checkUniqueKeyServerPropertiesCategory = async (req, res, next) => {
     try {
-        const existingCategory = await ServerPropertiesCategory.findOne({ key: req.body.key });
+        const existingCategory = await repository.findCategoryByKey(req.body.key);
         if (existingCategory) {
             return res.status(400).json({ error: 'Category with this key already exists' });
         }