chore: add performance logging and fix data controller bug

- Add console.time benchmarks for Redis JSON ops, API auth, Logger, and CORS

- Fix project ID reference in data.controller.js

Author: yash-pouranik

backend/app.js

@@ -30,12 +30,14 @@ if (process.env.NODE_ENV === 'development') {
 
 const adminCorsOptions = {
     origin: function (origin, callback) {
+        console.time("cors for admin chk")
         // Allow requests with no origin (like mobile apps or curl)
         if (!origin || adminWhitelist.indexOf(origin) !== -1) {
             callback(null, true);
         } else {
             callback(new Error('Not allowed by CORS for Admin access'));
         }
+        console.timeEnd("cors for admin chk")
     },
     credentials: true
 };

backend/controllers/data.controller.js

@@ -56,7 +56,7 @@ module.exports.insertData = async (req, res) => {
 
         if (!project.resources.db.isExternal) {
             await Project.updateOne(
-                { _id: cachedProject._id },
+                { _id: project._id },
                 { $inc: { databaseUsed: docSize } }
             );
         }
@@ -70,6 +70,7 @@ module.exports.insertData = async (req, res) => {
 // GET ALL DATA
 module.exports.getAllData = async (req, res) => {
     try {
+        console.time("getall")
         const { collectionName } = req.params;
         const project = req.project;
 
@@ -80,6 +81,7 @@ module.exports.getAllData = async (req, res) => {
         const Model = getCompiledModel(connection, collectionConfig, project._id, project.resources.db.isExternal);
 
         const data = await Model.find({}).limit(100).lean();
+        console.timeEnd("getall")
         res.json(data);
     } catch (err) {
         res.status(500).json({ error: err.message });

backend/middleware/api_usage.js

@@ -2,6 +2,7 @@ const rateLimit = require('express-rate-limit');
 const Log = require('../models/Log');
 
 // Rate Limiter 
+
 const limiter = rateLimit({
     windowMs: 15 * 60 * 1000,
     max: 100,
@@ -14,6 +15,7 @@ const limiter = rateLimit({
 
 // Logger 
 const logger = (req, res, next) => {
+    console.time("logger middleware")
     // Check for Data, Storage, AND UserAuth routes
     if (
         req.originalUrl.startsWith('/api/data') ||
@@ -38,6 +40,7 @@ const logger = (req, res, next) => {
             }
         });
     }
+    console.timeEnd("logger middleware")
     next();
 };
 

backend/middleware/verifyApiKey.js

@@ -7,16 +7,20 @@ const {
 
 module.exports = async (req, res, next) => {
     try {
+        console.time("api chk middleware")
         const apiKey = req.header('x-api-key');
         if (!apiKey) {
             return res.status(401).json({ error: 'API key not found' });
         }
 
         const hashedApi = hashApiKey(apiKey);
 
+        console.time("get project by api key cache")
         let project = await getProjectByApiKeyCache(hashedApi);
+        console.timeEnd("get project by api key cache")
 
         if (!project) {
+            console.time("get project by api key from db")
             project = await Project.findOne({ apiKey: hashedApi })
                 .select(`
                     owner
@@ -29,7 +33,7 @@ module.exports = async (req, res, next) => {
                 `)
                 .populate('owner', 'isVerified')
                 .lean();
-
+            console.timeEnd("get project by api key from db")
 
             if (!project) {
                 return res.status(401).json({
@@ -55,6 +59,7 @@ module.exports = async (req, res, next) => {
 
         req.project = project;
         req.hashedApiKey = hashedApi;
+        console.timeEnd("api chk middleware")
         next();
     } catch (err) {
         res.status(500).json({ error: err.message });

backend/services/redisCaching.js

@@ -2,9 +2,12 @@ const redis = require("../config/redis");
 
 async function setProjectByApiKeyCache(api, project) {
     try {
+        console.time("json");
+        const data = JSON.stringify(project);
+        console.timeEnd("json");
         await redis.set(
             `project:apikey:${api}`,
-            JSON.stringify(project),
+            data,
             'EX',
             60 * 60 * 2
         );
@@ -17,7 +20,10 @@ async function getProjectByApiKeyCache(api) {
     try {
         const data = await redis.get(`project:apikey:${api}`);
         if (!data) return null;
-        return JSON.parse(data);
+        console.time("json parse")
+        const parsedData = JSON.parse(data)
+        console.timeEnd("json parse")
+        return parsedData;
     } catch (err) {
         console.log(err);
         return null;