Merge remote-tracking branch 'refs/remotes/origin/feature/service-logs-ui' into feature/service-logs-ui

Author: meenu155

bin/lib/logs.js

@@ -2,6 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { spawn } from 'node:child_process';
 import chalk from 'chalk';
+import _ from 'lodash';
 
 // Log levels and their colors
 const LOG_LEVELS = {
@@ -121,7 +122,8 @@ async function readLogsFromFile(filePath, options = {}) {
   }
 
   if (options.filter) {
-    const regex = new RegExp(options.filter, 'i');
+    const safeFilter = _.escapeRegExp(options.filter);
+    const regex = new RegExp(safeFilter, 'i');
     logs = logs.filter(log => regex.test(log.message) || regex.test(log.raw));
   }
 

package.json

@@ -53,7 +53,8 @@
     "degit": "^2.8.4",
     "fs-extra": "^11.3.2",
     "prompts": "^2.4.2",
-    "execa": "^9.6.0"
+    "execa": "^9.6.0",
+    "lodash": "^4.17.21"
   },
   "devDependencies": {
     "vitepress": "^1.3.3",

templates/node/package.json

@@ -6,6 +6,7 @@
     "dev": "node src/index.js"
   },
   "dependencies": {
-    "express": "^4.19.2"
+    "express": "^4.19.2",
+    "express-rate-limit": "^8.2.1"
   }
 }

templates/node/src/index.js

@@ -1,6 +1,7 @@
 import express from 'express';
 import fs from 'fs';
 import path from 'path';
+import rateLimit from 'express-rate-limit';
 
 // Initialize logger
 let logger;
@@ -25,6 +26,13 @@ function log(level, message, data = {}) {
 const app = express();
 const port = process.env.PORT || 3001;
 
+// Set up rate limiter for the /logs endpoint (max 5 requests per minute)
+const logsLimiter = rateLimit({
+  windowMs: 60 * 1000, // 1 minute
+  max: 5,
+  message: { error: 'Too many requests, please try again later.' }
+});
+
 // Middleware for request logging
 app.use((req, res, next) => {
   log('info', `${req.method} ${req.path}`, { 
@@ -41,7 +49,7 @@ app.get('/health', (_req, res) => {
   res.json({ status: 'ok', service: 'node' });
 });
 
-app.get('/logs', (req, res) => {
+app.get('/logs', logsLimiter, (req, res) => {
   try {
     const { tail = 50, level, since } = req.query;