eslint-plugin-secure-coding 1.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/AGENTS.md +196 -0
- package/CHANGELOG.md +105 -0
- package/LICENSE +23 -0
- package/README.md +377 -0
- package/package.json +80 -0
- package/src/index.d.ts +32 -0
- package/src/index.js +345 -0
- package/src/index.js.map +1 -0
- package/src/rules/security/database-injection.d.ts +13 -0
- package/src/rules/security/database-injection.js +407 -0
- package/src/rules/security/database-injection.js.map +1 -0
- package/src/rules/security/detect-child-process.d.ts +11 -0
- package/src/rules/security/detect-child-process.js +460 -0
- package/src/rules/security/detect-child-process.js.map +1 -0
- package/src/rules/security/detect-eval-with-expression.d.ts +9 -0
- package/src/rules/security/detect-eval-with-expression.js +393 -0
- package/src/rules/security/detect-eval-with-expression.js.map +1 -0
- package/src/rules/security/detect-non-literal-fs-filename.d.ts +7 -0
- package/src/rules/security/detect-non-literal-fs-filename.js +322 -0
- package/src/rules/security/detect-non-literal-fs-filename.js.map +1 -0
- package/src/rules/security/detect-non-literal-regexp.d.ts +9 -0
- package/src/rules/security/detect-non-literal-regexp.js +387 -0
- package/src/rules/security/detect-non-literal-regexp.js.map +1 -0
- package/src/rules/security/detect-object-injection.d.ts +11 -0
- package/src/rules/security/detect-object-injection.js +411 -0
- package/src/rules/security/detect-object-injection.js.map +1 -0
- package/src/rules/security/no-buffer-overread.d.ts +14 -0
- package/src/rules/security/no-buffer-overread.js +519 -0
- package/src/rules/security/no-buffer-overread.js.map +1 -0
- package/src/rules/security/no-clickjacking.d.ts +10 -0
- package/src/rules/security/no-clickjacking.js +381 -0
- package/src/rules/security/no-clickjacking.js.map +1 -0
- package/src/rules/security/no-directive-injection.d.ts +12 -0
- package/src/rules/security/no-directive-injection.js +446 -0
- package/src/rules/security/no-directive-injection.js.map +1 -0
- package/src/rules/security/no-document-cookie.d.ts +5 -0
- package/src/rules/security/no-document-cookie.js +90 -0
- package/src/rules/security/no-document-cookie.js.map +1 -0
- package/src/rules/security/no-electron-security-issues.d.ts +10 -0
- package/src/rules/security/no-electron-security-issues.js +421 -0
- package/src/rules/security/no-electron-security-issues.js.map +1 -0
- package/src/rules/security/no-exposed-sensitive-data.d.ts +11 -0
- package/src/rules/security/no-exposed-sensitive-data.js +341 -0
- package/src/rules/security/no-exposed-sensitive-data.js.map +1 -0
- package/src/rules/security/no-format-string-injection.d.ts +17 -0
- package/src/rules/security/no-format-string-injection.js +653 -0
- package/src/rules/security/no-format-string-injection.js.map +1 -0
- package/src/rules/security/no-graphql-injection.d.ts +12 -0
- package/src/rules/security/no-graphql-injection.js +410 -0
- package/src/rules/security/no-graphql-injection.js.map +1 -0
- package/src/rules/security/no-hardcoded-credentials.d.ts +26 -0
- package/src/rules/security/no-hardcoded-credentials.js +377 -0
- package/src/rules/security/no-hardcoded-credentials.js.map +1 -0
- package/src/rules/security/no-improper-sanitization.d.ts +12 -0
- package/src/rules/security/no-improper-sanitization.js +408 -0
- package/src/rules/security/no-improper-sanitization.js.map +1 -0
- package/src/rules/security/no-improper-type-validation.d.ts +10 -0
- package/src/rules/security/no-improper-type-validation.js +420 -0
- package/src/rules/security/no-improper-type-validation.js.map +1 -0
- package/src/rules/security/no-insecure-comparison.d.ts +7 -0
- package/src/rules/security/no-insecure-comparison.js +125 -0
- package/src/rules/security/no-insecure-comparison.js.map +1 -0
- package/src/rules/security/no-insecure-cookie-settings.d.ts +9 -0
- package/src/rules/security/no-insecure-cookie-settings.js +305 -0
- package/src/rules/security/no-insecure-cookie-settings.js.map +1 -0
- package/src/rules/security/no-insecure-jwt.d.ts +10 -0
- package/src/rules/security/no-insecure-jwt.js +338 -0
- package/src/rules/security/no-insecure-jwt.js.map +1 -0
- package/src/rules/security/no-insecure-redirects.d.ts +7 -0
- package/src/rules/security/no-insecure-redirects.js +215 -0
- package/src/rules/security/no-insecure-redirects.js.map +1 -0
- package/src/rules/security/no-insufficient-postmessage-validation.d.ts +14 -0
- package/src/rules/security/no-insufficient-postmessage-validation.js +390 -0
- package/src/rules/security/no-insufficient-postmessage-validation.js.map +1 -0
- package/src/rules/security/no-insufficient-random.d.ts +9 -0
- package/src/rules/security/no-insufficient-random.js +207 -0
- package/src/rules/security/no-insufficient-random.js.map +1 -0
- package/src/rules/security/no-ldap-injection.d.ts +10 -0
- package/src/rules/security/no-ldap-injection.js +449 -0
- package/src/rules/security/no-ldap-injection.js.map +1 -0
- package/src/rules/security/no-missing-authentication.d.ts +13 -0
- package/src/rules/security/no-missing-authentication.js +322 -0
- package/src/rules/security/no-missing-authentication.js.map +1 -0
- package/src/rules/security/no-missing-cors-check.d.ts +9 -0
- package/src/rules/security/no-missing-cors-check.js +449 -0
- package/src/rules/security/no-missing-cors-check.js.map +1 -0
- package/src/rules/security/no-missing-csrf-protection.d.ts +11 -0
- package/src/rules/security/no-missing-csrf-protection.js +183 -0
- package/src/rules/security/no-missing-csrf-protection.js.map +1 -0
- package/src/rules/security/no-missing-security-headers.d.ts +7 -0
- package/src/rules/security/no-missing-security-headers.js +217 -0
- package/src/rules/security/no-missing-security-headers.js.map +1 -0
- package/src/rules/security/no-privilege-escalation.d.ts +13 -0
- package/src/rules/security/no-privilege-escalation.js +321 -0
- package/src/rules/security/no-privilege-escalation.js.map +1 -0
- package/src/rules/security/no-redos-vulnerable-regex.d.ts +7 -0
- package/src/rules/security/no-redos-vulnerable-regex.js +307 -0
- package/src/rules/security/no-redos-vulnerable-regex.js.map +1 -0
- package/src/rules/security/no-sensitive-data-exposure.d.ts +11 -0
- package/src/rules/security/no-sensitive-data-exposure.js +251 -0
- package/src/rules/security/no-sensitive-data-exposure.js.map +1 -0
- package/src/rules/security/no-sql-injection.d.ts +10 -0
- package/src/rules/security/no-sql-injection.js +332 -0
- package/src/rules/security/no-sql-injection.js.map +1 -0
- package/src/rules/security/no-timing-attack.d.ts +10 -0
- package/src/rules/security/no-timing-attack.js +358 -0
- package/src/rules/security/no-timing-attack.js.map +1 -0
- package/src/rules/security/no-toctou-vulnerability.d.ts +7 -0
- package/src/rules/security/no-toctou-vulnerability.js +165 -0
- package/src/rules/security/no-toctou-vulnerability.js.map +1 -0
- package/src/rules/security/no-unchecked-loop-condition.d.ts +12 -0
- package/src/rules/security/no-unchecked-loop-condition.js +635 -0
- package/src/rules/security/no-unchecked-loop-condition.js.map +1 -0
- package/src/rules/security/no-unencrypted-transmission.d.ts +11 -0
- package/src/rules/security/no-unencrypted-transmission.js +237 -0
- package/src/rules/security/no-unencrypted-transmission.js.map +1 -0
- package/src/rules/security/no-unescaped-url-parameter.d.ts +9 -0
- package/src/rules/security/no-unescaped-url-parameter.js +266 -0
- package/src/rules/security/no-unescaped-url-parameter.js.map +1 -0
- package/src/rules/security/no-unlimited-resource-allocation.d.ts +12 -0
- package/src/rules/security/no-unlimited-resource-allocation.js +659 -0
- package/src/rules/security/no-unlimited-resource-allocation.js.map +1 -0
- package/src/rules/security/no-unsafe-deserialization.d.ts +10 -0
- package/src/rules/security/no-unsafe-deserialization.js +501 -0
- package/src/rules/security/no-unsafe-deserialization.js.map +1 -0
- package/src/rules/security/no-unsafe-dynamic-require.d.ts +5 -0
- package/src/rules/security/no-unsafe-dynamic-require.js +107 -0
- package/src/rules/security/no-unsafe-dynamic-require.js.map +1 -0
- package/src/rules/security/no-unsafe-regex-construction.d.ts +9 -0
- package/src/rules/security/no-unsafe-regex-construction.js +292 -0
- package/src/rules/security/no-unsafe-regex-construction.js.map +1 -0
- package/src/rules/security/no-unsanitized-html.d.ts +9 -0
- package/src/rules/security/no-unsanitized-html.js +347 -0
- package/src/rules/security/no-unsanitized-html.js.map +1 -0
- package/src/rules/security/no-unvalidated-user-input.d.ts +9 -0
- package/src/rules/security/no-unvalidated-user-input.js +418 -0
- package/src/rules/security/no-unvalidated-user-input.js.map +1 -0
- package/src/rules/security/no-weak-crypto.d.ts +11 -0
- package/src/rules/security/no-weak-crypto.js +350 -0
- package/src/rules/security/no-weak-crypto.js.map +1 -0
- package/src/rules/security/no-weak-password-recovery.d.ts +12 -0
- package/src/rules/security/no-weak-password-recovery.js +401 -0
- package/src/rules/security/no-weak-password-recovery.js.map +1 -0
- package/src/rules/security/no-xpath-injection.d.ts +10 -0
- package/src/rules/security/no-xpath-injection.js +487 -0
- package/src/rules/security/no-xpath-injection.js.map +1 -0
- package/src/rules/security/no-xxe-injection.d.ts +7 -0
- package/src/rules/security/no-xxe-injection.js +270 -0
- package/src/rules/security/no-xxe-injection.js.map +1 -0
- package/src/rules/security/no-zip-slip.d.ts +9 -0
- package/src/rules/security/no-zip-slip.js +446 -0
- package/src/rules/security/no-zip-slip.js.map +1 -0
- package/src/types/index.d.ts +131 -0
- package/src/types/index.js +18 -0
- package/src/types/index.js.map +1 -0
|
@@ -0,0 +1,322 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.noMissingAuthentication = void 0;
|
|
4
|
+
const eslint_devkit_1 = require("@interlace/eslint-devkit");
|
|
5
|
+
const eslint_devkit_2 = require("@interlace/eslint-devkit");
|
|
6
|
+
/**
|
|
7
|
+
* Common authentication middleware patterns
|
|
8
|
+
*/
|
|
9
|
+
const DEFAULT_AUTH_MIDDLEWARE_PATTERNS = [
|
|
10
|
+
'authenticate',
|
|
11
|
+
'auth',
|
|
12
|
+
'requireAuth',
|
|
13
|
+
'isAuthenticated',
|
|
14
|
+
'verifyToken',
|
|
15
|
+
'checkAuth',
|
|
16
|
+
'ensureAuthenticated',
|
|
17
|
+
'passport.authenticate',
|
|
18
|
+
'jwt',
|
|
19
|
+
'session',
|
|
20
|
+
];
|
|
21
|
+
/**
|
|
22
|
+
* Common route handler patterns
|
|
23
|
+
*/
|
|
24
|
+
const DEFAULT_ROUTE_HANDLER_PATTERNS = [
|
|
25
|
+
'get',
|
|
26
|
+
'post',
|
|
27
|
+
'put',
|
|
28
|
+
'delete',
|
|
29
|
+
'patch',
|
|
30
|
+
'all',
|
|
31
|
+
'use',
|
|
32
|
+
];
|
|
33
|
+
/**
|
|
34
|
+
* Check if a node is inside an authentication middleware call
|
|
35
|
+
*/
|
|
36
|
+
function isInsideAuthMiddleware(node, sourceCode, authPatterns) {
|
|
37
|
+
let current = node;
|
|
38
|
+
while (current) {
|
|
39
|
+
// Check if current is an argument to a CallExpression
|
|
40
|
+
if (current.parent && current.parent.type === 'CallExpression') {
|
|
41
|
+
const callExpr = current.parent;
|
|
42
|
+
// Verify that current is actually an argument of this call
|
|
43
|
+
const isArgument = callExpr.arguments.some((arg) => arg === current);
|
|
44
|
+
if (!isArgument) {
|
|
45
|
+
// Not an argument, continue traversing
|
|
46
|
+
if ('parent' in current && current.parent) {
|
|
47
|
+
current = current.parent;
|
|
48
|
+
continue;
|
|
49
|
+
}
|
|
50
|
+
else {
|
|
51
|
+
break;
|
|
52
|
+
}
|
|
53
|
+
}
|
|
54
|
+
const callee = callExpr.callee;
|
|
55
|
+
// Check if it's an authentication middleware call
|
|
56
|
+
if (callee.type === 'Identifier') {
|
|
57
|
+
const calleeName = callee.name.toLowerCase();
|
|
58
|
+
if (authPatterns.some(pattern => calleeName.includes(pattern.toLowerCase()))) {
|
|
59
|
+
return true;
|
|
60
|
+
}
|
|
61
|
+
}
|
|
62
|
+
// Check if it's a member expression like app.use(auth())
|
|
63
|
+
if (callee.type === 'MemberExpression' && callee.property.type === 'Identifier') {
|
|
64
|
+
const propertyName = callee.property.name.toLowerCase();
|
|
65
|
+
if (propertyName === 'use' || propertyName === 'all') {
|
|
66
|
+
// Check if any argument is an auth middleware
|
|
67
|
+
for (const arg of callExpr.arguments) {
|
|
68
|
+
const argText = sourceCode.getText(arg);
|
|
69
|
+
if (authPatterns.some(pattern => argText.toLowerCase().includes(pattern.toLowerCase()))) {
|
|
70
|
+
return true;
|
|
71
|
+
}
|
|
72
|
+
}
|
|
73
|
+
}
|
|
74
|
+
}
|
|
75
|
+
}
|
|
76
|
+
// Traverse up the AST
|
|
77
|
+
if ('parent' in current && current.parent) {
|
|
78
|
+
current = current.parent;
|
|
79
|
+
}
|
|
80
|
+
else {
|
|
81
|
+
break;
|
|
82
|
+
}
|
|
83
|
+
}
|
|
84
|
+
return false;
|
|
85
|
+
}
|
|
86
|
+
/**
|
|
87
|
+
* Check if a string matches any ignore pattern
|
|
88
|
+
*/
|
|
89
|
+
function matchesIgnorePattern(text, patterns) {
|
|
90
|
+
return patterns.some(pattern => {
|
|
91
|
+
try {
|
|
92
|
+
const regex = new RegExp(pattern, 'i');
|
|
93
|
+
return regex.test(text);
|
|
94
|
+
}
|
|
95
|
+
catch {
|
|
96
|
+
// Invalid regex - treat as literal string match
|
|
97
|
+
return text.toLowerCase().includes(pattern.toLowerCase());
|
|
98
|
+
}
|
|
99
|
+
});
|
|
100
|
+
}
|
|
101
|
+
exports.noMissingAuthentication = (0, eslint_devkit_2.createRule)({
|
|
102
|
+
name: 'no-missing-authentication',
|
|
103
|
+
meta: {
|
|
104
|
+
type: 'problem',
|
|
105
|
+
docs: {
|
|
106
|
+
description: 'Detects missing authentication checks in route handlers',
|
|
107
|
+
},
|
|
108
|
+
hasSuggestions: true,
|
|
109
|
+
messages: {
|
|
110
|
+
missingAuthentication: (0, eslint_devkit_1.formatLLMMessage)({
|
|
111
|
+
icon: eslint_devkit_1.MessageIcons.SECURITY,
|
|
112
|
+
issueName: 'Missing Authentication',
|
|
113
|
+
cwe: 'CWE-287',
|
|
114
|
+
description: 'Route handler missing authentication check: {{route}}',
|
|
115
|
+
severity: 'CRITICAL',
|
|
116
|
+
fix: 'Add authentication middleware: app.{{method}}(\'{{path}}\', authenticate(), handler)',
|
|
117
|
+
documentationLink: 'https://cwe.mitre.org/data/definitions/287.html',
|
|
118
|
+
}),
|
|
119
|
+
addAuthentication: (0, eslint_devkit_1.formatLLMMessage)({
|
|
120
|
+
icon: eslint_devkit_1.MessageIcons.INFO,
|
|
121
|
+
issueName: 'Add Authentication',
|
|
122
|
+
description: 'Add authentication middleware before route handler',
|
|
123
|
+
severity: 'LOW',
|
|
124
|
+
fix: 'app.get("/path", authenticate(), handler)',
|
|
125
|
+
documentationLink: 'https://cwe.mitre.org/data/definitions/287.html',
|
|
126
|
+
}),
|
|
127
|
+
},
|
|
128
|
+
schema: [
|
|
129
|
+
{
|
|
130
|
+
type: 'object',
|
|
131
|
+
properties: {
|
|
132
|
+
allowInTests: {
|
|
133
|
+
type: 'boolean',
|
|
134
|
+
default: false,
|
|
135
|
+
description: 'Allow missing authentication in test files',
|
|
136
|
+
},
|
|
137
|
+
authMiddlewarePatterns: {
|
|
138
|
+
type: 'array',
|
|
139
|
+
items: { type: 'string' },
|
|
140
|
+
default: DEFAULT_AUTH_MIDDLEWARE_PATTERNS,
|
|
141
|
+
description: 'Authentication middleware patterns to recognize',
|
|
142
|
+
},
|
|
143
|
+
routeHandlerPatterns: {
|
|
144
|
+
type: 'array',
|
|
145
|
+
items: { type: 'string' },
|
|
146
|
+
default: DEFAULT_ROUTE_HANDLER_PATTERNS,
|
|
147
|
+
description: 'Route handler patterns to check',
|
|
148
|
+
},
|
|
149
|
+
ignorePatterns: {
|
|
150
|
+
type: 'array',
|
|
151
|
+
items: { type: 'string' },
|
|
152
|
+
default: [],
|
|
153
|
+
description: 'Additional patterns to ignore',
|
|
154
|
+
},
|
|
155
|
+
testFilePattern: {
|
|
156
|
+
type: 'string',
|
|
157
|
+
default: '\\.(test|spec)\\.(ts|tsx|js|jsx)$',
|
|
158
|
+
description: 'Test file pattern regex string',
|
|
159
|
+
},
|
|
160
|
+
},
|
|
161
|
+
additionalProperties: false,
|
|
162
|
+
},
|
|
163
|
+
],
|
|
164
|
+
},
|
|
165
|
+
defaultOptions: [
|
|
166
|
+
{
|
|
167
|
+
allowInTests: false,
|
|
168
|
+
testFilePattern: '\\.(test|spec)\\.(ts|tsx|js|jsx)$',
|
|
169
|
+
authMiddlewarePatterns: DEFAULT_AUTH_MIDDLEWARE_PATTERNS,
|
|
170
|
+
routeHandlerPatterns: DEFAULT_ROUTE_HANDLER_PATTERNS,
|
|
171
|
+
ignorePatterns: [],
|
|
172
|
+
},
|
|
173
|
+
],
|
|
174
|
+
create(context, [options = {}]) {
|
|
175
|
+
const { allowInTests = false, testFilePattern = '\\.(test|spec)\\.(ts|tsx|js|jsx)$', authMiddlewarePatterns = DEFAULT_AUTH_MIDDLEWARE_PATTERNS, routeHandlerPatterns = DEFAULT_ROUTE_HANDLER_PATTERNS, ignorePatterns = [], } = options;
|
|
176
|
+
const filename = context.getFilename();
|
|
177
|
+
const testFileRegex = new RegExp(testFilePattern);
|
|
178
|
+
const isTestFile = allowInTests && testFileRegex.test(filename);
|
|
179
|
+
const sourceCode = context.sourceCode || context.sourceCode;
|
|
180
|
+
/**
|
|
181
|
+
* Find variable declaration for an identifier
|
|
182
|
+
*/
|
|
183
|
+
function findVariableDeclaration(identifier) {
|
|
184
|
+
const varName = identifier.name;
|
|
185
|
+
let current = identifier;
|
|
186
|
+
while (current) {
|
|
187
|
+
// Search in current scope
|
|
188
|
+
if (current.type === 'Program' ||
|
|
189
|
+
current.type === 'FunctionDeclaration' ||
|
|
190
|
+
current.type === 'FunctionExpression' ||
|
|
191
|
+
current.type === 'ArrowFunctionExpression') {
|
|
192
|
+
const scopeBody = current.type === 'Program'
|
|
193
|
+
? current.body
|
|
194
|
+
: (current.body.type === 'BlockStatement' ? current.body.body : []);
|
|
195
|
+
for (const stmt of scopeBody) {
|
|
196
|
+
if (stmt.type === 'VariableDeclaration') {
|
|
197
|
+
for (const declarator of stmt.declarations) {
|
|
198
|
+
if (declarator.id.type === 'Identifier' && declarator.id.name === varName) {
|
|
199
|
+
return declarator;
|
|
200
|
+
}
|
|
201
|
+
}
|
|
202
|
+
}
|
|
203
|
+
}
|
|
204
|
+
}
|
|
205
|
+
// Traverse up
|
|
206
|
+
if ('parent' in current && current.parent) {
|
|
207
|
+
current = current.parent;
|
|
208
|
+
}
|
|
209
|
+
else {
|
|
210
|
+
break;
|
|
211
|
+
}
|
|
212
|
+
}
|
|
213
|
+
return null;
|
|
214
|
+
}
|
|
215
|
+
/**
|
|
216
|
+
* Check if an identifier was assigned from an auth middleware call
|
|
217
|
+
*/
|
|
218
|
+
function isIdentifierFromAuthMiddleware(identifier) {
|
|
219
|
+
const declarator = findVariableDeclaration(identifier);
|
|
220
|
+
if (!declarator || !declarator.init) {
|
|
221
|
+
return false;
|
|
222
|
+
}
|
|
223
|
+
// Check if init is a CallExpression to auth middleware
|
|
224
|
+
if (declarator.init.type === 'CallExpression' && declarator.init.callee.type === 'Identifier') {
|
|
225
|
+
const calleeName = declarator.init.callee.name.toLowerCase();
|
|
226
|
+
return authMiddlewarePatterns.some(pattern => calleeName.includes(pattern.toLowerCase()));
|
|
227
|
+
}
|
|
228
|
+
return false;
|
|
229
|
+
}
|
|
230
|
+
/**
|
|
231
|
+
* Check CallExpression for route handlers without authentication
|
|
232
|
+
*/
|
|
233
|
+
function checkCallExpression(node) {
|
|
234
|
+
if (isTestFile) {
|
|
235
|
+
return;
|
|
236
|
+
}
|
|
237
|
+
// Check if it's a route handler call (app.get, router.post, etc.)
|
|
238
|
+
if (node.callee.type === 'MemberExpression') {
|
|
239
|
+
const property = node.callee.property;
|
|
240
|
+
if (property.type === 'Identifier') {
|
|
241
|
+
const methodName = property.name.toLowerCase();
|
|
242
|
+
if (routeHandlerPatterns.includes(methodName)) {
|
|
243
|
+
// Extract route path if available
|
|
244
|
+
let routePath = 'unknown';
|
|
245
|
+
if (node.arguments.length > 0 && node.arguments[0].type === 'Literal') {
|
|
246
|
+
routePath = String(node.arguments[0].value);
|
|
247
|
+
}
|
|
248
|
+
else if (node.arguments.length > 0) {
|
|
249
|
+
const pathText = sourceCode.getText(node.arguments[0]);
|
|
250
|
+
routePath = pathText;
|
|
251
|
+
}
|
|
252
|
+
const text = sourceCode.getText(node);
|
|
253
|
+
const pathIgnored = matchesIgnorePattern(routePath, ignorePatterns);
|
|
254
|
+
if (pathIgnored) {
|
|
255
|
+
console.log('DEBUG MSG: Ignored path:', routePath);
|
|
256
|
+
return;
|
|
257
|
+
}
|
|
258
|
+
if (matchesIgnorePattern(text, ignorePatterns)) {
|
|
259
|
+
console.log('DEBUG MSG: Ignored text:', text);
|
|
260
|
+
return;
|
|
261
|
+
}
|
|
262
|
+
// Check if authentication middleware is present in arguments
|
|
263
|
+
let hasAuth = false;
|
|
264
|
+
for (const arg of node.arguments) {
|
|
265
|
+
const argText = sourceCode.getText(arg);
|
|
266
|
+
if (authMiddlewarePatterns.some(pattern => argText.toLowerCase().includes(pattern.toLowerCase()))) {
|
|
267
|
+
hasAuth = true;
|
|
268
|
+
break;
|
|
269
|
+
}
|
|
270
|
+
// Check if argument is a call to auth middleware
|
|
271
|
+
if (arg.type === 'CallExpression' && arg.callee.type === 'Identifier') {
|
|
272
|
+
const calleeName = arg.callee.name.toLowerCase();
|
|
273
|
+
if (authMiddlewarePatterns.some(pattern => calleeName.includes(pattern.toLowerCase()))) {
|
|
274
|
+
hasAuth = true;
|
|
275
|
+
break;
|
|
276
|
+
}
|
|
277
|
+
}
|
|
278
|
+
// Check if argument is an identifier assigned from auth middleware
|
|
279
|
+
if (arg.type === 'Identifier' && isIdentifierFromAuthMiddleware(arg)) {
|
|
280
|
+
hasAuth = true;
|
|
281
|
+
break;
|
|
282
|
+
}
|
|
283
|
+
}
|
|
284
|
+
// Check if the handler function itself is inside an auth middleware context
|
|
285
|
+
if (!hasAuth && node.arguments.length > 0) {
|
|
286
|
+
const lastArg = node.arguments[node.arguments.length - 1];
|
|
287
|
+
if (lastArg.type === 'ArrowFunctionExpression' ||
|
|
288
|
+
lastArg.type === 'FunctionExpression') {
|
|
289
|
+
// Check if the handler is inside an auth middleware call
|
|
290
|
+
if (isInsideAuthMiddleware(lastArg, sourceCode, authMiddlewarePatterns)) {
|
|
291
|
+
hasAuth = true;
|
|
292
|
+
}
|
|
293
|
+
}
|
|
294
|
+
}
|
|
295
|
+
if (!hasAuth) {
|
|
296
|
+
context.report({
|
|
297
|
+
node: node.callee,
|
|
298
|
+
messageId: 'missingAuthentication',
|
|
299
|
+
data: {
|
|
300
|
+
route: `${methodName}(${routePath})`,
|
|
301
|
+
method: methodName,
|
|
302
|
+
path: routePath,
|
|
303
|
+
},
|
|
304
|
+
suggest: [
|
|
305
|
+
{
|
|
306
|
+
messageId: 'addAuthentication',
|
|
307
|
+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
|
|
308
|
+
fix: (_fixer) => null,
|
|
309
|
+
},
|
|
310
|
+
],
|
|
311
|
+
});
|
|
312
|
+
}
|
|
313
|
+
}
|
|
314
|
+
}
|
|
315
|
+
}
|
|
316
|
+
}
|
|
317
|
+
return {
|
|
318
|
+
CallExpression: checkCallExpression,
|
|
319
|
+
};
|
|
320
|
+
},
|
|
321
|
+
});
|
|
322
|
+
//# sourceMappingURL=no-missing-authentication.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-missing-authentication.js","sourceRoot":"","sources":["../../../../../../packages/eslint-plugin-secure-coding/src/rules/security/no-missing-authentication.ts"],"names":[],"mappings":";;;AASA,4DAA0E;AAC1E,4DAAsD;AAuBtD;;GAEG;AACH,MAAM,gCAAgC,GAAG;IACvC,cAAc;IACd,MAAM;IACN,aAAa;IACb,iBAAiB;IACjB,aAAa;IACb,WAAW;IACX,qBAAqB;IACrB,uBAAuB;IACvB,KAAK;IACL,SAAS;CACV,CAAC;AAEF;;GAEG;AACH,MAAM,8BAA8B,GAAG;IACrC,KAAK;IACL,MAAM;IACN,KAAK;IACL,QAAQ;IACR,OAAO;IACP,KAAK;IACL,KAAK;CACN,CAAC;AAEF;;GAEG;AACH,SAAS,sBAAsB,CAC7B,IAAmB,EACnB,UAA+B,EAC/B,YAAsB;IAEtB,IAAI,OAAO,GAAyB,IAAI,CAAC;IAEzC,OAAO,OAAO,EAAE,CAAC;QACf,sDAAsD;QACtD,IAAI,OAAO,CAAC,MAAM,IAAI,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,gBAAgB,EAAE,CAAC;YAC/D,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAiC,CAAC;YAE3D,2DAA2D;YAC3D,MAAM,UAAU,GAAG,QAAQ,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,GAAoC,EAAE,EAAE,CAAC,GAAG,KAAK,OAAO,CAAC,CAAC;YACtG,IAAI,CAAC,UAAU,EAAE,CAAC;gBAChB,uCAAuC;gBACvC,IAAI,QAAQ,IAAI,OAAO,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;oBAC1C,OAAO,GAAG,OAAO,CAAC,MAAuB,CAAC;oBAC1C,SAAS;gBACX,CAAC;qBAAM,CAAC;oBACN,MAAM;gBACR,CAAC;YACH,CAAC;YAED,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC;YAE/B,kDAAkD;YAClD,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gBACjC,MAAM,UAAU,GAAG,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;gBAC7C,IAAI,YAAY,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC,EAAE,CAAC;oBAC7E,OAAO,IAAI,CAAC;gBACd,CAAC;YACH,CAAC;YAED,yDAAyD;YACzD,IAAI,MAAM,CAAC,IAAI,KAAK,kBAAkB,IAAI,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gBAChF,MAAM,YAAY,GAAG,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;gBACxD,IAAI,YAAY,KAAK,KAAK,IAAI,YAAY,KAAK,KAAK,EAAE,CAAC;oBACrD,8CAA8C;oBAC9C,KAAK,MAAM,GAAG,IAAI,QAAQ,CAAC,SAAS,EAAE,CAAC;wBACrC,MAAM,OAAO,GAAG,UAAU,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;wBACxC,IAAI,YAAY,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC,EAAE,CAAC;4BACxF,OAAO,IAAI,CAAC;wBACd,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED,sBAAsB;QACtB,IAAI,QAAQ,IAAI,OAAO,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;YAC1C,OAAO,GAAG,OAAO,CAAC,MAAuB,CAAC;QAC5C,CAAC;aAAM,CAAC;YACN,MAAM;QACR,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;GAEG;AACH,SAAS,oBAAoB,CAAC,IAAY,EAAE,QAAkB;IAC5D,OAAO,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE;QAC7B,IAAI,CAAC;YACH,MAAM,KAAK,GAAG,IAAI,MAAM,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC;YACvC,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1B,CAAC;QAAC,MAAM,CAAC;YACP,gDAAgD;YAChD,OAAO,IAAI,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC;QAC5D,CAAC;IACH,CAAC,CAAC,CAAC;AACL,CAAC;AAEY,QAAA,uBAAuB,GAAG,IAAA,0BAAU,EAA0B;IACzE,IAAI,EAAE,2BAA2B;IACjC,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,yDAAyD;SACvE;QACD,cAAc,EAAE,IAAI;QACpB,QAAQ,EAAE;YACR,qBAAqB,EAAE,IAAA,gCAAgB,EAAC;gBACtC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,wBAAwB;gBACnC,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,uDAAuD;gBACpE,QAAQ,EAAE,UAAU;gBACpB,GAAG,EAAE,sFAAsF;gBAC3F,iBAAiB,EAAE,iDAAiD;aACrE,CAAC;YACF,iBAAiB,EAAE,IAAA,gCAAgB,EAAC;gBAClC,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,oBAAoB;gBAC/B,WAAW,EAAE,oDAAoD;gBACjE,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,2CAA2C;gBAChD,iBAAiB,EAAE,iDAAiD;aACrE,CAAC;SACH;QACD,MAAM,EAAE;YACN;gBACE,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACV,YAAY,EAAE;wBACZ,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,KAAK;wBACd,WAAW,EAAE,4CAA4C;qBAC1D;oBACD,sBAAsB,EAAE;wBACtB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,gCAAgC;wBACzC,WAAW,EAAE,iDAAiD;qBAC/D;oBACD,oBAAoB,EAAE;wBACpB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,8BAA8B;wBACvC,WAAW,EAAE,iCAAiC;qBAC/C;oBACD,cAAc,EAAE;wBACd,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,+BAA+B;qBAC7C;oBACD,eAAe,EAAE;wBACf,IAAI,EAAE,QAAQ;wBACd,OAAO,EAAE,mCAAmC;wBAC5C,WAAW,EAAE,gCAAgC;qBAC9C;iBACF;gBACD,oBAAoB,EAAE,KAAK;aAC5B;SACF;KACF;IACD,cAAc,EAAE;QACd;YACE,YAAY,EAAE,KAAK;YACnB,eAAe,EAAE,mCAAmC;YACpD,sBAAsB,EAAE,gCAAgC;YACxD,oBAAoB,EAAE,8BAA8B;YACpD,cAAc,EAAE,EAAE;SACnB;KACF;IACD,MAAM,CACJ,OAAsD,EACtD,CAAC,OAAO,GAAG,EAAE,CAAC;QAEd,MAAM,EACJ,YAAY,GAAG,KAAK,EACpB,eAAe,GAAG,mCAAmC,EACrD,sBAAsB,GAAG,gCAAgC,EACzD,oBAAoB,GAAG,8BAA8B,EACrD,cAAc,GAAG,EAAE,GACpB,GAAG,OAAkB,CAAC;QAEvB,MAAM,QAAQ,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;QACvC,MAAM,aAAa,GAAG,IAAI,MAAM,CAAC,eAAe,CAAC,CAAC;QAClD,MAAM,UAAU,GAAG,YAAY,IAAI,aAAa,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAChE,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;QAE5D;;WAEG;QACH,SAAS,uBAAuB,CAAC,UAA+B;YAC9D,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,CAAC;YAChC,IAAI,OAAO,GAAyB,UAAU,CAAC;YAE/C,OAAO,OAAO,EAAE,CAAC;gBACf,0BAA0B;gBAC1B,IAAI,OAAO,CAAC,IAAI,KAAK,SAAS;oBAC1B,OAAO,CAAC,IAAI,KAAK,qBAAqB;oBACtC,OAAO,CAAC,IAAI,KAAK,oBAAoB;oBACrC,OAAO,CAAC,IAAI,KAAK,yBAAyB,EAAE,CAAC;oBAC/C,MAAM,SAAS,GAAG,OAAO,CAAC,IAAI,KAAK,SAAS;wBAC1C,CAAC,CAAC,OAAO,CAAC,IAAI;wBACd,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,KAAK,gBAAgB,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;oBAEtE,KAAK,MAAM,IAAI,IAAI,SAAS,EAAE,CAAC;wBAC7B,IAAI,IAAI,CAAC,IAAI,KAAK,qBAAqB,EAAE,CAAC;4BACxC,KAAK,MAAM,UAAU,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;gCAC3C,IAAI,UAAU,CAAC,EAAE,CAAC,IAAI,KAAK,YAAY,IAAI,UAAU,CAAC,EAAE,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;oCAC1E,OAAO,UAAU,CAAC;gCACpB,CAAC;4BACH,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;gBAED,cAAc;gBACd,IAAI,QAAQ,IAAI,OAAO,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;oBAC1C,OAAO,GAAG,OAAO,CAAC,MAAuB,CAAC;gBAC5C,CAAC;qBAAM,CAAC;oBACN,MAAM;gBACR,CAAC;YACH,CAAC;YAED,OAAO,IAAI,CAAC;QACd,CAAC;QAED;;WAEG;QACH,SAAS,8BAA8B,CAAC,UAA+B;YACrE,MAAM,UAAU,GAAG,uBAAuB,CAAC,UAAU,CAAC,CAAC;YACvD,IAAI,CAAC,UAAU,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,CAAC;gBACpC,OAAO,KAAK,CAAC;YACf,CAAC;YAED,uDAAuD;YACvD,IAAI,UAAU,CAAC,IAAI,CAAC,IAAI,KAAK,gBAAgB,IAAI,UAAU,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gBAC9F,MAAM,UAAU,GAAG,UAAU,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;gBAC7D,OAAO,sBAAsB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC;YAC5F,CAAC;YAED,OAAO,KAAK,CAAC;QACf,CAAC;QAED;;WAEG;QACH,SAAS,mBAAmB,CAAC,IAA6B;YACxD,IAAI,UAAU,EAAE,CAAC;gBACf,OAAO;YACT,CAAC;YAED,kEAAkE;YAClE,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB,EAAE,CAAC;gBAC5C,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC;gBACtC,IAAI,QAAQ,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;oBACnC,MAAM,UAAU,GAAG,QAAQ,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;oBAE/C,IAAI,oBAAoB,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;wBAC9C,kCAAkC;wBAClC,IAAI,SAAS,GAAG,SAAS,CAAC;wBAC1B,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;4BACtE,SAAS,GAAG,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;wBAC9C,CAAC;6BAAM,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;4BACrC,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC;4BACvD,SAAS,GAAG,QAAQ,CAAC;wBACvB,CAAC;wBAED,MAAM,IAAI,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;wBACtC,MAAM,WAAW,GAAG,oBAAoB,CAAC,SAAS,EAAE,cAAc,CAAC,CAAC;wBACpE,IAAI,WAAW,EAAE,CAAC;4BACf,OAAO,CAAC,GAAG,CAAC,0BAA0B,EAAE,SAAS,CAAC,CAAC;4BACnD,OAAO;wBACV,CAAC;wBACD,IAAI,oBAAoB,CAAC,IAAI,EAAE,cAAc,CAAC,EAAE,CAAC;4BAC9C,OAAO,CAAC,GAAG,CAAC,0BAA0B,EAAE,IAAI,CAAC,CAAC;4BAC9C,OAAO;wBACV,CAAC;wBAED,6DAA6D;wBAC7D,IAAI,OAAO,GAAG,KAAK,CAAC;wBACpB,KAAK,MAAM,GAAG,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;4BACjC,MAAM,OAAO,GAAG,UAAU,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;4BACxC,IAAI,sBAAsB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CACxC,OAAO,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CACtD,EAAE,CAAC;gCACF,OAAO,GAAG,IAAI,CAAC;gCACf,MAAM;4BACR,CAAC;4BAED,iDAAiD;4BACjD,IAAI,GAAG,CAAC,IAAI,KAAK,gBAAgB,IAAI,GAAG,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gCACtE,MAAM,UAAU,GAAG,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;gCACjD,IAAI,sBAAsB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CACxC,UAAU,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAC3C,EAAE,CAAC;oCACF,OAAO,GAAG,IAAI,CAAC;oCACf,MAAM;gCACR,CAAC;4BACH,CAAC;4BAED,mEAAmE;4BACnE,IAAI,GAAG,CAAC,IAAI,KAAK,YAAY,IAAI,8BAA8B,CAAC,GAAG,CAAC,EAAE,CAAC;gCACrE,OAAO,GAAG,IAAI,CAAC;gCACf,MAAM;4BACR,CAAC;wBACH,CAAC;wBAED,4EAA4E;wBAC5E,IAAI,CAAC,OAAO,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;4BAC1C,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;4BAC1D,IAAI,OAAO,CAAC,IAAI,KAAK,yBAAyB;gCAC1C,OAAO,CAAC,IAAI,KAAK,oBAAoB,EAAE,CAAC;gCAC1C,yDAAyD;gCACzD,IAAI,sBAAsB,CAAC,OAAO,EAAE,UAAU,EAAE,sBAAsB,CAAC,EAAE,CAAC;oCACxE,OAAO,GAAG,IAAI,CAAC;gCACjB,CAAC;4BACH,CAAC;wBACH,CAAC;wBAED,IAAI,CAAC,OAAO,EAAE,CAAC;4BACb,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,EAAE,IAAI,CAAC,MAAM;gCACjB,SAAS,EAAE,uBAAuB;gCAClC,IAAI,EAAE;oCACJ,KAAK,EAAE,GAAG,UAAU,IAAI,SAAS,GAAG;oCACpC,MAAM,EAAE,UAAU;oCAClB,IAAI,EAAE,SAAS;iCAChB;gCACD,OAAO,EAAE;oCACP;wCACE,SAAS,EAAE,mBAAmB;wCAC9B,6DAA6D;wCAC7D,GAAG,EAAE,CAAC,MAA0B,EAAE,EAAE,CAAC,IAAI;qCAC1C;iCACF;6BACF,CAAC,CAAC;wBACL,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO;YACL,cAAc,EAAE,mBAAmB;SACpC,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
export interface Options {
|
|
2
|
+
/** Allow missing CORS checks in test files. Default: false */
|
|
3
|
+
allowInTests?: boolean;
|
|
4
|
+
/** Trusted CORS libraries. Default: ['cors', '@koa/cors', 'express-cors'] */
|
|
5
|
+
trustedLibraries?: string[];
|
|
6
|
+
/** Additional safe patterns to ignore. Default: [] */
|
|
7
|
+
ignorePatterns?: string[];
|
|
8
|
+
}
|
|
9
|
+
export declare const noMissingCorsCheck: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|