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.
Files changed (155) hide show
  1. package/AGENTS.md +196 -0
  2. package/CHANGELOG.md +105 -0
  3. package/LICENSE +23 -0
  4. package/README.md +377 -0
  5. package/package.json +80 -0
  6. package/src/index.d.ts +32 -0
  7. package/src/index.js +345 -0
  8. package/src/index.js.map +1 -0
  9. package/src/rules/security/database-injection.d.ts +13 -0
  10. package/src/rules/security/database-injection.js +407 -0
  11. package/src/rules/security/database-injection.js.map +1 -0
  12. package/src/rules/security/detect-child-process.d.ts +11 -0
  13. package/src/rules/security/detect-child-process.js +460 -0
  14. package/src/rules/security/detect-child-process.js.map +1 -0
  15. package/src/rules/security/detect-eval-with-expression.d.ts +9 -0
  16. package/src/rules/security/detect-eval-with-expression.js +393 -0
  17. package/src/rules/security/detect-eval-with-expression.js.map +1 -0
  18. package/src/rules/security/detect-non-literal-fs-filename.d.ts +7 -0
  19. package/src/rules/security/detect-non-literal-fs-filename.js +322 -0
  20. package/src/rules/security/detect-non-literal-fs-filename.js.map +1 -0
  21. package/src/rules/security/detect-non-literal-regexp.d.ts +9 -0
  22. package/src/rules/security/detect-non-literal-regexp.js +387 -0
  23. package/src/rules/security/detect-non-literal-regexp.js.map +1 -0
  24. package/src/rules/security/detect-object-injection.d.ts +11 -0
  25. package/src/rules/security/detect-object-injection.js +411 -0
  26. package/src/rules/security/detect-object-injection.js.map +1 -0
  27. package/src/rules/security/no-buffer-overread.d.ts +14 -0
  28. package/src/rules/security/no-buffer-overread.js +519 -0
  29. package/src/rules/security/no-buffer-overread.js.map +1 -0
  30. package/src/rules/security/no-clickjacking.d.ts +10 -0
  31. package/src/rules/security/no-clickjacking.js +381 -0
  32. package/src/rules/security/no-clickjacking.js.map +1 -0
  33. package/src/rules/security/no-directive-injection.d.ts +12 -0
  34. package/src/rules/security/no-directive-injection.js +446 -0
  35. package/src/rules/security/no-directive-injection.js.map +1 -0
  36. package/src/rules/security/no-document-cookie.d.ts +5 -0
  37. package/src/rules/security/no-document-cookie.js +90 -0
  38. package/src/rules/security/no-document-cookie.js.map +1 -0
  39. package/src/rules/security/no-electron-security-issues.d.ts +10 -0
  40. package/src/rules/security/no-electron-security-issues.js +421 -0
  41. package/src/rules/security/no-electron-security-issues.js.map +1 -0
  42. package/src/rules/security/no-exposed-sensitive-data.d.ts +11 -0
  43. package/src/rules/security/no-exposed-sensitive-data.js +341 -0
  44. package/src/rules/security/no-exposed-sensitive-data.js.map +1 -0
  45. package/src/rules/security/no-format-string-injection.d.ts +17 -0
  46. package/src/rules/security/no-format-string-injection.js +653 -0
  47. package/src/rules/security/no-format-string-injection.js.map +1 -0
  48. package/src/rules/security/no-graphql-injection.d.ts +12 -0
  49. package/src/rules/security/no-graphql-injection.js +410 -0
  50. package/src/rules/security/no-graphql-injection.js.map +1 -0
  51. package/src/rules/security/no-hardcoded-credentials.d.ts +26 -0
  52. package/src/rules/security/no-hardcoded-credentials.js +377 -0
  53. package/src/rules/security/no-hardcoded-credentials.js.map +1 -0
  54. package/src/rules/security/no-improper-sanitization.d.ts +12 -0
  55. package/src/rules/security/no-improper-sanitization.js +408 -0
  56. package/src/rules/security/no-improper-sanitization.js.map +1 -0
  57. package/src/rules/security/no-improper-type-validation.d.ts +10 -0
  58. package/src/rules/security/no-improper-type-validation.js +420 -0
  59. package/src/rules/security/no-improper-type-validation.js.map +1 -0
  60. package/src/rules/security/no-insecure-comparison.d.ts +7 -0
  61. package/src/rules/security/no-insecure-comparison.js +125 -0
  62. package/src/rules/security/no-insecure-comparison.js.map +1 -0
  63. package/src/rules/security/no-insecure-cookie-settings.d.ts +9 -0
  64. package/src/rules/security/no-insecure-cookie-settings.js +305 -0
  65. package/src/rules/security/no-insecure-cookie-settings.js.map +1 -0
  66. package/src/rules/security/no-insecure-jwt.d.ts +10 -0
  67. package/src/rules/security/no-insecure-jwt.js +338 -0
  68. package/src/rules/security/no-insecure-jwt.js.map +1 -0
  69. package/src/rules/security/no-insecure-redirects.d.ts +7 -0
  70. package/src/rules/security/no-insecure-redirects.js +215 -0
  71. package/src/rules/security/no-insecure-redirects.js.map +1 -0
  72. package/src/rules/security/no-insufficient-postmessage-validation.d.ts +14 -0
  73. package/src/rules/security/no-insufficient-postmessage-validation.js +390 -0
  74. package/src/rules/security/no-insufficient-postmessage-validation.js.map +1 -0
  75. package/src/rules/security/no-insufficient-random.d.ts +9 -0
  76. package/src/rules/security/no-insufficient-random.js +207 -0
  77. package/src/rules/security/no-insufficient-random.js.map +1 -0
  78. package/src/rules/security/no-ldap-injection.d.ts +10 -0
  79. package/src/rules/security/no-ldap-injection.js +449 -0
  80. package/src/rules/security/no-ldap-injection.js.map +1 -0
  81. package/src/rules/security/no-missing-authentication.d.ts +13 -0
  82. package/src/rules/security/no-missing-authentication.js +322 -0
  83. package/src/rules/security/no-missing-authentication.js.map +1 -0
  84. package/src/rules/security/no-missing-cors-check.d.ts +9 -0
  85. package/src/rules/security/no-missing-cors-check.js +449 -0
  86. package/src/rules/security/no-missing-cors-check.js.map +1 -0
  87. package/src/rules/security/no-missing-csrf-protection.d.ts +11 -0
  88. package/src/rules/security/no-missing-csrf-protection.js +183 -0
  89. package/src/rules/security/no-missing-csrf-protection.js.map +1 -0
  90. package/src/rules/security/no-missing-security-headers.d.ts +7 -0
  91. package/src/rules/security/no-missing-security-headers.js +217 -0
  92. package/src/rules/security/no-missing-security-headers.js.map +1 -0
  93. package/src/rules/security/no-privilege-escalation.d.ts +13 -0
  94. package/src/rules/security/no-privilege-escalation.js +321 -0
  95. package/src/rules/security/no-privilege-escalation.js.map +1 -0
  96. package/src/rules/security/no-redos-vulnerable-regex.d.ts +7 -0
  97. package/src/rules/security/no-redos-vulnerable-regex.js +307 -0
  98. package/src/rules/security/no-redos-vulnerable-regex.js.map +1 -0
  99. package/src/rules/security/no-sensitive-data-exposure.d.ts +11 -0
  100. package/src/rules/security/no-sensitive-data-exposure.js +251 -0
  101. package/src/rules/security/no-sensitive-data-exposure.js.map +1 -0
  102. package/src/rules/security/no-sql-injection.d.ts +10 -0
  103. package/src/rules/security/no-sql-injection.js +332 -0
  104. package/src/rules/security/no-sql-injection.js.map +1 -0
  105. package/src/rules/security/no-timing-attack.d.ts +10 -0
  106. package/src/rules/security/no-timing-attack.js +358 -0
  107. package/src/rules/security/no-timing-attack.js.map +1 -0
  108. package/src/rules/security/no-toctou-vulnerability.d.ts +7 -0
  109. package/src/rules/security/no-toctou-vulnerability.js +165 -0
  110. package/src/rules/security/no-toctou-vulnerability.js.map +1 -0
  111. package/src/rules/security/no-unchecked-loop-condition.d.ts +12 -0
  112. package/src/rules/security/no-unchecked-loop-condition.js +635 -0
  113. package/src/rules/security/no-unchecked-loop-condition.js.map +1 -0
  114. package/src/rules/security/no-unencrypted-transmission.d.ts +11 -0
  115. package/src/rules/security/no-unencrypted-transmission.js +237 -0
  116. package/src/rules/security/no-unencrypted-transmission.js.map +1 -0
  117. package/src/rules/security/no-unescaped-url-parameter.d.ts +9 -0
  118. package/src/rules/security/no-unescaped-url-parameter.js +266 -0
  119. package/src/rules/security/no-unescaped-url-parameter.js.map +1 -0
  120. package/src/rules/security/no-unlimited-resource-allocation.d.ts +12 -0
  121. package/src/rules/security/no-unlimited-resource-allocation.js +659 -0
  122. package/src/rules/security/no-unlimited-resource-allocation.js.map +1 -0
  123. package/src/rules/security/no-unsafe-deserialization.d.ts +10 -0
  124. package/src/rules/security/no-unsafe-deserialization.js +501 -0
  125. package/src/rules/security/no-unsafe-deserialization.js.map +1 -0
  126. package/src/rules/security/no-unsafe-dynamic-require.d.ts +5 -0
  127. package/src/rules/security/no-unsafe-dynamic-require.js +107 -0
  128. package/src/rules/security/no-unsafe-dynamic-require.js.map +1 -0
  129. package/src/rules/security/no-unsafe-regex-construction.d.ts +9 -0
  130. package/src/rules/security/no-unsafe-regex-construction.js +292 -0
  131. package/src/rules/security/no-unsafe-regex-construction.js.map +1 -0
  132. package/src/rules/security/no-unsanitized-html.d.ts +9 -0
  133. package/src/rules/security/no-unsanitized-html.js +347 -0
  134. package/src/rules/security/no-unsanitized-html.js.map +1 -0
  135. package/src/rules/security/no-unvalidated-user-input.d.ts +9 -0
  136. package/src/rules/security/no-unvalidated-user-input.js +418 -0
  137. package/src/rules/security/no-unvalidated-user-input.js.map +1 -0
  138. package/src/rules/security/no-weak-crypto.d.ts +11 -0
  139. package/src/rules/security/no-weak-crypto.js +350 -0
  140. package/src/rules/security/no-weak-crypto.js.map +1 -0
  141. package/src/rules/security/no-weak-password-recovery.d.ts +12 -0
  142. package/src/rules/security/no-weak-password-recovery.js +401 -0
  143. package/src/rules/security/no-weak-password-recovery.js.map +1 -0
  144. package/src/rules/security/no-xpath-injection.d.ts +10 -0
  145. package/src/rules/security/no-xpath-injection.js +487 -0
  146. package/src/rules/security/no-xpath-injection.js.map +1 -0
  147. package/src/rules/security/no-xxe-injection.d.ts +7 -0
  148. package/src/rules/security/no-xxe-injection.js +270 -0
  149. package/src/rules/security/no-xxe-injection.js.map +1 -0
  150. package/src/rules/security/no-zip-slip.d.ts +9 -0
  151. package/src/rules/security/no-zip-slip.js +446 -0
  152. package/src/rules/security/no-zip-slip.js.map +1 -0
  153. package/src/types/index.d.ts +131 -0
  154. package/src/types/index.js +18 -0
  155. package/src/types/index.js.map +1 -0
@@ -0,0 +1,393 @@
1
+ "use strict";
2
+ Object.defineProperty(exports, "__esModule", { value: true });
3
+ exports.detectEvalWithExpression = void 0;
4
+ const eslint_devkit_1 = require("@interlace/eslint-devkit");
5
+ const eslint_devkit_2 = require("@interlace/eslint-devkit");
6
+ const EVAL_PATTERNS = [
7
+ {
8
+ pattern: 'JSON\\.parse|parse\\(.*\\)',
9
+ category: 'json',
10
+ safeAlternative: 'JSON.parse()',
11
+ example: {
12
+ bad: 'eval(\'{"key": "\' + value + \'"}"\')',
13
+ good: 'JSON.parse(\'{"key": "\' + value + \'"}"\')'
14
+ },
15
+ effort: '2 minutes'
16
+ },
17
+ {
18
+ pattern: 'Math\\.|parseInt|parseFloat',
19
+ category: 'math',
20
+ safeAlternative: 'Math functions or parseInt/parseFloat',
21
+ example: {
22
+ bad: 'eval(\'Math.\' + method + \'(\' + arg + \')\')',
23
+ good: 'const mathMethods = {sin: Math.sin, cos: Math.cos}; mathMethods[method](arg)'
24
+ },
25
+ effort: '5 minutes'
26
+ },
27
+ {
28
+ pattern: '\\$\\{|template|interpolat',
29
+ category: 'template',
30
+ safeAlternative: 'Template literals or template engine',
31
+ example: {
32
+ bad: 'eval(\'Hello \' + userName + \'!\')',
33
+ good: 'const template = `Hello ${userName}!`;'
34
+ },
35
+ effort: '3 minutes'
36
+ },
37
+ {
38
+ pattern: '\\[.*\\]|object\\[|obj\\.|\\.',
39
+ category: 'object',
40
+ safeAlternative: 'Direct property access or Map',
41
+ example: {
42
+ bad: 'eval(\'obj.\' + property)',
43
+ good: 'const allowedProps = {name: true, age: true}; if (allowedProps[property]) obj[property]'
44
+ },
45
+ effort: '8 minutes'
46
+ }
47
+ ];
48
+ exports.detectEvalWithExpression = (0, eslint_devkit_2.createRule)({
49
+ name: 'detect-eval-with-expression',
50
+ meta: {
51
+ type: 'problem',
52
+ docs: {
53
+ description: 'Detects eval(variable) which can allow an attacker to run arbitrary code',
54
+ },
55
+ messages: {
56
+ // 🎯 Token optimization: 38% reduction (47→29 tokens) - compact format saves LLM processing
57
+ evalWithExpression: (0, eslint_devkit_1.formatLLMMessage)({
58
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
59
+ issueName: 'eval() with dynamic code',
60
+ cwe: 'CWE-95',
61
+ description: 'eval() with dynamic code',
62
+ severity: 'CRITICAL',
63
+ fix: '{{safeAlternative}}',
64
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
65
+ }),
66
+ useJsonParse: (0, eslint_devkit_1.formatLLMMessage)({
67
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
68
+ issueName: 'Unsafe eval() for JSON parsing',
69
+ cwe: 'CWE-95',
70
+ description: 'Use JSON.parse() instead of eval() for JSON string parsing',
71
+ severity: 'HIGH',
72
+ fix: 'Replace eval() with JSON.parse()',
73
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
74
+ }),
75
+ useObjectAccess: (0, eslint_devkit_1.formatLLMMessage)({
76
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
77
+ issueName: 'Unsafe eval() for property access',
78
+ cwe: 'CWE-95',
79
+ description: 'Use direct property access instead of eval() for dynamic property access',
80
+ severity: 'HIGH',
81
+ fix: 'Use obj[key] or Map.get(key) instead of eval()',
82
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
83
+ }),
84
+ useTemplateLiteral: (0, eslint_devkit_1.formatLLMMessage)({
85
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
86
+ issueName: 'Unsafe eval() for string interpolation',
87
+ cwe: 'CWE-95',
88
+ description: 'Use template literals instead of eval() for string interpolation',
89
+ severity: 'HIGH',
90
+ fix: 'Replace eval() with template literals: `Hello ${name}`',
91
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
92
+ }),
93
+ useFunctionConstructor: (0, eslint_devkit_1.formatLLMMessage)({
94
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
95
+ issueName: 'Unsafe eval() for function creation',
96
+ cwe: 'CWE-95',
97
+ description: 'Use Function constructor with validation instead of eval()',
98
+ severity: 'HIGH',
99
+ fix: 'Replace eval() with validated Function constructor',
100
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
101
+ }),
102
+ useSaferAlternative: (0, eslint_devkit_1.formatLLMMessage)({
103
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
104
+ issueName: 'Unsafe eval() usage detected',
105
+ cwe: 'CWE-95',
106
+ description: 'eval() with dynamic code execution detected',
107
+ severity: 'HIGH',
108
+ fix: '{{alternative}}',
109
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
110
+ }),
111
+ strategyRemove: (0, eslint_devkit_1.formatLLMMessage)({
112
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
113
+ issueName: 'Critical eval() security vulnerability',
114
+ cwe: 'CWE-95',
115
+ description: 'eval() usage poses severe security risk',
116
+ severity: 'CRITICAL',
117
+ fix: 'Remove eval() entirely - security risk too high',
118
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
119
+ }),
120
+ strategyRefactor: (0, eslint_devkit_1.formatLLMMessage)({
121
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
122
+ issueName: 'eval() refactoring required',
123
+ cwe: 'CWE-95',
124
+ description: 'eval() can be refactored to safer alternative',
125
+ severity: 'HIGH',
126
+ fix: '{{safeAlternative}}',
127
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
128
+ }),
129
+ strategyValidate: (0, eslint_devkit_1.formatLLMMessage)({
130
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
131
+ issueName: 'eval() input validation needed',
132
+ cwe: 'CWE-95',
133
+ description: 'eval() requires input validation for security',
134
+ severity: 'MEDIUM',
135
+ fix: 'Add input validation before using eval()',
136
+ documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
137
+ })
138
+ },
139
+ schema: [
140
+ {
141
+ type: 'object',
142
+ properties: {
143
+ allowLiteralStrings: {
144
+ type: 'boolean',
145
+ default: false,
146
+ description: 'Allow eval with literal strings (false = stricter)'
147
+ },
148
+ additionalEvalFunctions: {
149
+ type: 'array',
150
+ items: { type: 'string' },
151
+ default: [],
152
+ description: 'Additional functions to treat as eval-like'
153
+ },
154
+ strategy: {
155
+ type: 'string',
156
+ enum: ['remove', 'refactor', 'validate', 'auto'],
157
+ default: 'auto',
158
+ description: 'Strategy for fixing eval usage (auto = smart detection)'
159
+ }
160
+ },
161
+ additionalProperties: false,
162
+ },
163
+ ],
164
+ },
165
+ defaultOptions: [
166
+ {
167
+ allowLiteralStrings: false,
168
+ additionalEvalFunctions: [],
169
+ strategy: 'auto'
170
+ },
171
+ ],
172
+ create(context) {
173
+ const options = context.options[0] || {};
174
+ const { allowLiteralStrings = false, additionalEvalFunctions = [], strategy = 'auto' } = options || {};
175
+ /**
176
+ * All functions that can execute arbitrary code
177
+ * NOTE: setTimeout/setInterval are NOT eval-like - they don't execute code strings
178
+ */
179
+ const evalFunctions = [
180
+ 'eval',
181
+ 'Function',
182
+ ...additionalEvalFunctions
183
+ ];
184
+ /**
185
+ * Check if a node is a literal string (safe)
186
+ */
187
+ const isLiteralString = (node) => {
188
+ return node.type === 'Literal' && typeof node.value === 'string';
189
+ };
190
+ /**
191
+ * Select message ID based on strategy
192
+ */
193
+ const selectStrategyMessage = (pattern) => {
194
+ switch (strategy) {
195
+ case 'remove':
196
+ return 'strategyRemove';
197
+ case 'refactor':
198
+ return 'strategyRefactor';
199
+ case 'validate':
200
+ return 'strategyValidate';
201
+ case 'auto':
202
+ default:
203
+ // Auto mode: choose based on pattern confidence
204
+ if (pattern && pattern.category === 'json') {
205
+ return 'useJsonParse';
206
+ }
207
+ if (pattern && pattern.category === 'object') {
208
+ return 'useObjectAccess';
209
+ }
210
+ if (pattern && pattern.category === 'template') {
211
+ return 'useTemplateLiteral';
212
+ }
213
+ return 'strategyRefactor'; // Default to refactor for unknown patterns
214
+ }
215
+ };
216
+ /**
217
+ * Detect the pattern category from the expression
218
+ */
219
+ const detectPattern = (expression) => {
220
+ for (const pattern of EVAL_PATTERNS) {
221
+ if (new RegExp(pattern.pattern, 'i').test(expression)) {
222
+ return pattern;
223
+ }
224
+ }
225
+ return null;
226
+ };
227
+ /**
228
+ * Generate refactoring steps based on pattern
229
+ */
230
+ const generateRefactoringSteps = (pattern) => {
231
+ if (!pattern) {
232
+ return [
233
+ ' 1. Remove eval() usage entirely',
234
+ ' 2. Identify what the code is trying to achieve',
235
+ ' 3. Use appropriate safe alternative (JSON.parse, Map, etc.)',
236
+ ' 4. Add input validation if dynamic behavior needed',
237
+ ' 5. Test thoroughly for edge cases'
238
+ ].join('\n');
239
+ }
240
+ switch (pattern.category) {
241
+ case 'json':
242
+ return [
243
+ ' 1. Replace eval() with JSON.parse()',
244
+ ' 2. Ensure input is valid JSON string',
245
+ ' 3. Add try/catch for JSON parsing errors',
246
+ ' 4. Consider using a JSON schema validator'
247
+ ].join('\n');
248
+ case 'math':
249
+ return [
250
+ ' 1. Create whitelist of allowed Math functions',
251
+ ' 2. Use direct function calls: Math.sin(x)',
252
+ ' 3. Validate inputs are numbers',
253
+ ' 4. Consider using a math expression parser library'
254
+ ].join('\n');
255
+ case 'template':
256
+ return [
257
+ ' 1. Use template literals: `Hello ${name}`',
258
+ ' 2. Sanitize variables before interpolation',
259
+ ' 3. Use a template engine like Handlebars if complex',
260
+ ' 4. Validate template structure'
261
+ ].join('\n');
262
+ case 'object':
263
+ return [
264
+ ' 1. Use Map or plain object for key-value access',
265
+ ' 2. Whitelist allowed property names',
266
+ ' 3. Use hasOwnProperty() check',
267
+ ' 4. Consider Object.create(null) for clean objects'
268
+ ].join('\n');
269
+ default:
270
+ return [
271
+ ' 1. Identify the specific use case',
272
+ ' 2. Find a safer alternative approach',
273
+ ' 3. Add comprehensive input validation',
274
+ ' 4. Use static analysis if possible'
275
+ ].join('\n');
276
+ }
277
+ };
278
+ /**
279
+ * Extract expression text for pattern analysis
280
+ */
281
+ const extractExpression = (node) => {
282
+ const sourceCode = context.sourceCode || context.sourceCode;
283
+ // Try to get the argument text
284
+ if (node.arguments.length > 0) {
285
+ return sourceCode.getText(node.arguments[0]);
286
+ }
287
+ return 'dynamic expression';
288
+ };
289
+ /**
290
+ * Check call expressions for dangerous eval usage
291
+ */
292
+ const checkCallExpression = (node) => {
293
+ // Check if it's a call to an eval-like function
294
+ if (node.callee.type === 'Identifier' &&
295
+ evalFunctions.includes(node.callee.name)) {
296
+ // Skip if it's a literal string and literals are allowed
297
+ if (allowLiteralStrings &&
298
+ node.arguments.length > 0 &&
299
+ isLiteralString(node.arguments[0])) {
300
+ return;
301
+ }
302
+ // Skip if it's a direct string literal (safe)
303
+ if (node.arguments.length > 0 &&
304
+ node.callee.name === 'eval' &&
305
+ isLiteralString(node.arguments[0])) {
306
+ return;
307
+ }
308
+ const expression = extractExpression(node);
309
+ const pattern = detectPattern(expression);
310
+ const steps = generateRefactoringSteps(pattern);
311
+ const strategyMessageId = selectStrategyMessage(pattern);
312
+ context.report({
313
+ node,
314
+ messageId: strategyMessageId,
315
+ data: {
316
+ expression,
317
+ patternCategory: pattern?.category || 'dynamic code execution',
318
+ safeAlternative: pattern?.safeAlternative || 'Remove eval entirely',
319
+ steps,
320
+ effort: pattern?.effort || '15-30 minutes'
321
+ },
322
+ suggest: pattern ? [
323
+ {
324
+ messageId: strategyMessageId,
325
+ data: {
326
+ safeAlternative: pattern.safeAlternative,
327
+ alternative: pattern.safeAlternative
328
+ },
329
+ fix: () => null // Complex refactoring, cannot auto-fix safely
330
+ }
331
+ ] : undefined
332
+ });
333
+ }
334
+ // Also check for Function constructor usage
335
+ if (node.callee.type === 'NewExpression' &&
336
+ node.callee.callee.type === 'Identifier' &&
337
+ node.callee.callee.name === 'Function') {
338
+ const expression = extractExpression(node);
339
+ const pattern = detectPattern(expression);
340
+ const strategyMessageId = selectStrategyMessage(pattern);
341
+ context.report({
342
+ node,
343
+ messageId: strategyMessageId,
344
+ data: {
345
+ expression: `new Function(${expression})`,
346
+ patternCategory: 'function constructor',
347
+ safeAlternative: 'Arrow function or regular function',
348
+ steps: [
349
+ ' 1. Replace Function constructor with arrow function',
350
+ ' 2. Use regular function declaration',
351
+ ' 3. Validate any dynamic parts',
352
+ ' 4. Consider module imports instead'
353
+ ].join('\n'),
354
+ effort: '10 minutes'
355
+ }
356
+ });
357
+ }
358
+ };
359
+ /**
360
+ * Check NewExpression for Function constructor usage
361
+ */
362
+ const checkNewExpression = (node) => {
363
+ // Check for new Function() usage
364
+ if (node.callee.type === 'Identifier' && node.callee.name === 'Function') {
365
+ const sourceCode = context.sourceCode || context.sourceCode;
366
+ const expression = node.arguments.map((arg) => sourceCode.getText(arg)).join(', ');
367
+ const pattern = detectPattern(expression);
368
+ const strategyMessageId = selectStrategyMessage(pattern);
369
+ context.report({
370
+ node,
371
+ messageId: strategyMessageId,
372
+ data: {
373
+ expression: `new Function(${expression})`,
374
+ patternCategory: 'function constructor',
375
+ safeAlternative: 'Arrow function or regular function',
376
+ steps: [
377
+ ' 1. Replace Function constructor with arrow function',
378
+ ' 2. Use regular function declaration',
379
+ ' 3. Validate any dynamic parts',
380
+ ' 4. Consider module imports instead'
381
+ ].join('\n'),
382
+ effort: '10 minutes'
383
+ }
384
+ });
385
+ }
386
+ };
387
+ return {
388
+ CallExpression: checkCallExpression,
389
+ NewExpression: checkNewExpression
390
+ };
391
+ },
392
+ });
393
+ //# sourceMappingURL=detect-eval-with-expression.js.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"detect-eval-with-expression.js","sourceRoot":"","sources":["../../../../../../packages/eslint-plugin-secure-coding/src/rules/security/detect-eval-with-expression.ts"],"names":[],"mappings":";;;AASA,4DAA0E;AAC1E,4DAAsD;AAqCtD,MAAM,aAAa,GAAkB;IACnC;QACE,OAAO,EAAE,4BAA4B;QACrC,QAAQ,EAAE,MAAM;QAChB,eAAe,EAAE,cAAc;QAC/B,OAAO,EAAE;YACP,GAAG,EAAE,uCAAuC;YAC5C,IAAI,EAAE,6CAA6C;SACpD;QACD,MAAM,EAAE,WAAW;KACpB;IACD;QACE,OAAO,EAAE,6BAA6B;QACtC,QAAQ,EAAE,MAAM;QAChB,eAAe,EAAE,uCAAuC;QACxD,OAAO,EAAE;YACP,GAAG,EAAE,gDAAgD;YACrD,IAAI,EAAE,8EAA8E;SACrF;QACD,MAAM,EAAE,WAAW;KACpB;IACD;QACE,OAAO,EAAE,4BAA4B;QACrC,QAAQ,EAAE,UAAU;QACpB,eAAe,EAAE,sCAAsC;QACvD,OAAO,EAAE;YACP,GAAG,EAAE,qCAAqC;YAC1C,IAAI,EAAE,wCAAwC;SAC/C;QACD,MAAM,EAAE,WAAW;KACpB;IACD;QACE,OAAO,EAAE,+BAA+B;QACxC,QAAQ,EAAE,QAAQ;QAClB,eAAe,EAAE,+BAA+B;QAChD,OAAO,EAAE;YACP,GAAG,EAAE,2BAA2B;YAChC,IAAI,EAAE,yFAAyF;SAChG;QACD,MAAM,EAAE,WAAW;KACpB;CACF,CAAC;AAEW,QAAA,wBAAwB,GAAG,IAAA,0BAAU,EAA0B;IAC1E,IAAI,EAAE,6BAA6B;IACnC,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,0EAA0E;SACxF;QACD,QAAQ,EAAE;YACR,4FAA4F;YAC5F,kBAAkB,EAAE,IAAA,gCAAgB,EAAC;gBACnC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,0BAA0B;gBACrC,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,0BAA0B;gBACvC,QAAQ,EAAE,UAAU;gBACpB,GAAG,EAAE,qBAAqB;gBAC1B,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,YAAY,EAAE,IAAA,gCAAgB,EAAC;gBAC7B,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,gCAAgC;gBAC3C,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,4DAA4D;gBACzE,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,kCAAkC;gBACvC,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,eAAe,EAAE,IAAA,gCAAgB,EAAC;gBAChC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,mCAAmC;gBAC9C,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,0EAA0E;gBACvF,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,gDAAgD;gBACrD,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,kBAAkB,EAAE,IAAA,gCAAgB,EAAC;gBACnC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,wCAAwC;gBACnD,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,kEAAkE;gBAC/E,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,wDAAwD;gBAC7D,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,sBAAsB,EAAE,IAAA,gCAAgB,EAAC;gBACvC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,qCAAqC;gBAChD,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,4DAA4D;gBACzE,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,oDAAoD;gBACzD,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,mBAAmB,EAAE,IAAA,gCAAgB,EAAC;gBACpC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,8BAA8B;gBACzC,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,6CAA6C;gBAC1D,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,iBAAiB;gBACtB,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,cAAc,EAAE,IAAA,gCAAgB,EAAC;gBAC/B,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,wCAAwC;gBACnD,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,yCAAyC;gBACtD,QAAQ,EAAE,UAAU;gBACpB,GAAG,EAAE,iDAAiD;gBACtD,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,6BAA6B;gBACxC,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,+CAA+C;gBAC5D,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,qBAAqB;gBAC1B,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,gCAAgC;gBAC3C,GAAG,EAAE,QAAQ;gBACb,WAAW,EAAE,+CAA+C;gBAC5D,QAAQ,EAAE,QAAQ;gBAClB,GAAG,EAAE,0CAA0C;gBAC/C,iBAAiB,EAAE,wDAAwD;aAC5E,CAAC;SACH;QACD,MAAM,EAAE;YACN;gBACE,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACV,mBAAmB,EAAE;wBACnB,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,KAAK;wBACd,WAAW,EAAE,oDAAoD;qBAClE;oBACD,uBAAuB,EAAE;wBACvB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,4CAA4C;qBAC1D;oBACD,QAAQ,EAAE;wBACR,IAAI,EAAE,QAAQ;wBACd,IAAI,EAAE,CAAC,QAAQ,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,CAAC;wBAChD,OAAO,EAAE,MAAM;wBACf,WAAW,EAAE,yDAAyD;qBACvE;iBACF;gBACD,oBAAoB,EAAE,KAAK;aAC5B;SACF;KACF;IACD,cAAc,EAAE;QACd;YACE,mBAAmB,EAAE,KAAK;YAC1B,uBAAuB,EAAE,EAAE;YAC3B,QAAQ,EAAE,MAAM;SACjB;KACF;IACD,MAAM,CAAC,OAAsD;QAC3D,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACzC,MAAM,EACJ,mBAAmB,GAAG,KAAK,EAC3B,uBAAuB,GAAG,EAAE,EAC5B,QAAQ,GAAG,MAAM,EAClB,GAAY,OAAO,IAAI,EAAE,CAAC;QAE3B;;;WAGG;QACH,MAAM,aAAa,GAAG;YACpB,MAAM;YACN,UAAU;YACV,GAAG,uBAAuB;SAC3B,CAAC;QAEF;;WAEG;QACH,MAAM,eAAe,GAAG,CAAC,IAAmB,EAAW,EAAE;YACvD,OAAO,IAAI,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,IAAI,CAAC,KAAK,KAAK,QAAQ,CAAC;QACnE,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,qBAAqB,GAAG,CAAC,OAA2B,EAAc,EAAE;YACxE,QAAQ,QAAQ,EAAE,CAAC;gBACjB,KAAK,QAAQ;oBACX,OAAO,gBAAgB,CAAC;gBAC1B,KAAK,UAAU;oBACb,OAAO,kBAAkB,CAAC;gBAC5B,KAAK,UAAU;oBACb,OAAO,kBAAkB,CAAC;gBAC5B,KAAK,MAAM,CAAC;gBACZ;oBACE,gDAAgD;oBAChD,IAAI,OAAO,IAAI,OAAO,CAAC,QAAQ,KAAK,MAAM,EAAE,CAAC;wBAC3C,OAAO,cAAc,CAAC;oBACxB,CAAC;oBACD,IAAI,OAAO,IAAI,OAAO,CAAC,QAAQ,KAAK,QAAQ,EAAE,CAAC;wBAC7C,OAAO,iBAAiB,CAAC;oBAC3B,CAAC;oBACD,IAAI,OAAO,IAAI,OAAO,CAAC,QAAQ,KAAK,UAAU,EAAE,CAAC;wBAC/C,OAAO,oBAAoB,CAAC;oBAC9B,CAAC;oBACD,OAAO,kBAAkB,CAAC,CAAC,2CAA2C;YAC1E,CAAC;QACH,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,aAAa,GAAG,CAAC,UAAkB,EAAsB,EAAE;YAC/D,KAAK,MAAM,OAAO,IAAI,aAAa,EAAE,CAAC;gBACpC,IAAI,IAAI,MAAM,CAAC,OAAO,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,EAAE,CAAC;oBACtD,OAAO,OAAO,CAAC;gBACjB,CAAC;YACH,CAAC;YACD,OAAO,IAAI,CAAC;QACd,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,wBAAwB,GAAG,CAAC,OAA2B,EAAU,EAAE;YACvE,IAAI,CAAC,OAAO,EAAE,CAAC;gBACb,OAAO;oBACL,oCAAoC;oBACpC,mDAAmD;oBACnD,gEAAgE;oBAChE,uDAAuD;oBACvD,sCAAsC;iBACvC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACf,CAAC;YAED,QAAQ,OAAO,CAAC,QAAQ,EAAE,CAAC;gBACzB,KAAK,MAAM;oBACT,OAAO;wBACL,wCAAwC;wBACxC,yCAAyC;wBACzC,6CAA6C;wBAC7C,8CAA8C;qBAC/C,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAEf,KAAK,MAAM;oBACT,OAAO;wBACL,kDAAkD;wBAClD,8CAA8C;wBAC9C,mCAAmC;wBACnC,uDAAuD;qBACxD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAEf,KAAK,UAAU;oBACb,OAAO;wBACL,8CAA8C;wBAC9C,+CAA+C;wBAC/C,wDAAwD;wBACxD,mCAAmC;qBACpC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAEf,KAAK,QAAQ;oBACX,OAAO;wBACL,oDAAoD;wBACpD,wCAAwC;wBACxC,kCAAkC;wBAClC,sDAAsD;qBACvD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAEf;oBACE,OAAO;wBACL,sCAAsC;wBACtC,yCAAyC;wBACzC,0CAA0C;wBAC1C,uCAAuC;qBACxC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACjB,CAAC;QACH,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,iBAAiB,GAAG,CAAC,IAA6B,EAAU,EAAE;YAClE,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;YAE5D,+BAA+B;YAC/B,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAC9B,OAAO,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC;YAC/C,CAAC;YAED,OAAO,oBAAoB,CAAC;QAC9B,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,mBAAmB,GAAG,CAAC,IAA6B,EAAE,EAAE;YAC5D,gDAAgD;YAChD,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY;gBACjC,aAAa,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;gBAE7C,yDAAyD;gBACzD,IAAI,mBAAmB;oBACnB,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC;oBACzB,eAAe,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;oBACvC,OAAO;gBACT,CAAC;gBAED,8CAA8C;gBAC9C,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC;oBACzB,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,MAAM;oBAC3B,eAAe,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;oBACvC,OAAO;gBACT,CAAC;gBAED,MAAM,UAAU,GAAG,iBAAiB,CAAC,IAAI,CAAC,CAAC;gBAC3C,MAAM,OAAO,GAAG,aAAa,CAAC,UAAU,CAAC,CAAC;gBAC1C,MAAM,KAAK,GAAG,wBAAwB,CAAC,OAAO,CAAC,CAAC;gBAChD,MAAM,iBAAiB,GAAG,qBAAqB,CAAC,OAAO,CAAC,CAAC;gBAEzD,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAE,iBAAiB;oBAC5B,IAAI,EAAE;wBACJ,UAAU;wBACV,eAAe,EAAE,OAAO,EAAE,QAAQ,IAAI,wBAAwB;wBAC9D,eAAe,EAAE,OAAO,EAAE,eAAe,IAAI,sBAAsB;wBACnE,KAAK;wBACL,MAAM,EAAE,OAAO,EAAE,MAAM,IAAI,eAAe;qBAC3C;oBACD,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC;wBACjB;4BACE,SAAS,EAAE,iBAAiB;4BAC5B,IAAI,EAAE;gCACJ,eAAe,EAAE,OAAO,CAAC,eAAe;gCACxC,WAAW,EAAE,OAAO,CAAC,eAAe;6BACrC;4BACD,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,CAAC,8CAA8C;yBAC/D;qBACF,CAAC,CAAC,CAAC,SAAS;iBACd,CAAC,CAAC;YACL,CAAC;YAED,4CAA4C;YAC5C,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,eAAe;gBACpC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY;gBACxC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;gBAE3C,MAAM,UAAU,GAAG,iBAAiB,CAAC,IAAI,CAAC,CAAC;gBAC3C,MAAM,OAAO,GAAG,aAAa,CAAC,UAAU,CAAC,CAAC;gBAC1C,MAAM,iBAAiB,GAAG,qBAAqB,CAAC,OAAO,CAAC,CAAC;gBAEzD,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAE,iBAAiB;oBAC5B,IAAI,EAAE;wBACJ,UAAU,EAAE,gBAAgB,UAAU,GAAG;wBACzC,eAAe,EAAE,sBAAsB;wBACvC,eAAe,EAAE,oCAAoC;wBACrD,KAAK,EAAE;4BACL,wDAAwD;4BACxD,wCAAwC;4BACxC,kCAAkC;4BAClC,uCAAuC;yBACxC,CAAC,IAAI,CAAC,IAAI,CAAC;wBACZ,MAAM,EAAE,YAAY;qBACrB;iBACF,CAAC,CAAC;YACL,CAAC;QACH,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,kBAAkB,GAAG,CAAC,IAA4B,EAAE,EAAE;YAC1D,iCAAiC;YACjC,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;gBACzE,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;gBAC5D,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,GAAkB,EAAE,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAClG,MAAM,OAAO,GAAG,aAAa,CAAC,UAAU,CAAC,CAAC;gBAC1C,MAAM,iBAAiB,GAAG,qBAAqB,CAAC,OAAO,CAAC,CAAC;gBAEzD,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAE,iBAAiB;oBAC5B,IAAI,EAAE;wBACJ,UAAU,EAAE,gBAAgB,UAAU,GAAG;wBACzC,eAAe,EAAE,sBAAsB;wBACvC,eAAe,EAAE,oCAAoC;wBACrD,KAAK,EAAE;4BACL,wDAAwD;4BACxD,wCAAwC;4BACxC,kCAAkC;4BAClC,uCAAuC;yBACxC,CAAC,IAAI,CAAC,IAAI,CAAC;wBACZ,MAAM,EAAE,YAAY;qBACrB;iBACF,CAAC,CAAC;YACL,CAAC;QACH,CAAC,CAAC;QAEF,OAAO;YACL,cAAc,EAAE,mBAAmB;YACnC,aAAa,EAAE,kBAAkB;SAClC,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}
@@ -0,0 +1,7 @@
1
+ export interface Options {
2
+ /** Allow literal strings. Default: false (stricter) */
3
+ allowLiterals?: boolean;
4
+ /** Additional fs methods to check */
5
+ additionalMethods?: string[];
6
+ }
7
+ export declare const detectNonLiteralFsFilename: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;