Skip to content

Commit 422058c

Browse files
author
Megha Narayanan
committed
rate limiting comment
1 parent e5d1da0 commit 422058c

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

packages/cli/src/commands/sandbox/sandbox-devtools/sandbox_devtools_command.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,8 @@ export class SandboxDevToolsCommand implements CommandModule<object> {
125125
app.use(express.static(publicPath));
126126

127127
// Apply rate limiting to all routes
128+
// DevTools runs locally on developers' machines, and developers are the sole users, so rate limiting is not strictly necessary
129+
// Included to satisfy CodeQL rule on rate limiting: https://codeql.github.com/codeql-query-help/javascript/js-missing-rate-limiting/
128130
const limiter = rateLimit({
129131
windowMs: 15 * 60 * 1000, // 15 minutes
130132
max: 100, // Limit each IP to 100 requests per windowMs

0 commit comments

Comments
 (0)