vanta-api 1.3.0 → 1.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vanta-api",
-  "version": "1.3.0",
+  "version": "1.3.2",
   "description": "Advanced API features and security configuration for Node.js/MongoDB.",
   "main": "index.js",
   "scripts": {
@@ -19,7 +19,7 @@
   "author": "Alireza Aghaee",
   "license": "MIT",
   "dependencies": {
-    "bson": "^6.10.3",
+    "bson": "^7.0.0",
     "mongoose": "^7.0.0",
     "pluralize": "^8.0.0",
     "winston": "^3.0.0"

package/src/api-features.js

@@ -18,8 +18,11 @@ export class ApiFeatures {
   constructor(model, query = {}, userRole = "") {
     this.model = model;
     this.query = { ...query };
-    if (!userRole || ! Object.keys(securityConfig.accessLevels).includes(userRole)) {
-      this.userRole = 'guest';
+    if (
+      !userRole ||
+      !Object.keys(securityConfig.accessLevels).includes(userRole)
+    ) {
+      this.userRole = "guest";
     } else {
       this.userRole = userRole;
     }
@@ -66,46 +69,43 @@ export class ApiFeatures {
     return this;
   }
 
- limitFields(input = "") {
-  const rawFields = [input, this.query.fields].filter(Boolean).join(",");
-  if (!rawFields) return this;
+  limitFields(input = "") {
+    const rawFields = [input, this.query.fields].filter(Boolean).join(",");
+    if (!rawFields) return this;
 
-  const validFields = Object.keys(this.model.schema.paths).filter(
-    (f) => !securityConfig.forbiddenFields.includes(f)
-  );
-
-  const fieldsArray = rawFields
-    .split(",")
-    .map((f) => f.trim())
-    .filter(Boolean);
-
-  const includeFields = new Set();
-  const excludeFields = new Set();
+    const validFields = Object.keys(this.model.schema.paths).filter(
+      (f) => !securityConfig.forbiddenFields.includes(f)
+    );
 
-  fieldsArray.forEach((f) => {
-    if (f.startsWith("-")) excludeFields.add(f.slice(1));
-    else includeFields.add(f);
-  });
+    const fieldsArray = rawFields
+      .split(",")
+      .map((f) => f.trim())
+      .filter(Boolean);
 
-  const project = {};
+    const includeFields = new Set();
+    const excludeFields = new Set();
 
-  if (includeFields.size > 0) {
-    includeFields.forEach((f) => {
-      if (validFields.includes(f)) project[f] = 1;
-    });
-  } else if (excludeFields.size > 0) {
-    validFields.forEach((f) => {
-      if (!excludeFields.has(f)) project[f] = 1;
+    fieldsArray.forEach((f) => {
+      if (f.startsWith("-")) excludeFields.add(f.slice(1));
+      else includeFields.add(f);
     });
-  }
-
-  if (Object.keys(project).length) {
-    this.pipeline.push({ $project: project });
-  }
+    const project = {};
+    if (includeFields.size > 0) {
+      includeFields.forEach((f) => {
+        if (validFields.includes(f)) project[f] = 1;
+      });
+    } else if (excludeFields.size > 0) {
+      validFields.forEach((f) => {
+        if (!excludeFields.has(f)) project[f] = 1;
+      });
+    }
 
-  return this;
-}
+    if (Object.keys(project).length) {
+      this.pipeline.push({ $project: project });
+    }
 
+    return this;
+  }
   paginate() {
     const { maxLimit } = securityConfig.accessLevels[this.userRole] || {
       maxLimit: 100,
@@ -252,7 +252,6 @@ export class ApiFeatures {
 
   _parseQueryFilters() {
     const obj = { ...this.query };
-    // پاک کردن پارامترهای سیستماتیک
     ["page", "limit", "sort", "fields", "populate"].forEach(
       (k) => delete obj[k]
     );
@@ -320,9 +319,14 @@ export class ApiFeatures {
       }
 
       if (typeof val === "string" && /^[0-9]+$/.test(val)) {
-        resultObj[keyObj] = parseInt(val, 10);
+        if (val.length > 1 && val.startsWith("0")) {
+          resultObj[keyObj] = val; // keep leading zero
+        } else {
+          resultObj[keyObj] = parseInt(val, 10);
+        }
         return;
       }
+
       if (typeof val === "string" && this.#isStrictObjectId(val)) {
         resultObj[keyObj] = new ObjectId(val);
         return;
@@ -343,9 +347,6 @@ export class ApiFeatures {
   _applySecurityFilters(filters) {
     let res = { ...filters };
     securityConfig.forbiddenFields.forEach((f) => delete res[f]);
-    if (this.userRole !== "admin" && this.model.schema.path("isActive")) {
-      res.isActive = true;
-    }
     return res;
   }