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,11 @@
1
+ export interface Options {
2
+ /** Sensitive data patterns. Default: ['password', 'secret', 'token', 'key', 'ssn', 'credit', 'card'] */
3
+ sensitivePatterns?: string[];
4
+ /** Check console.log statements. Default: true */
5
+ checkConsoleLog?: boolean;
6
+ /** Check error messages. Default: true */
7
+ checkErrorMessages?: boolean;
8
+ /** Check API responses. Default: true */
9
+ checkApiResponses?: boolean;
10
+ }
11
+ export declare const noSensitiveDataExposure: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
@@ -0,0 +1,251 @@
1
+ "use strict";
2
+ Object.defineProperty(exports, "__esModule", { value: true });
3
+ exports.noSensitiveDataExposure = void 0;
4
+ const eslint_devkit_1 = require("@interlace/eslint-devkit");
5
+ const eslint_devkit_2 = require("@interlace/eslint-devkit");
6
+ /**
7
+ * Check if string contains sensitive data patterns
8
+ */
9
+ function containsSensitiveData(text, patterns) {
10
+ const lowerText = text.toLowerCase();
11
+ return patterns.some(pattern => lowerText.includes(pattern.toLowerCase()));
12
+ }
13
+ exports.noSensitiveDataExposure = (0, eslint_devkit_2.createRule)({
14
+ name: 'no-sensitive-data-exposure',
15
+ meta: {
16
+ type: 'problem',
17
+ docs: {
18
+ description: 'Detects PII/credentials in logs, responses, or error messages',
19
+ },
20
+ hasSuggestions: true,
21
+ messages: {
22
+ sensitiveDataExposure: (0, eslint_devkit_1.formatLLMMessage)({
23
+ icon: eslint_devkit_1.MessageIcons.SECURITY,
24
+ issueName: 'Sensitive data exposure',
25
+ cwe: 'CWE-532',
26
+ description: 'Sensitive data detected in {{context}}: {{dataType}}',
27
+ severity: 'HIGH',
28
+ fix: 'Redact or mask sensitive data before logging/exposing',
29
+ documentationLink: 'https://cwe.mitre.org/data/definitions/532.html',
30
+ }),
31
+ redactData: (0, eslint_devkit_1.formatLLMMessage)({
32
+ icon: eslint_devkit_1.MessageIcons.INFO,
33
+ issueName: 'Redact Data',
34
+ description: 'Redact sensitive data before logging',
35
+ severity: 'LOW',
36
+ fix: 'Redact sensitive fields before logging',
37
+ documentationLink: 'https://cwe.mitre.org/data/definitions/532.html',
38
+ }),
39
+ useMasking: (0, eslint_devkit_1.formatLLMMessage)({
40
+ icon: eslint_devkit_1.MessageIcons.INFO,
41
+ issueName: 'Use Masking',
42
+ description: 'Use data masking function',
43
+ severity: 'LOW',
44
+ fix: 'maskSensitive(data)',
45
+ documentationLink: 'https://cwe.mitre.org/data/definitions/532.html',
46
+ }),
47
+ removeFromLogs: (0, eslint_devkit_1.formatLLMMessage)({
48
+ icon: eslint_devkit_1.MessageIcons.INFO,
49
+ issueName: 'Remove From Logs',
50
+ description: 'Remove sensitive data from logs and errors',
51
+ severity: 'LOW',
52
+ fix: 'Filter sensitive data before logging',
53
+ documentationLink: 'https://cwe.mitre.org/data/definitions/532.html',
54
+ }),
55
+ },
56
+ schema: [
57
+ {
58
+ type: 'object',
59
+ properties: {
60
+ sensitivePatterns: {
61
+ type: 'array',
62
+ items: { type: 'string' },
63
+ default: ['password', 'secret', 'token', 'key', 'ssn', 'credit', 'card', 'api_key', 'apikey'],
64
+ description: 'Sensitive data patterns',
65
+ },
66
+ checkConsoleLog: {
67
+ type: 'boolean',
68
+ default: true,
69
+ description: 'Check console.log statements',
70
+ },
71
+ checkErrorMessages: {
72
+ type: 'boolean',
73
+ default: true,
74
+ description: 'Check error messages',
75
+ },
76
+ checkApiResponses: {
77
+ type: 'boolean',
78
+ default: true,
79
+ description: 'Check API responses',
80
+ },
81
+ },
82
+ additionalProperties: false,
83
+ },
84
+ ],
85
+ },
86
+ defaultOptions: [
87
+ {
88
+ sensitivePatterns: ['password', 'secret', 'token', 'key', 'ssn', 'credit', 'card', 'api_key', 'apikey'],
89
+ checkConsoleLog: true,
90
+ checkErrorMessages: true,
91
+ checkApiResponses: true,
92
+ },
93
+ ],
94
+ create(context, [options = {}]) {
95
+ const { sensitivePatterns = ['password', 'secret', 'token', 'key', 'ssn', 'credit', 'card', 'api_key', 'apikey'], checkConsoleLog = true, checkErrorMessages = true, } = options || {};
96
+ /**
97
+ * Check CallExpression for logging calls with sensitive data
98
+ */
99
+ function checkCallExpression(node) {
100
+ // Check if it's a logging call (console.*, logger.*)
101
+ const isLoggingCall = (() => {
102
+ if (node.callee.type === 'MemberExpression') {
103
+ const object = node.callee.object;
104
+ const property = node.callee.property;
105
+ if (property.type === 'Identifier') {
106
+ const methodName = property.name.toLowerCase();
107
+ if (['log', 'info', 'warn', 'error', 'debug', 'trace'].includes(methodName)) {
108
+ // Check if it's console.* or logger.*
109
+ if (object.type === 'Identifier') {
110
+ const objName = object.name.toLowerCase();
111
+ if (objName === 'console' || objName === 'logger') {
112
+ return true;
113
+ }
114
+ }
115
+ }
116
+ }
117
+ }
118
+ else if (node.callee.type === 'Identifier') {
119
+ // Check for logger.info() pattern
120
+ const calleeName = node.callee.name.toLowerCase();
121
+ if (calleeName.includes('log') || calleeName.includes('logger')) {
122
+ return true;
123
+ }
124
+ }
125
+ return false;
126
+ })();
127
+ if (isLoggingCall && checkConsoleLog) {
128
+ // Check if any argument contains sensitive data
129
+ for (const arg of node.arguments) {
130
+ if (arg.type === 'Literal' && typeof arg.value === 'string') {
131
+ const text = arg.value;
132
+ if (containsSensitiveData(text, sensitivePatterns)) {
133
+ context.report({
134
+ node: arg,
135
+ messageId: 'sensitiveDataExposure',
136
+ data: {
137
+ context: 'logs',
138
+ dataType: 'password',
139
+ },
140
+ suggest: [
141
+ { messageId: 'redactData', fix: () => null },
142
+ { messageId: 'useMasking', fix: () => null },
143
+ { messageId: 'removeFromLogs', fix: () => null },
144
+ ],
145
+ });
146
+ return; // Only report once per call
147
+ }
148
+ }
149
+ else if (arg.type === 'Identifier' && arg.name) {
150
+ const name = arg.name.toLowerCase();
151
+ if (containsSensitiveData(name, sensitivePatterns)) {
152
+ context.report({
153
+ node: arg,
154
+ messageId: 'sensitiveDataExposure',
155
+ data: {
156
+ context: 'logs',
157
+ dataType: 'password',
158
+ },
159
+ suggest: [
160
+ { messageId: 'redactData', fix: () => null },
161
+ { messageId: 'useMasking', fix: () => null },
162
+ { messageId: 'removeFromLogs', fix: () => null },
163
+ ],
164
+ });
165
+ return; // Only report once per call
166
+ }
167
+ }
168
+ }
169
+ }
170
+ }
171
+ /**
172
+ * Check NewExpression for Error with sensitive data
173
+ */
174
+ function checkNewExpression(node) {
175
+ if (!checkErrorMessages) {
176
+ return;
177
+ }
178
+ if (node.callee && node.callee.type === 'Identifier' && node.callee.name === 'Error') {
179
+ // Check all arguments for sensitive data (report only once per error)
180
+ for (const arg of node.arguments) {
181
+ if (arg.type === 'Literal' && typeof arg.value === 'string') {
182
+ const text = arg.value;
183
+ if (containsSensitiveData(text, sensitivePatterns)) {
184
+ context.report({
185
+ node: arg,
186
+ messageId: 'sensitiveDataExposure',
187
+ data: {
188
+ context: 'error messages',
189
+ dataType: 'password',
190
+ },
191
+ suggest: [
192
+ { messageId: 'redactData', fix: () => null },
193
+ { messageId: 'useMasking', fix: () => null },
194
+ { messageId: 'removeFromLogs', fix: () => null },
195
+ ],
196
+ });
197
+ return; // Only report once per error
198
+ }
199
+ }
200
+ else if (arg.type === 'BinaryExpression' && arg.operator === '+') {
201
+ // Check left side if it's a literal
202
+ if (arg.left && arg.left.type === 'Literal' && typeof arg.left.value === 'string') {
203
+ const leftText = arg.left.value;
204
+ if (containsSensitiveData(leftText, sensitivePatterns)) {
205
+ context.report({
206
+ node: arg.left,
207
+ messageId: 'sensitiveDataExposure',
208
+ data: {
209
+ context: 'error messages',
210
+ dataType: 'password',
211
+ },
212
+ suggest: [
213
+ { messageId: 'redactData', fix: () => null },
214
+ { messageId: 'useMasking', fix: () => null },
215
+ { messageId: 'removeFromLogs', fix: () => null },
216
+ ],
217
+ });
218
+ return; // Only report once per error
219
+ }
220
+ }
221
+ // Check right side if it's an identifier
222
+ if (arg.right && arg.right.type === 'Identifier' && arg.right.name) {
223
+ const rightName = arg.right.name.toLowerCase();
224
+ if (containsSensitiveData(rightName, sensitivePatterns)) {
225
+ context.report({
226
+ node: arg.right,
227
+ messageId: 'sensitiveDataExposure',
228
+ data: {
229
+ context: 'error messages',
230
+ dataType: 'password',
231
+ },
232
+ suggest: [
233
+ { messageId: 'redactData', fix: () => null },
234
+ { messageId: 'useMasking', fix: () => null },
235
+ { messageId: 'removeFromLogs', fix: () => null },
236
+ ],
237
+ });
238
+ return; // Only report once per error
239
+ }
240
+ }
241
+ }
242
+ }
243
+ }
244
+ }
245
+ return {
246
+ CallExpression: checkCallExpression,
247
+ NewExpression: checkNewExpression,
248
+ };
249
+ },
250
+ });
251
+ //# sourceMappingURL=no-sensitive-data-exposure.js.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"no-sensitive-data-exposure.js","sourceRoot":"","sources":["../../../../../../packages/eslint-plugin-secure-coding/src/rules/security/no-sensitive-data-exposure.ts"],"names":[],"mappings":";;;AASA,4DAA0E;AAC1E,4DAAsD;AAwBtD;;GAEG;AACH,SAAS,qBAAqB,CAC5B,IAAY,EACZ,QAAkB;IAElB,MAAM,SAAS,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IACrC,OAAO,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,SAAS,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC;AAC7E,CAAC;AAGY,QAAA,uBAAuB,GAAG,IAAA,0BAAU,EAA0B;IACzE,IAAI,EAAE,4BAA4B;IAClC,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,+DAA+D;SAC7E;QACD,cAAc,EAAE,IAAI;QACpB,QAAQ,EAAE;YACR,qBAAqB,EAAE,IAAA,gCAAgB,EAAC;gBACtC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,yBAAyB;gBACpC,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,sDAAsD;gBACnE,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,uDAAuD;gBAC5D,iBAAiB,EAAE,iDAAiD;aACrE,CAAC;YACF,UAAU,EAAE,IAAA,gCAAgB,EAAC;gBAC3B,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,aAAa;gBACxB,WAAW,EAAE,sCAAsC;gBACnD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,wCAAwC;gBAC7C,iBAAiB,EAAE,iDAAiD;aACrE,CAAC;YACF,UAAU,EAAE,IAAA,gCAAgB,EAAC;gBAC3B,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,aAAa;gBACxB,WAAW,EAAE,2BAA2B;gBACxC,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,qBAAqB;gBAC1B,iBAAiB,EAAE,iDAAiD;aACrE,CAAC;YACF,cAAc,EAAE,IAAA,gCAAgB,EAAC;gBAC/B,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,kBAAkB;gBAC7B,WAAW,EAAE,4CAA4C;gBACzD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,sCAAsC;gBAC3C,iBAAiB,EAAE,iDAAiD;aACrE,CAAC;SACH;QACD,MAAM,EAAE;YACN;gBACE,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACV,iBAAiB,EAAE;wBACjB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,CAAC,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC;wBAC7F,WAAW,EAAE,yBAAyB;qBACvC;oBACD,eAAe,EAAE;wBACf,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,IAAI;wBACb,WAAW,EAAE,8BAA8B;qBAC5C;oBACD,kBAAkB,EAAE;wBAClB,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,IAAI;wBACb,WAAW,EAAE,sBAAsB;qBACpC;oBACD,iBAAiB,EAAE;wBACjB,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,IAAI;wBACb,WAAW,EAAE,qBAAqB;qBACnC;iBACF;gBACD,oBAAoB,EAAE,KAAK;aAC5B;SACF;KACF;IACD,cAAc,EAAE;QACd;YACE,iBAAiB,EAAE,CAAC,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC;YACvG,eAAe,EAAE,IAAI;YACrB,kBAAkB,EAAE,IAAI;YACxB,iBAAiB,EAAE,IAAI;SACxB;KACF;IACD,MAAM,CAAC,OAAsD,EAAE,CAAC,OAAO,GAAG,EAAE,CAAC;QAC3E,MAAM,EACV,iBAAiB,GAAG,CAAC,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC,EAClG,eAAe,GAAG,IAAI,EACtB,kBAAkB,GAAG,IAAI,GAE9B,GAAY,OAAO,IAAI,EAAE,CAAC;QAEvB;;WAEG;QACH,SAAS,mBAAmB,CAAC,IAA6B;YACxD,qDAAqD;YACrD,MAAM,aAAa,GAAG,CAAC,GAAG,EAAE;gBAC1B,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB,EAAE,CAAC;oBAC5C,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC;oBAClC,MAAM,QAAQ,GAAG,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC;oBACtC,IAAI,QAAQ,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;wBACnC,MAAM,UAAU,GAAG,QAAQ,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;wBAC/C,IAAI,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;4BAC5E,sCAAsC;4BACtC,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gCACjC,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;gCAC1C,IAAI,OAAO,KAAK,SAAS,IAAI,OAAO,KAAK,QAAQ,EAAE,CAAC;oCAClD,OAAO,IAAI,CAAC;gCACd,CAAC;4BACH,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;qBAAM,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;oBAC7C,kCAAkC;oBAClC,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;oBAClD,IAAI,UAAU,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,UAAU,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;wBAChE,OAAO,IAAI,CAAC;oBACd,CAAC;gBACH,CAAC;gBACD,OAAO,KAAK,CAAC;YACf,CAAC,CAAC,EAAE,CAAC;YAEL,IAAI,aAAa,IAAI,eAAe,EAAE,CAAC;gBAErC,gDAAgD;gBAChD,KAAK,MAAM,GAAG,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;oBACjC,IAAI,GAAG,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,GAAG,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;wBAC5D,MAAM,IAAI,GAAG,GAAG,CAAC,KAAK,CAAC;wBACvB,IAAI,qBAAqB,CAAC,IAAI,EAAE,iBAAiB,CAAC,EAAE,CAAC;4BACnD,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,EAAE,GAAG;gCACT,SAAS,EAAE,uBAAuB;gCAClC,IAAI,EAAE;oCACJ,OAAO,EAAE,MAAM;oCACf,QAAQ,EAAE,UAAU;iCACrB;gCACD,OAAO,EAAE;oCACP,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;oCAC5C,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;oCAC5C,EAAE,SAAS,EAAE,gBAAgB,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;iCACjD;6BACF,CAAC,CAAC;4BACH,OAAO,CAAC,4BAA4B;wBACtC,CAAC;oBACH,CAAC;yBAAM,IAAI,GAAG,CAAC,IAAI,KAAK,YAAY,IAAI,GAAG,CAAC,IAAI,EAAE,CAAC;wBACjD,MAAM,IAAI,GAAG,GAAG,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;wBACpC,IAAI,qBAAqB,CAAC,IAAI,EAAE,iBAAiB,CAAC,EAAE,CAAC;4BACnD,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,EAAE,GAAG;gCACT,SAAS,EAAE,uBAAuB;gCAClC,IAAI,EAAE;oCACJ,OAAO,EAAE,MAAM;oCACf,QAAQ,EAAE,UAAU;iCACrB;gCACD,OAAO,EAAE;oCACP,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;oCAC5C,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;oCAC5C,EAAE,SAAS,EAAE,gBAAgB,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;iCACjD;6BACF,CAAC,CAAC;4BACH,OAAO,CAAC,4BAA4B;wBACtC,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED;;WAEG;QACH,SAAS,kBAAkB,CAAC,IAA4B;YACtD,IAAI,CAAC,kBAAkB,EAAE,CAAC;gBACxB,OAAO;YACT,CAAC;YAED,IAAI,IAAI,CAAC,MAAM,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;gBACrF,sEAAsE;gBACtE,KAAK,MAAM,GAAG,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;oBACjC,IAAI,GAAG,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,GAAG,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;wBAC5D,MAAM,IAAI,GAAG,GAAG,CAAC,KAAK,CAAC;wBACvB,IAAI,qBAAqB,CAAC,IAAI,EAAE,iBAAiB,CAAC,EAAE,CAAC;4BACnD,OAAO,CAAC,MAAM,CAAC;gCACb,IAAI,EAAE,GAAG;gCACT,SAAS,EAAE,uBAAuB;gCAClC,IAAI,EAAE;oCACJ,OAAO,EAAE,gBAAgB;oCACzB,QAAQ,EAAE,UAAU;iCACrB;gCACD,OAAO,EAAE;oCACP,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;oCAC5C,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;oCAC5C,EAAE,SAAS,EAAE,gBAAgB,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;iCACjD;6BACF,CAAC,CAAC;4BACH,OAAO,CAAC,6BAA6B;wBACvC,CAAC;oBACH,CAAC;yBAAM,IAAI,GAAG,CAAC,IAAI,KAAK,kBAAkB,IAAI,GAAG,CAAC,QAAQ,KAAK,GAAG,EAAE,CAAC;wBACnE,oCAAoC;wBACpC,IAAI,GAAG,CAAC,IAAI,IAAI,GAAG,CAAC,IAAI,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,GAAG,CAAC,IAAI,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;4BAClF,MAAM,QAAQ,GAAG,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC;4BAChC,IAAI,qBAAqB,CAAC,QAAQ,EAAE,iBAAiB,CAAC,EAAE,CAAC;gCACvD,OAAO,CAAC,MAAM,CAAC;oCACb,IAAI,EAAE,GAAG,CAAC,IAAI;oCACd,SAAS,EAAE,uBAAuB;oCAClC,IAAI,EAAE;wCACJ,OAAO,EAAE,gBAAgB;wCACzB,QAAQ,EAAE,UAAU;qCACrB;oCACD,OAAO,EAAE;wCACP,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;wCAC5C,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;wCAC5C,EAAE,SAAS,EAAE,gBAAgB,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;qCACjD;iCACF,CAAC,CAAC;gCACH,OAAO,CAAC,6BAA6B;4BACvC,CAAC;wBACH,CAAC;wBACD,yCAAyC;wBACzC,IAAI,GAAG,CAAC,KAAK,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,KAAK,YAAY,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;4BACnE,MAAM,SAAS,GAAG,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;4BAC/C,IAAI,qBAAqB,CAAC,SAAS,EAAE,iBAAiB,CAAC,EAAE,CAAC;gCACxD,OAAO,CAAC,MAAM,CAAC;oCACb,IAAI,EAAE,GAAG,CAAC,KAAK;oCACf,SAAS,EAAE,uBAAuB;oCAClC,IAAI,EAAE;wCACJ,OAAO,EAAE,gBAAgB;wCACzB,QAAQ,EAAE,UAAU;qCACrB;oCACD,OAAO,EAAE;wCACP,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;wCAC5C,EAAE,SAAS,EAAE,YAAY,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;wCAC5C,EAAE,SAAS,EAAE,gBAAgB,EAAE,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,EAAE;qCACjD;iCACF,CAAC,CAAC;gCACH,OAAO,CAAC,6BAA6B;4BACvC,CAAC;wBACH,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO;YACL,cAAc,EAAE,mBAAmB;YACnC,aAAa,EAAE,kBAAkB;SAClC,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}
@@ -0,0 +1,10 @@
1
+ import { type SecurityRuleOptions } from '@interlace/eslint-devkit';
2
+ export interface Options extends SecurityRuleOptions {
3
+ /** Allow dynamic table names in queries. Default: false (stricter) */
4
+ allowDynamicTableNames?: boolean;
5
+ /** Functions considered safe for building queries */
6
+ trustedFunctions?: string[];
7
+ /** Strategy for fixing SQL injection: 'parameterize', 'orm', 'sanitize', or 'auto' */
8
+ strategy?: 'parameterize' | 'orm' | 'sanitize' | 'auto';
9
+ }
10
+ export declare const noSqlInjection: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
@@ -0,0 +1,332 @@
1
+ "use strict";
2
+ Object.defineProperty(exports, "__esModule", { value: true });
3
+ exports.noSqlInjection = void 0;
4
+ const eslint_devkit_1 = require("@interlace/eslint-devkit");
5
+ const eslint_devkit_2 = require("@interlace/eslint-devkit");
6
+ const eslint_devkit_3 = require("@interlace/eslint-devkit");
7
+ exports.noSqlInjection = (0, eslint_devkit_1.createRule)({
8
+ name: 'no-sql-injection',
9
+ meta: {
10
+ type: 'problem',
11
+ docs: {
12
+ description: 'Prevent SQL injection vulnerabilities with educational context',
13
+ },
14
+ fixable: 'code',
15
+ hasSuggestions: true,
16
+ messages: {
17
+ // Enterprise-grade message with auto-enriched OWASP 2025, CVSS, and compliance data
18
+ sqlInjection: (0, eslint_devkit_2.formatLLMMessage)({
19
+ icon: eslint_devkit_2.MessageIcons.SECURITY,
20
+ issueName: 'SQL Injection',
21
+ cwe: 'CWE-89', // Auto-enriches: A05:2025, CVSS:9.8, [SOC2,PCI-DSS,HIPAA,ISO27001]
22
+ description: 'SQL Injection detected',
23
+ severity: 'CRITICAL',
24
+ fix: 'Use parameterized query: db.query("SELECT * FROM users WHERE id = ?", [userId])',
25
+ documentationLink: 'https://owasp.org/Top10/2025/A05_2025-Injection/',
26
+ }),
27
+ useParameterized: (0, eslint_devkit_2.formatLLMMessage)({
28
+ icon: eslint_devkit_2.MessageIcons.INFO,
29
+ issueName: 'Use Parameterized Query',
30
+ description: 'Use parameterized query',
31
+ severity: 'LOW',
32
+ fix: 'db.query("SELECT * FROM users WHERE id = ?", [userId])',
33
+ documentationLink: 'https://owasp.org/www-community/attacks/SQL_Injection',
34
+ }),
35
+ useORM: (0, eslint_devkit_2.formatLLMMessage)({
36
+ icon: eslint_devkit_2.MessageIcons.INFO,
37
+ issueName: 'Use ORM',
38
+ description: 'Use ORM/Query Builder',
39
+ severity: 'LOW',
40
+ fix: 'db.user.findWhere({ id: userId })',
41
+ documentationLink: 'https://www.prisma.io/docs',
42
+ }),
43
+ strategyParameterize: (0, eslint_devkit_2.formatLLMMessage)({
44
+ icon: eslint_devkit_2.MessageIcons.STRATEGY,
45
+ issueName: 'Parameterize Strategy',
46
+ description: 'Use parameterized queries',
47
+ severity: 'LOW',
48
+ fix: 'Use ? or :name placeholders',
49
+ documentationLink: 'https://owasp.org/www-community/attacks/SQL_Injection',
50
+ }),
51
+ strategyORM: (0, eslint_devkit_2.formatLLMMessage)({
52
+ icon: eslint_devkit_2.MessageIcons.STRATEGY,
53
+ issueName: 'ORM Strategy',
54
+ description: 'Migrate to ORM for automatic protection',
55
+ severity: 'LOW',
56
+ fix: 'Use Prisma, TypeORM, or Sequelize',
57
+ documentationLink: 'https://www.prisma.io/docs',
58
+ }),
59
+ strategySanitize: (0, eslint_devkit_2.formatLLMMessage)({
60
+ icon: eslint_devkit_2.MessageIcons.STRATEGY,
61
+ issueName: 'Sanitize Strategy',
62
+ description: 'Add input sanitization (last resort)',
63
+ severity: 'LOW',
64
+ fix: 'Sanitize input as last resort',
65
+ documentationLink: 'https://owasp.org/www-community/attacks/SQL_Injection',
66
+ }),
67
+ },
68
+ schema: [
69
+ {
70
+ type: 'object',
71
+ properties: {
72
+ allowDynamicTableNames: {
73
+ type: 'boolean',
74
+ default: false,
75
+ },
76
+ trustedFunctions: {
77
+ type: 'array',
78
+ items: { type: 'string' },
79
+ default: [],
80
+ },
81
+ strategy: {
82
+ type: 'string',
83
+ enum: ['parameterize', 'orm', 'sanitize', 'auto'],
84
+ default: 'auto',
85
+ description: 'Strategy for fixing SQL injection (auto = smart detection)'
86
+ },
87
+ // Security utilities options for false positive reduction
88
+ trustedSanitizers: {
89
+ type: 'array',
90
+ items: { type: 'string' },
91
+ default: [],
92
+ description: 'Additional function names to consider as sanitizers',
93
+ },
94
+ trustedAnnotations: {
95
+ type: 'array',
96
+ items: { type: 'string' },
97
+ default: [],
98
+ description: 'Additional JSDoc annotations to consider as safe markers (e.g., @safe)',
99
+ },
100
+ trustedOrmPatterns: {
101
+ type: 'array',
102
+ items: { type: 'string' },
103
+ default: [],
104
+ description: 'Additional ORM patterns to consider safe',
105
+ },
106
+ strictMode: {
107
+ type: 'boolean',
108
+ default: false,
109
+ description: 'Disable all false positive detection (strict mode)',
110
+ },
111
+ },
112
+ additionalProperties: false,
113
+ },
114
+ ],
115
+ },
116
+ defaultOptions: [
117
+ {
118
+ allowDynamicTableNames: false,
119
+ trustedFunctions: [],
120
+ strategy: 'auto',
121
+ trustedSanitizers: [],
122
+ trustedAnnotations: [],
123
+ trustedOrmPatterns: [],
124
+ strictMode: false,
125
+ },
126
+ ],
127
+ create(context) {
128
+ const opts = context.options[0] || {};
129
+ const { allowDynamicTableNames = false, trustedSanitizers = [], trustedAnnotations = [], trustedOrmPatterns = [], strictMode = false, } = opts;
130
+ const sourceCode = context.sourceCode || context.sourceCode;
131
+ const filename = context.filename || context.getFilename();
132
+ // Create safety checker for false positive detection
133
+ const safetyChecker = (0, eslint_devkit_3.createSafetyChecker)({
134
+ trustedSanitizers,
135
+ trustedAnnotations,
136
+ trustedOrmPatterns,
137
+ strictMode,
138
+ });
139
+ /**
140
+ * Check if a node contains SQL keywords
141
+ */
142
+ const containsSQLKeywords = (node) => {
143
+ // Use getText for all node types - it works correctly for template literals
144
+ const text = sourceCode.getText(node).toLowerCase();
145
+ const sqlKeywords = [
146
+ 'select',
147
+ 'insert',
148
+ 'update',
149
+ 'delete',
150
+ 'drop',
151
+ 'create',
152
+ 'alter',
153
+ 'exec',
154
+ 'execute',
155
+ ];
156
+ return sqlKeywords.some((keyword) => text.includes(keyword));
157
+ };
158
+ /**
159
+ * Check if string interpolation contains user input
160
+ */
161
+ const containsUnsafeInterpolation = (node) => {
162
+ if (node.type === 'TemplateLiteral') {
163
+ // Template literal with expressions is potentially unsafe
164
+ return node.expressions.length > 0;
165
+ }
166
+ if (node.type === 'BinaryExpression' && node.operator === '+') {
167
+ // String concatenation with variables is unsafe
168
+ return (node.left.type !== 'Literal' || node.right.type !== 'Literal');
169
+ }
170
+ return false;
171
+ };
172
+ /**
173
+ * Check if template literal only has dynamic table name (if option allows it)
174
+ */
175
+ const isOnlyDynamicTableName = (node) => {
176
+ if (!allowDynamicTableNames) {
177
+ return false;
178
+ }
179
+ // Must have exactly one expression (the table name)
180
+ if (node.expressions.length !== 1) {
181
+ return false;
182
+ }
183
+ // Check if the pattern is just "SELECT * FROM ${tableName}" (no WHERE clause)
184
+ // sourceCode.getText(node) returns the template literal with backticks
185
+ const text = sourceCode.getText(node).toLowerCase();
186
+ // Pattern: `SELECT * FROM ${tableName}` - must match the full template literal
187
+ // The text will be like: `select * from ${tablename}`
188
+ const tableNameOnlyPattern = /^`select\s+\*\s+from\s+\$\{[^}]+\}`$/;
189
+ // Also check if it has WHERE or other clauses (still unsafe)
190
+ const hasWhereOrOtherClauses = /where|order\s+by|group\s+by|having|limit|offset/i.test(text);
191
+ return !hasWhereOrOtherClauses && tableNameOnlyPattern.test(text);
192
+ };
193
+ /**
194
+ * Check if all interpolated expressions in a template literal are safe
195
+ */
196
+ const areAllExpressionsSafe = (node) => {
197
+ return node.expressions.every((expr) => {
198
+ // Check if the expression is sanitized or has safe annotation
199
+ if (safetyChecker.isSafe(expr, context)) {
200
+ return true;
201
+ }
202
+ // Check if expression is from a sanitization call directly
203
+ if ((0, eslint_devkit_3.isSanitizedInput)(expr, context, trustedSanitizers)) {
204
+ return true;
205
+ }
206
+ return false;
207
+ });
208
+ };
209
+ /**
210
+ * Find the parent statement (VariableDeclaration, ExpressionStatement, etc.)
211
+ */
212
+ const findParentStatement = (node) => {
213
+ let current = node;
214
+ while (current?.parent) {
215
+ if (current.parent.type === 'VariableDeclaration' ||
216
+ current.parent.type === 'ExpressionStatement' ||
217
+ current.parent.type === 'ReturnStatement') {
218
+ return current.parent;
219
+ }
220
+ current = current.parent;
221
+ }
222
+ return null;
223
+ };
224
+ /**
225
+ * Check if the parent call is using an ORM or parameterized query
226
+ */
227
+ const isInSafeContext = (node) => {
228
+ // Check if parent is an ORM call
229
+ let current = node;
230
+ while (current) {
231
+ if (current.type === 'CallExpression') {
232
+ if ((0, eslint_devkit_3.isOrmMethodCall)(current, context, trustedOrmPatterns)) {
233
+ return true;
234
+ }
235
+ }
236
+ current = current.parent;
237
+ }
238
+ // Check for safe annotation on containing function
239
+ if ((0, eslint_devkit_3.hasSafeAnnotation)(node, context, trustedAnnotations)) {
240
+ return true;
241
+ }
242
+ return false;
243
+ };
244
+ return {
245
+ // Check template literals
246
+ TemplateLiteral(node) {
247
+ if (!containsSQLKeywords(node) ||
248
+ !containsUnsafeInterpolation(node)) {
249
+ return;
250
+ }
251
+ // Allow dynamic table names if option is set and it's only a table name
252
+ if (isOnlyDynamicTableName(node)) {
253
+ return;
254
+ }
255
+ // FALSE POSITIVE REDUCTION:
256
+ // Skip if all interpolated expressions are sanitized
257
+ if (areAllExpressionsSafe(node)) {
258
+ return;
259
+ }
260
+ // Skip if in a safe context (ORM call, @safe annotation)
261
+ if (isInSafeContext(node)) {
262
+ return;
263
+ }
264
+ const queryText = sourceCode.getText(node);
265
+ // Find the parent statement to understand context
266
+ const parentStatement = findParentStatement(node);
267
+ const isInVariableDeclaration = parentStatement?.type === 'VariableDeclaration';
268
+ context.report({
269
+ node,
270
+ messageId: 'sqlInjection',
271
+ data: {
272
+ filePath: filename,
273
+ line: String(node.loc?.start.line ?? 0),
274
+ },
275
+ suggest: [
276
+ {
277
+ messageId: 'useParameterized',
278
+ fix: (fixer) => {
279
+ // Generate parameterized version
280
+ const params = [];
281
+ let paramIndex = 1;
282
+ const parameterized = queryText.replace(/\$\{([^}]+)\}/g, (_, expr) => {
283
+ params.push(expr);
284
+ return `$${paramIndex++}`;
285
+ });
286
+ // If assigned to a variable, wrap in db.query() call
287
+ // This produces valid JavaScript syntax
288
+ if (isInVariableDeclaration && parentStatement) {
289
+ return fixer.replaceText(parentStatement, `${sourceCode.getText(parentStatement).split('=')[0].trim()} = db.query(${parameterized}, [${params.join(', ')}])`);
290
+ }
291
+ // For standalone template literals, wrap in db.query()
292
+ return fixer.replaceText(node, `db.query(${parameterized}, [${params.join(', ')}])`);
293
+ },
294
+ },
295
+ ],
296
+ });
297
+ },
298
+ // Check binary expressions (string concatenation)
299
+ BinaryExpression(node) {
300
+ if (node.operator !== '+')
301
+ return;
302
+ if (!containsSQLKeywords(node))
303
+ return;
304
+ if (!containsUnsafeInterpolation(node))
305
+ return;
306
+ // FALSE POSITIVE REDUCTION:
307
+ // Check if in a safe context (ORM call, @safe annotation)
308
+ if (isInSafeContext(node)) {
309
+ return;
310
+ }
311
+ // Check if concatenated values are sanitized
312
+ const checkSide = (side) => {
313
+ if (side.type === 'Literal')
314
+ return true;
315
+ return safetyChecker.isSafe(side, context);
316
+ };
317
+ if (checkSide(node.left) && checkSide(node.right)) {
318
+ return;
319
+ }
320
+ context.report({
321
+ node,
322
+ messageId: 'sqlInjection',
323
+ data: {
324
+ filePath: filename,
325
+ line: String(node.loc?.start.line ?? 0),
326
+ },
327
+ });
328
+ },
329
+ };
330
+ },
331
+ });
332
+ //# sourceMappingURL=no-sql-injection.js.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"no-sql-injection.js","sourceRoot":"","sources":["../../../../../../packages/eslint-plugin-secure-coding/src/rules/security/no-sql-injection.ts"],"names":[],"mappings":";;;AAYA,4DAAsD;AACtD,4DAA0E;AAC1E,4DAMkC;AAiBrB,QAAA,cAAc,GAAG,IAAA,0BAAU,EAA0B;IAChE,IAAI,EAAE,kBAAkB;IACxB,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EACT,gEAAgE;SACnE;QACD,OAAO,EAAE,MAAM;QACf,cAAc,EAAE,IAAI;QACpB,QAAQ,EAAE;YACR,oFAAoF;YACpF,YAAY,EAAE,IAAA,gCAAgB,EAAC;gBAC7B,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,eAAe;gBAC1B,GAAG,EAAE,QAAQ,EAAE,mEAAmE;gBAClF,WAAW,EAAE,wBAAwB;gBACrC,QAAQ,EAAE,UAAU;gBACpB,GAAG,EAAE,iFAAiF;gBACtF,iBAAiB,EAAE,kDAAkD;aACtE,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,yBAAyB;gBACpC,WAAW,EAAE,yBAAyB;gBACtC,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,wDAAwD;gBAC7D,iBAAiB,EAAE,uDAAuD;aAC3E,CAAC;YACF,MAAM,EAAE,IAAA,gCAAgB,EAAC;gBACvB,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,SAAS;gBACpB,WAAW,EAAE,uBAAuB;gBACpC,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,mCAAmC;gBACxC,iBAAiB,EAAE,4BAA4B;aAChD,CAAC;YACF,oBAAoB,EAAE,IAAA,gCAAgB,EAAC;gBACrC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,uBAAuB;gBAClC,WAAW,EAAE,2BAA2B;gBACxC,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,6BAA6B;gBAClC,iBAAiB,EAAE,uDAAuD;aAC3E,CAAC;YACF,WAAW,EAAE,IAAA,gCAAgB,EAAC;gBAC5B,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,cAAc;gBACzB,WAAW,EAAE,yCAAyC;gBACtD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,mCAAmC;gBACxC,iBAAiB,EAAE,4BAA4B;aAChD,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,mBAAmB;gBAC9B,WAAW,EAAE,sCAAsC;gBACnD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,+BAA+B;gBACpC,iBAAiB,EAAE,uDAAuD;aAC3E,CAAC;SACH;QACD,MAAM,EAAE;YACN;gBACE,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACV,sBAAsB,EAAE;wBACtB,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,KAAK;qBACf;oBACD,gBAAgB,EAAE;wBAChB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;qBACZ;oBACD,QAAQ,EAAE;wBACR,IAAI,EAAE,QAAQ;wBACd,IAAI,EAAE,CAAC,cAAc,EAAE,KAAK,EAAE,UAAU,EAAE,MAAM,CAAC;wBACjD,OAAO,EAAE,MAAM;wBACf,WAAW,EAAE,4DAA4D;qBAC1E;oBACD,0DAA0D;oBAC1D,iBAAiB,EAAE;wBACjB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,qDAAqD;qBACnE;oBACD,kBAAkB,EAAE;wBAClB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,wEAAwE;qBACtF;oBACD,kBAAkB,EAAE;wBAClB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,0CAA0C;qBACxD;oBACD,UAAU,EAAE;wBACV,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,KAAK;wBACd,WAAW,EAAE,oDAAoD;qBAClE;iBACF;gBACD,oBAAoB,EAAE,KAAK;aAC5B;SACF;KACF;IACD,cAAc,EAAE;QACd;YACE,sBAAsB,EAAE,KAAK;YAC7B,gBAAgB,EAAE,EAAE;YACpB,QAAQ,EAAE,MAAM;YAChB,iBAAiB,EAAE,EAAE;YACrB,kBAAkB,EAAE,EAAE;YACtB,kBAAkB,EAAE,EAAE;YACtB,UAAU,EAAE,KAAK;SAClB;KACF;IACD,MAAM,CAAC,OAAsD;QAC3D,MAAM,IAAI,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACtC,MAAM,EACJ,sBAAsB,GAAG,KAAK,EAC9B,iBAAiB,GAAG,EAAE,EACtB,kBAAkB,GAAG,EAAE,EACvB,kBAAkB,GAAG,EAAE,EACvB,UAAU,GAAG,KAAK,GACnB,GAAG,IAAI,CAAC;QAET,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;QAC5D,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;QAE3D,qDAAqD;QACrD,MAAM,aAAa,GAAG,IAAA,mCAAmB,EAAC;YACxC,iBAAiB;YACjB,kBAAkB;YAClB,kBAAkB;YAClB,UAAU;SACX,CAAC,CAAC;QAGH;;WAEG;QACH,MAAM,mBAAmB,GAAG,CAAC,IAAmB,EAAW,EAAE;YAC3D,4EAA4E;YAC5E,MAAM,IAAI,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,WAAW,EAAE,CAAC;YAEpD,MAAM,WAAW,GAAG;gBAClB,QAAQ;gBACR,QAAQ;gBACR,QAAQ;gBACR,QAAQ;gBACR,MAAM;gBACN,QAAQ;gBACR,OAAO;gBACP,MAAM;gBACN,SAAS;aACV,CAAC;YACF,OAAO,WAAW,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC;QAC/D,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,2BAA2B,GAAG,CAClC,IAA0D,EACjD,EAAE;YACX,IAAI,IAAI,CAAC,IAAI,KAAK,iBAAiB,EAAE,CAAC;gBACpC,0DAA0D;gBAC1D,OAAO,IAAI,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC;YACrC,CAAC;YAED,IAAI,IAAI,CAAC,IAAI,KAAK,kBAAkB,IAAI,IAAI,CAAC,QAAQ,KAAK,GAAG,EAAE,CAAC;gBAC9D,gDAAgD;gBAChD,OAAO,CACL,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,SAAS,IAAI,IAAI,CAAC,KAAK,CAAC,IAAI,KAAK,SAAS,CAC9D,CAAC;YACJ,CAAC;YAED,OAAO,KAAK,CAAC;QACf,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,sBAAsB,GAAG,CAAC,IAA8B,EAAW,EAAE;YACzE,IAAI,CAAC,sBAAsB,EAAE,CAAC;gBAC5B,OAAO,KAAK,CAAC;YACf,CAAC;YAED,oDAAoD;YACpD,IAAI,IAAI,CAAC,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBAClC,OAAO,KAAK,CAAC;YACf,CAAC;YAED,8EAA8E;YAC9E,uEAAuE;YACvE,MAAM,IAAI,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,WAAW,EAAE,CAAC;YAEpD,+EAA+E;YAC/E,sDAAsD;YACtD,MAAM,oBAAoB,GAAG,sCAAsC,CAAC;YAEpE,6DAA6D;YAC7D,MAAM,sBAAsB,GAAG,kDAAkD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAE7F,OAAO,CAAC,sBAAsB,IAAI,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACpE,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,qBAAqB,GAAG,CAAC,IAA8B,EAAW,EAAE;YACxE,OAAO,IAAI,CAAC,WAAW,CAAC,KAAK,CAAC,CAAC,IAAyB,EAAE,EAAE;gBAC1D,8DAA8D;gBAC9D,IAAI,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,OAAO,CAAC,EAAE,CAAC;oBACxC,OAAO,IAAI,CAAC;gBACd,CAAC;gBAED,2DAA2D;gBAC3D,IAAI,IAAA,gCAAgB,EAAC,IAAI,EAAE,OAAO,EAAE,iBAAiB,CAAC,EAAE,CAAC;oBACvD,OAAO,IAAI,CAAC;gBACd,CAAC;gBAED,OAAO,KAAK,CAAC;YACf,CAAC,CAAC,CAAC;QACL,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,mBAAmB,GAAG,CAAC,IAAmB,EAAwB,EAAE;YACxE,IAAI,OAAO,GAA8B,IAAI,CAAC;YAC9C,OAAO,OAAO,EAAE,MAAM,EAAE,CAAC;gBACvB,IACE,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,qBAAqB;oBAC7C,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,qBAAqB;oBAC7C,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,iBAAiB,EACzC,CAAC;oBACD,OAAO,OAAO,CAAC,MAAM,CAAC;gBACxB,CAAC;gBACD,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC;YAC3B,CAAC;YACD,OAAO,IAAI,CAAC;QACd,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,eAAe,GAAG,CAAC,IAAmB,EAAW,EAAE;YACvD,iCAAiC;YACjC,IAAI,OAAO,GAA8B,IAAI,CAAC;YAC9C,OAAO,OAAO,EAAE,CAAC;gBACf,IAAI,OAAO,CAAC,IAAI,KAAK,gBAAgB,EAAE,CAAC;oBACtC,IAAI,IAAA,+BAAe,EAAC,OAAO,EAAE,OAAO,EAAE,kBAAkB,CAAC,EAAE,CAAC;wBAC1D,OAAO,IAAI,CAAC;oBACd,CAAC;gBACH,CAAC;gBACD,OAAO,GAAG,OAAO,CAAC,MAAmC,CAAC;YACxD,CAAC;YAED,mDAAmD;YACnD,IAAI,IAAA,iCAAiB,EAAC,IAAI,EAAE,OAAO,EAAE,kBAAkB,CAAC,EAAE,CAAC;gBACzD,OAAO,IAAI,CAAC;YACd,CAAC;YAED,OAAO,KAAK,CAAC;QACf,CAAC,CAAC;QAEF,OAAO;YACL,0BAA0B;YAC1B,eAAe,CAAC,IAA8B;gBAC5C,IACE,CAAC,mBAAmB,CAAC,IAAI,CAAC;oBAC1B,CAAC,2BAA2B,CAAC,IAAI,CAAC,EAClC,CAAC;oBACD,OAAO;gBACT,CAAC;gBAED,wEAAwE;gBACxE,IAAI,sBAAsB,CAAC,IAAI,CAAC,EAAE,CAAC;oBACjC,OAAO;gBACT,CAAC;gBAED,4BAA4B;gBAC5B,qDAAqD;gBACrD,IAAI,qBAAqB,CAAC,IAAI,CAAC,EAAE,CAAC;oBAChC,OAAO;gBACT,CAAC;gBAED,yDAAyD;gBACzD,IAAI,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC1B,OAAO;gBACT,CAAC;gBAED,MAAM,SAAS,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;gBAE3C,kDAAkD;gBAClD,MAAM,eAAe,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;gBAClD,MAAM,uBAAuB,GAAG,eAAe,EAAE,IAAI,KAAK,qBAAqB,CAAC;gBAEhF,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAE,cAAc;oBACzB,IAAI,EAAE;wBACJ,QAAQ,EAAE,QAAQ;wBAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;qBACxC;oBACD,OAAO,EAAE;wBACP;4BACE,SAAS,EAAE,kBAAkB;4BAC7B,GAAG,EAAE,CAAC,KAAyB,EAAE,EAAE;gCACjC,iCAAiC;gCACjC,MAAM,MAAM,GAAa,EAAE,CAAC;gCAC5B,IAAI,UAAU,GAAG,CAAC,CAAC;gCAEnB,MAAM,aAAa,GAAG,SAAS,CAAC,OAAO,CACrC,gBAAgB,EAChB,CAAC,CAAS,EAAE,IAAY,EAAE,EAAE;oCAC1B,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;oCAClB,OAAO,IAAI,UAAU,EAAE,EAAE,CAAC;gCAC5B,CAAC,CACF,CAAC;gCAEF,qDAAqD;gCACrD,wCAAwC;gCACxC,IAAI,uBAAuB,IAAI,eAAe,EAAE,CAAC;oCAC/C,OAAO,KAAK,CAAC,WAAW,CACtB,eAAe,EACf,GAAG,UAAU,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,eAAe,aAAa,MAAM,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CACnH,CAAC;gCACJ,CAAC;gCAED,uDAAuD;gCACvD,OAAO,KAAK,CAAC,WAAW,CACtB,IAAI,EACJ,YAAY,aAAa,MAAM,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CACrD,CAAC;4BACJ,CAAC;yBACF;qBACF;iBACF,CAAC,CAAC;YACL,CAAC;YAED,kDAAkD;YAClD,gBAAgB,CAAC,IAA+B;gBAC9C,IAAI,IAAI,CAAC,QAAQ,KAAK,GAAG;oBAAE,OAAO;gBAElC,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC;oBAAE,OAAO;gBACvC,IAAI,CAAC,2BAA2B,CAAC,IAAI,CAAC;oBAAE,OAAO;gBAE/C,4BAA4B;gBAC5B,0DAA0D;gBAC1D,IAAI,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC1B,OAAO;gBACT,CAAC;gBAED,6CAA6C;gBAC7C,MAAM,SAAS,GAAG,CAAC,IAAmB,EAAW,EAAE;oBACjD,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS;wBAAE,OAAO,IAAI,CAAC;oBACzC,OAAO,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;gBAC7C,CAAC,CAAC;gBAEF,IAAI,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;oBAClD,OAAO;gBACT,CAAC;gBAED,OAAO,CAAC,MAAM,CAAC;oBACb,IAAI;oBACJ,SAAS,EAAE,cAAc;oBACzB,IAAI,EAAE;wBACJ,QAAQ,EAAE,QAAQ;wBAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;qBACxC;iBACF,CAAC,CAAC;YACL,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}
@@ -0,0 +1,10 @@
1
+ import { type SecurityRuleOptions } from '@interlace/eslint-devkit';
2
+ export interface Options extends SecurityRuleOptions {
3
+ /** Functions that are considered authentication-related */
4
+ authFunctions?: string[];
5
+ /** Variables that contain sensitive/auth data */
6
+ sensitiveVariables?: string[];
7
+ /** Allow early returns in non-sensitive contexts */
8
+ allowEarlyReturns?: boolean;
9
+ }
10
+ export declare const noTimingAttack: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;