eslint-plugin-secure-coding 3.0.0 → 3.0.2

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 (158) hide show
  1. package/AGENTS.md +1 -1
  2. package/CHANGELOG.md +1 -1
  3. package/README.md +90 -422
  4. package/package.json +6 -5
  5. package/src/index.d.ts +5 -14
  6. package/src/index.js +36 -265
  7. package/src/rules/detect-non-literal-regexp/index.d.ts +20 -1
  8. package/src/rules/detect-non-literal-regexp/index.js +5 -0
  9. package/src/rules/detect-object-injection/index.d.ts +25 -1
  10. package/src/rules/detect-object-injection/index.js +5 -0
  11. package/src/rules/detect-weak-password-validation/index.d.ts +8 -2
  12. package/src/rules/detect-weak-password-validation/index.js +6 -1
  13. package/src/rules/no-directive-injection/index.d.ts +27 -1
  14. package/src/rules/no-directive-injection/index.js +5 -0
  15. package/src/rules/no-electron-security-issues/index.d.ts +27 -1
  16. package/src/rules/no-electron-security-issues/index.js +5 -0
  17. package/src/rules/no-format-string-injection/index.d.ts +28 -1
  18. package/src/rules/no-format-string-injection/index.js +5 -0
  19. package/src/rules/no-graphql-injection/index.d.ts +29 -1
  20. package/src/rules/no-graphql-injection/index.js +5 -0
  21. package/src/rules/no-hardcoded-credentials/index.d.ts +19 -1
  22. package/src/rules/no-hardcoded-credentials/index.js +5 -0
  23. package/src/rules/no-hardcoded-session-tokens/index.d.ts +8 -2
  24. package/src/rules/no-hardcoded-session-tokens/index.js +6 -1
  25. package/src/rules/no-improper-sanitization/index.d.ts +27 -1
  26. package/src/rules/no-improper-sanitization/index.js +5 -0
  27. package/src/rules/no-improper-type-validation/index.d.ts +27 -1
  28. package/src/rules/no-improper-type-validation/index.js +5 -0
  29. package/src/rules/no-insecure-comparison/index.d.ts +20 -1
  30. package/src/rules/no-insecure-comparison/index.js +5 -0
  31. package/src/rules/no-ldap-injection/index.d.ts +30 -1
  32. package/src/rules/no-ldap-injection/index.js +5 -0
  33. package/src/rules/no-missing-authentication/index.d.ts +20 -1
  34. package/src/rules/no-missing-authentication/index.js +5 -1
  35. package/src/rules/no-pii-in-logs/index.d.ts +8 -4
  36. package/src/rules/no-pii-in-logs/index.js +15 -12
  37. package/src/rules/no-privilege-escalation/index.d.ts +20 -1
  38. package/src/rules/no-privilege-escalation/index.js +5 -0
  39. package/src/rules/no-redos-vulnerable-regex/index.d.ts +22 -1
  40. package/src/rules/no-redos-vulnerable-regex/index.js +5 -0
  41. package/src/rules/no-sensitive-data-exposure/index.d.ts +20 -1
  42. package/src/rules/no-sensitive-data-exposure/index.js +5 -0
  43. package/src/rules/no-unchecked-loop-condition/index.d.ts +27 -1
  44. package/src/rules/no-unchecked-loop-condition/index.js +5 -0
  45. package/src/rules/no-unlimited-resource-allocation/index.d.ts +27 -1
  46. package/src/rules/no-unlimited-resource-allocation/index.js +5 -0
  47. package/src/rules/no-unsafe-deserialization/index.d.ts +31 -1
  48. package/src/rules/no-unsafe-deserialization/index.js +5 -0
  49. package/src/rules/no-unsafe-regex-construction/index.d.ts +22 -1
  50. package/src/rules/no-unsafe-regex-construction/index.js +5 -0
  51. package/src/rules/no-weak-password-recovery/index.d.ts +27 -1
  52. package/src/rules/no-weak-password-recovery/index.js +5 -0
  53. package/src/rules/no-xpath-injection/index.d.ts +30 -1
  54. package/src/rules/no-xpath-injection/index.js +5 -0
  55. package/src/rules/no-xxe-injection/index.d.ts +30 -1
  56. package/src/rules/no-xxe-injection/index.js +5 -0
  57. package/src/rules/require-backend-authorization/index.d.ts +8 -2
  58. package/src/rules/require-backend-authorization/index.js +6 -1
  59. package/src/rules/require-secure-defaults/index.d.ts +8 -4
  60. package/src/rules/require-secure-defaults/index.js +7 -6
  61. package/src/types/index.d.ts +10 -52
  62. package/src/types/index.js +3 -12
  63. package/src/rules/detect-child-process/index.d.ts +0 -11
  64. package/src/rules/detect-child-process/index.js +0 -529
  65. package/src/rules/detect-eval-with-expression/index.d.ts +0 -9
  66. package/src/rules/detect-eval-with-expression/index.js +0 -392
  67. package/src/rules/detect-mixed-content/index.d.ts +0 -8
  68. package/src/rules/detect-mixed-content/index.js +0 -44
  69. package/src/rules/detect-non-literal-fs-filename/index.d.ts +0 -7
  70. package/src/rules/detect-non-literal-fs-filename/index.js +0 -454
  71. package/src/rules/detect-suspicious-dependencies/index.d.ts +0 -8
  72. package/src/rules/detect-suspicious-dependencies/index.js +0 -71
  73. package/src/rules/no-allow-arbitrary-loads/index.d.ts +0 -8
  74. package/src/rules/no-allow-arbitrary-loads/index.js +0 -47
  75. package/src/rules/no-arbitrary-file-access/index.d.ts +0 -13
  76. package/src/rules/no-arbitrary-file-access/index.js +0 -195
  77. package/src/rules/no-buffer-overread/index.d.ts +0 -29
  78. package/src/rules/no-buffer-overread/index.js +0 -606
  79. package/src/rules/no-clickjacking/index.d.ts +0 -10
  80. package/src/rules/no-clickjacking/index.js +0 -396
  81. package/src/rules/no-client-side-auth-logic/index.d.ts +0 -6
  82. package/src/rules/no-client-side-auth-logic/index.js +0 -69
  83. package/src/rules/no-credentials-in-query-params/index.d.ts +0 -8
  84. package/src/rules/no-credentials-in-query-params/index.js +0 -57
  85. package/src/rules/no-data-in-temp-storage/index.d.ts +0 -6
  86. package/src/rules/no-data-in-temp-storage/index.js +0 -64
  87. package/src/rules/no-debug-code-in-production/index.d.ts +0 -8
  88. package/src/rules/no-debug-code-in-production/index.js +0 -51
  89. package/src/rules/no-disabled-certificate-validation/index.d.ts +0 -6
  90. package/src/rules/no-disabled-certificate-validation/index.js +0 -61
  91. package/src/rules/no-dynamic-dependency-loading/index.d.ts +0 -8
  92. package/src/rules/no-dynamic-dependency-loading/index.js +0 -51
  93. package/src/rules/no-exposed-debug-endpoints/index.d.ts +0 -6
  94. package/src/rules/no-exposed-debug-endpoints/index.js +0 -62
  95. package/src/rules/no-exposed-sensitive-data/index.d.ts +0 -11
  96. package/src/rules/no-exposed-sensitive-data/index.js +0 -340
  97. package/src/rules/no-http-urls/index.d.ts +0 -12
  98. package/src/rules/no-http-urls/index.js +0 -114
  99. package/src/rules/no-insecure-redirects/index.d.ts +0 -7
  100. package/src/rules/no-insecure-redirects/index.js +0 -216
  101. package/src/rules/no-insecure-websocket/index.d.ts +0 -6
  102. package/src/rules/no-insecure-websocket/index.js +0 -61
  103. package/src/rules/no-missing-cors-check/index.d.ts +0 -9
  104. package/src/rules/no-missing-cors-check/index.js +0 -399
  105. package/src/rules/no-missing-csrf-protection/index.d.ts +0 -11
  106. package/src/rules/no-missing-csrf-protection/index.js +0 -180
  107. package/src/rules/no-missing-security-headers/index.d.ts +0 -7
  108. package/src/rules/no-missing-security-headers/index.js +0 -218
  109. package/src/rules/no-password-in-url/index.d.ts +0 -8
  110. package/src/rules/no-password-in-url/index.js +0 -54
  111. package/src/rules/no-permissive-cors/index.d.ts +0 -8
  112. package/src/rules/no-permissive-cors/index.js +0 -65
  113. package/src/rules/no-sensitive-data-in-analytics/index.d.ts +0 -8
  114. package/src/rules/no-sensitive-data-in-analytics/index.js +0 -62
  115. package/src/rules/no-sensitive-data-in-cache/index.d.ts +0 -8
  116. package/src/rules/no-sensitive-data-in-cache/index.js +0 -52
  117. package/src/rules/no-toctou-vulnerability/index.d.ts +0 -7
  118. package/src/rules/no-toctou-vulnerability/index.js +0 -208
  119. package/src/rules/no-tracking-without-consent/index.d.ts +0 -6
  120. package/src/rules/no-tracking-without-consent/index.js +0 -67
  121. package/src/rules/no-unencrypted-transmission/index.d.ts +0 -11
  122. package/src/rules/no-unencrypted-transmission/index.js +0 -236
  123. package/src/rules/no-unescaped-url-parameter/index.d.ts +0 -9
  124. package/src/rules/no-unescaped-url-parameter/index.js +0 -355
  125. package/src/rules/no-unsafe-dynamic-require/index.d.ts +0 -5
  126. package/src/rules/no-unsafe-dynamic-require/index.js +0 -106
  127. package/src/rules/no-unvalidated-deeplinks/index.d.ts +0 -6
  128. package/src/rules/no-unvalidated-deeplinks/index.js +0 -62
  129. package/src/rules/no-unvalidated-user-input/index.d.ts +0 -9
  130. package/src/rules/no-unvalidated-user-input/index.js +0 -420
  131. package/src/rules/no-verbose-error-messages/index.d.ts +0 -8
  132. package/src/rules/no-verbose-error-messages/index.js +0 -68
  133. package/src/rules/no-zip-slip/index.d.ts +0 -9
  134. package/src/rules/no-zip-slip/index.js +0 -445
  135. package/src/rules/require-code-minification/index.d.ts +0 -8
  136. package/src/rules/require-code-minification/index.js +0 -47
  137. package/src/rules/require-csp-headers/index.d.ts +0 -6
  138. package/src/rules/require-csp-headers/index.js +0 -64
  139. package/src/rules/require-data-minimization/index.d.ts +0 -8
  140. package/src/rules/require-data-minimization/index.js +0 -53
  141. package/src/rules/require-dependency-integrity/index.d.ts +0 -6
  142. package/src/rules/require-dependency-integrity/index.js +0 -64
  143. package/src/rules/require-https-only/index.d.ts +0 -8
  144. package/src/rules/require-https-only/index.js +0 -62
  145. package/src/rules/require-mime-type-validation/index.d.ts +0 -6
  146. package/src/rules/require-mime-type-validation/index.js +0 -66
  147. package/src/rules/require-network-timeout/index.d.ts +0 -8
  148. package/src/rules/require-network-timeout/index.js +0 -50
  149. package/src/rules/require-package-lock/index.d.ts +0 -8
  150. package/src/rules/require-package-lock/index.js +0 -63
  151. package/src/rules/require-secure-credential-storage/index.d.ts +0 -8
  152. package/src/rules/require-secure-credential-storage/index.js +0 -50
  153. package/src/rules/require-secure-deletion/index.d.ts +0 -8
  154. package/src/rules/require-secure-deletion/index.js +0 -44
  155. package/src/rules/require-storage-encryption/index.d.ts +0 -8
  156. package/src/rules/require-storage-encryption/index.js +0 -50
  157. package/src/rules/require-url-validation/index.d.ts +0 -6
  158. package/src/rules/require-url-validation/index.js +0 -72
@@ -1,396 +0,0 @@
1
- "use strict";
2
- Object.defineProperty(exports, "__esModule", { value: true });
3
- exports.noClickjacking = 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.noClickjacking = (0, eslint_devkit_1.createRule)({
8
- name: 'no-clickjacking',
9
- meta: {
10
- type: 'problem',
11
- deprecated: true,
12
- replacedBy: ['@see eslint-plugin-express-security/require-helmet'],
13
- docs: {
14
- description: 'Detects clickjacking vulnerabilities and missing frame protections',
15
- },
16
- fixable: 'code',
17
- hasSuggestions: true,
18
- messages: {
19
- clickjackingVulnerability: (0, eslint_devkit_2.formatLLMMessage)({
20
- icon: eslint_devkit_2.MessageIcons.SECURITY,
21
- issueName: 'Clickjacking Vulnerability',
22
- cwe: 'CWE-1021',
23
- description: 'Clickjacking protection missing',
24
- severity: '{{severity}}',
25
- fix: '{{safeAlternative}}',
26
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
27
- }),
28
- missingFrameBusting: (0, eslint_devkit_2.formatLLMMessage)({
29
- icon: eslint_devkit_2.MessageIcons.SECURITY,
30
- issueName: 'Missing Frame Busting',
31
- cwe: 'CWE-1021',
32
- description: 'No frame-busting code to prevent clickjacking',
33
- severity: 'HIGH',
34
- fix: 'Add frame-busting JavaScript to prevent framing',
35
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
36
- }),
37
- unsafeIframeUsage: (0, eslint_devkit_2.formatLLMMessage)({
38
- icon: eslint_devkit_2.MessageIcons.SECURITY,
39
- issueName: 'Unsafe iframe Usage',
40
- cwe: 'CWE-1021',
41
- description: 'iframe may enable clickjacking attacks',
42
- severity: 'MEDIUM',
43
- fix: 'Add X-Frame-Options or CSP frame-ancestors protection',
44
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
45
- }),
46
- missingXFrameOptions: (0, eslint_devkit_2.formatLLMMessage)({
47
- icon: eslint_devkit_2.MessageIcons.SECURITY,
48
- issueName: 'Missing X-Frame-Options',
49
- cwe: 'CWE-1021',
50
- description: 'X-Frame-Options header not set',
51
- severity: 'HIGH',
52
- fix: 'Set X-Frame-Options: DENY or SAMEORIGIN',
53
- documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Frame-Options',
54
- }),
55
- missingCspFrameAncestors: (0, eslint_devkit_2.formatLLMMessage)({
56
- icon: eslint_devkit_2.MessageIcons.SECURITY,
57
- issueName: 'Missing CSP frame-ancestors',
58
- cwe: 'CWE-1021',
59
- description: 'CSP frame-ancestors directive not configured',
60
- severity: 'HIGH',
61
- fix: 'Add frame-ancestors to Content-Security-Policy',
62
- documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/frame-ancestors',
63
- }),
64
- transparentFrameOverlay: (0, eslint_devkit_2.formatLLMMessage)({
65
- icon: eslint_devkit_2.MessageIcons.SECURITY,
66
- issueName: 'Transparent Frame Overlay',
67
- cwe: 'CWE-1021',
68
- description: 'Transparent elements may hide clickjacking attacks',
69
- severity: 'MEDIUM',
70
- fix: 'Use frame-busting or CSP protections',
71
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
72
- }),
73
- frameManipulation: (0, eslint_devkit_2.formatLLMMessage)({
74
- icon: eslint_devkit_2.MessageIcons.SECURITY,
75
- issueName: 'Frame Manipulation',
76
- cwe: 'CWE-1021',
77
- description: 'Code attempts to manipulate parent frames',
78
- severity: 'LOW',
79
- fix: 'Implement proper frame communication or prevent framing',
80
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
81
- }),
82
- implementFrameBusting: (0, eslint_devkit_2.formatLLMMessage)({
83
- icon: eslint_devkit_2.MessageIcons.INFO,
84
- issueName: 'Implement Frame Busting',
85
- description: 'Add JavaScript to prevent framing',
86
- severity: 'LOW',
87
- fix: 'if (top != self) top.location = location;',
88
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
89
- }),
90
- useXFrameOptions: (0, eslint_devkit_2.formatLLMMessage)({
91
- icon: eslint_devkit_2.MessageIcons.INFO,
92
- issueName: 'Use X-Frame-Options',
93
- description: 'Set X-Frame-Options HTTP header',
94
- severity: 'LOW',
95
- fix: 'X-Frame-Options: DENY',
96
- documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Frame-Options',
97
- }),
98
- setCspFrameAncestors: (0, eslint_devkit_2.formatLLMMessage)({
99
- icon: eslint_devkit_2.MessageIcons.INFO,
100
- issueName: 'Set CSP frame-ancestors',
101
- description: 'Configure CSP frame-ancestors directive',
102
- severity: 'LOW',
103
- fix: 'frame-ancestors \'self\' https://trusted.com',
104
- documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/frame-ancestors',
105
- }),
106
- strategyFrameProtection: (0, eslint_devkit_2.formatLLMMessage)({
107
- icon: eslint_devkit_2.MessageIcons.STRATEGY,
108
- issueName: 'Frame Protection Strategy',
109
- description: 'Implement multiple layers of frame protection',
110
- severity: 'LOW',
111
- fix: 'Use X-Frame-Options, CSP, and frame-busting together',
112
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
113
- }),
114
- strategyContentSecurity: (0, eslint_devkit_2.formatLLMMessage)({
115
- icon: eslint_devkit_2.MessageIcons.STRATEGY,
116
- issueName: 'Content Security Strategy',
117
- description: 'Use CSP for comprehensive frame control',
118
- severity: 'LOW',
119
- fix: 'Implement strict CSP with frame-ancestors',
120
- documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/HTTP/CSP',
121
- }),
122
- strategyUserInteraction: (0, eslint_devkit_2.formatLLMMessage)({
123
- icon: eslint_devkit_2.MessageIcons.STRATEGY,
124
- issueName: 'User Interaction Strategy',
125
- description: 'Protect user interactions from framing attacks',
126
- severity: 'LOW',
127
- fix: 'Validate user intent for sensitive actions',
128
- documentationLink: 'https://cheatsheetseries.owasp.org/cheatsheets/Clickjacking_Defense_Cheat_Sheet.html',
129
- })
130
- },
131
- schema: [
132
- {
133
- type: 'object',
134
- properties: {
135
- trustedSources: {
136
- type: 'array',
137
- items: { type: 'string' },
138
- default: ['self', 'same-origin'],
139
- },
140
- requireFrameBusting: {
141
- type: 'boolean',
142
- default: true,
143
- },
144
- detectTransparentOverlays: {
145
- type: 'boolean',
146
- default: true,
147
- },
148
- trustedSanitizers: {
149
- type: 'array',
150
- items: { type: 'string' },
151
- default: [],
152
- description: 'Additional function names to consider as frame protectors',
153
- },
154
- trustedAnnotations: {
155
- type: 'array',
156
- items: { type: 'string' },
157
- default: [],
158
- description: 'Additional JSDoc annotations to consider as safe markers',
159
- },
160
- strictMode: {
161
- type: 'boolean',
162
- default: false,
163
- description: 'Disable all false positive detection (strict mode)',
164
- },
165
- },
166
- additionalProperties: false,
167
- },
168
- ],
169
- },
170
- defaultOptions: [
171
- {
172
- trustedSources: ['self', 'same-origin'],
173
- requireFrameBusting: true,
174
- detectTransparentOverlays: true,
175
- trustedSanitizers: [],
176
- trustedAnnotations: [],
177
- strictMode: false,
178
- },
179
- ],
180
- create(context) {
181
- const options = context.options[0] || {};
182
- const { trustedSources = ['self', 'same-origin'], requireFrameBusting = true, detectTransparentOverlays = true, trustedSanitizers = [], trustedAnnotations = [], strictMode = false, } = options;
183
- const sourceCode = context.sourceCode || context.sourceCode;
184
- const filename = context.filename || context.getFilename();
185
- // Create safety checker for false positive detection
186
- const safetyChecker = (0, eslint_devkit_3.createSafetyChecker)({
187
- trustedSanitizers,
188
- trustedAnnotations,
189
- trustedOrmPatterns: [],
190
- strictMode,
191
- });
192
- // Track if frame-busting code is present
193
- let hasFrameBusting = false;
194
- /**
195
- * Check if source is trusted
196
- */
197
- const isTrustedSource = (source) => {
198
- return trustedSources.some(trusted => source.includes(trusted) ||
199
- (trusted === 'self' && (source === 'self' || source.startsWith('/'))) ||
200
- (trusted === 'same-origin' && source === 'same-origin'));
201
- };
202
- /**
203
- * Check if this is frame-busting code
204
- */
205
- const isFrameBustingCode = (node) => {
206
- const test = node.test;
207
- const testText = sourceCode.getText(test).toLowerCase();
208
- // Look for common frame-busting patterns
209
- return testText.includes('top != self') ||
210
- testText.includes('top !== self') ||
211
- testText.includes('window.top !== window.self') ||
212
- testText.includes('parent != self') ||
213
- testText.includes('top.location') ||
214
- testText.includes('self.location');
215
- };
216
- /**
217
- * Check for transparent/invisible elements that could hide clickjacking
218
- */
219
- const hasTransparentStyles = (styleText) => {
220
- const styles = styleText.toLowerCase();
221
- return styles.includes('opacity: 0') ||
222
- styles.includes('opacity:0') ||
223
- styles.includes('visibility: hidden') ||
224
- styles.includes('display: none') ||
225
- styles.includes('z-index: -1') ||
226
- styles.includes('position: absolute') && styles.includes('top: 0') && styles.includes('left: 0');
227
- };
228
- return {
229
- // Check for frame-busting code
230
- IfStatement(node) {
231
- if (isFrameBustingCode(node)) {
232
- hasFrameBusting = true;
233
- }
234
- },
235
- // Check iframe elements (in JSX/TSX)
236
- JSXElement(node) {
237
- if (node.openingElement.name.type === 'JSXIdentifier' &&
238
- node.openingElement.name.name === 'iframe') {
239
- // Check iframe attributes
240
- const attributes = node.openingElement.attributes;
241
- let hasSrc = false;
242
- let srcValue = '';
243
- for (const attr of attributes) {
244
- if (attr.type === 'JSXAttribute' &&
245
- attr.name.type === 'JSXIdentifier' &&
246
- attr.name.name === 'src' &&
247
- attr.value) {
248
- hasSrc = true;
249
- if (attr.value.type === 'Literal' && typeof attr.value.value === 'string') {
250
- srcValue = attr.value.value;
251
- }
252
- }
253
- }
254
- if (hasSrc && srcValue && !isTrustedSource(srcValue)) {
255
- /* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
256
- if (safetyChecker.isSafe(node, context)) {
257
- return;
258
- }
259
- /* c8 ignore stop */
260
- context.report({
261
- node: node.openingElement,
262
- messageId: 'unsafeIframeUsage',
263
- data: {
264
- filePath: filename,
265
- line: String(node.loc?.start.line ?? 0),
266
- },
267
- });
268
- }
269
- }
270
- },
271
- // Check for frame manipulation code
272
- MemberExpression(node) {
273
- // Look for top.location or window.top manipulation
274
- if (node.object.type === 'Identifier' &&
275
- (node.object.name === 'top' || node.object.name === 'window')) {
276
- if (node.property.type === 'Identifier' &&
277
- (node.property.name === 'location' || node.property.name === 'top')) {
278
- // Check if this is being assigned or compared
279
- let current = node;
280
- let isFrameManipulation = false;
281
- // Walk up to see if this is an assignment or comparison
282
- while (current && !isFrameManipulation) {
283
- if (current.type === 'AssignmentExpression' &&
284
- current.left === node) {
285
- isFrameManipulation = true;
286
- break;
287
- }
288
- if (current.type === 'BinaryExpression' &&
289
- (current.left === node || current.right === node)) {
290
- // Comparison like top != self
291
- const operator = current.operator;
292
- if (operator === '!=' || operator === '!==' ||
293
- operator === '==' || operator === '===') {
294
- // This might be frame-busting code
295
- break;
296
- }
297
- isFrameManipulation = true;
298
- break;
299
- }
300
- current = current.parent;
301
- }
302
- if (isFrameManipulation) {
303
- /* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
304
- if (safetyChecker.isSafe(node, context)) {
305
- return;
306
- }
307
- /* c8 ignore stop */
308
- context.report({
309
- node,
310
- messageId: 'frameManipulation',
311
- data: {
312
- filePath: filename,
313
- line: String(node.loc?.start.line ?? 0),
314
- },
315
- });
316
- }
317
- }
318
- }
319
- },
320
- // Check for CSS that could hide clickjacking attacks
321
- Literal(node) {
322
- if (typeof node.value === 'string' && detectTransparentOverlays) {
323
- // Check if this looks like CSS
324
- const text = node.value.toLowerCase();
325
- if ((text.includes('style=') || text.includes('css')) &&
326
- hasTransparentStyles(text)) {
327
- /* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
328
- if (safetyChecker.isSafe(node, context)) {
329
- return;
330
- }
331
- /* c8 ignore stop */
332
- context.report({
333
- node,
334
- messageId: 'transparentFrameOverlay',
335
- data: {
336
- filePath: filename,
337
- line: String(node.loc?.start.line ?? 0),
338
- },
339
- });
340
- }
341
- }
342
- },
343
- // Check template literals for CSS
344
- TemplateLiteral(node) {
345
- if (detectTransparentOverlays) {
346
- const text = sourceCode.getText(node).toLowerCase();
347
- if (text.includes('style') && hasTransparentStyles(text)) {
348
- /* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
349
- if (safetyChecker.isSafe(node, context)) {
350
- return;
351
- }
352
- /* c8 ignore stop */
353
- context.report({
354
- node,
355
- messageId: 'transparentFrameOverlay',
356
- data: {
357
- filePath: filename,
358
- line: String(node.loc?.start.line ?? 0),
359
- },
360
- });
361
- }
362
- }
363
- },
364
- // At the end of the file, check if frame-busting is required but missing
365
- 'Program:exit'() {
366
- if (requireFrameBusting && !hasFrameBusting) {
367
- // Only check files that are likely entry points or render HTML
368
- const isEntryPoint = /\.(html|htm)$/.test(filename) ||
369
- /(index|app|main|page)\.(tsx|jsx)$/i.test(filename) ||
370
- /pages?\/.*\.(tsx|jsx)$/i.test(filename) ||
371
- /layout\.(tsx|jsx)$/i.test(filename);
372
- // Skip non-entry point files
373
- if (!isEntryPoint) {
374
- return;
375
- }
376
- // Check if this file has actual UI rendering (JSX elements with event handlers)
377
- const fileContent = sourceCode.getText();
378
- const hasUIElements = fileContent.includes('<button') ||
379
- fileContent.includes('<form') ||
380
- fileContent.includes('<input') ||
381
- (fileContent.includes('onClick') && fileContent.includes('<'));
382
- if (hasUIElements) {
383
- context.report({
384
- node: context.sourceCode.ast,
385
- messageId: 'missingFrameBusting',
386
- data: {
387
- filePath: filename,
388
- line: '1',
389
- },
390
- });
391
- }
392
- }
393
- }
394
- };
395
- },
396
- });
@@ -1,6 +0,0 @@
1
- /**
2
- * @fileoverview Prevent authentication logic in client code
3
- */
4
- export interface Options {
5
- }
6
- export declare const noClientSideAuthLogic: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
@@ -1,69 +0,0 @@
1
- "use strict";
2
- /**
3
- * @fileoverview Prevent authentication logic in client code
4
- */
5
- Object.defineProperty(exports, "__esModule", { value: true });
6
- exports.noClientSideAuthLogic = void 0;
7
- const eslint_devkit_1 = require("@interlace/eslint-devkit");
8
- exports.noClientSideAuthLogic = (0, eslint_devkit_1.createRule)({
9
- name: 'no-client-side-auth-logic',
10
- meta: {
11
- type: 'problem',
12
- docs: {
13
- description: 'Prevent authentication logic in client code',
14
- },
15
- messages: {
16
- violationDetected: (0, eslint_devkit_1.formatLLMMessage)({
17
- icon: eslint_devkit_1.MessageIcons.SECURITY,
18
- issueName: 'Client-Side Auth Logic',
19
- cwe: 'CWE-602',
20
- description: 'Authentication logic in client code - easily bypassed',
21
- severity: 'CRITICAL',
22
- fix: 'Move authentication checks to the server',
23
- documentationLink: 'https://cwe.mitre.org/data/definitions/602.html',
24
- })
25
- },
26
- schema: [],
27
- },
28
- defaultOptions: [],
29
- create(context) {
30
- function report(node) {
31
- context.report({ node, messageId: 'violationDetected' });
32
- }
33
- const authKeywords = ['admin', 'authenticated', 'authorized', 'isAdmin', 'isAuthenticated', 'role'];
34
- return {
35
- IfStatement(node) {
36
- // Detect role/auth checks from localStorage
37
- if (node.test.type === 'CallExpression' &&
38
- node.test.callee.type === 'MemberExpression' &&
39
- node.test.callee.object.type === 'Identifier' &&
40
- node.test.callee.object.name === 'localStorage' &&
41
- node.test.callee.property.type === 'Identifier' &&
42
- node.test.callee.property.name === 'getItem') {
43
- const keyArg = node.test.arguments[0];
44
- if (keyArg && keyArg.type === 'Literal') {
45
- const key = String(keyArg.value).toLowerCase();
46
- if (authKeywords.some(kw => key.includes(kw))) {
47
- report(node);
48
- }
49
- }
50
- }
51
- // Detect password comparison
52
- if (node.test.type === 'BinaryExpression') {
53
- const checkMember = (expr) => {
54
- if (expr.type === 'MemberExpression' &&
55
- expr.property.type === 'Identifier' &&
56
- ['password', 'secret', 'token'].includes(expr.property.name)) {
57
- return true;
58
- }
59
- return false;
60
- };
61
- if (checkMember(node.test.left) ||
62
- checkMember(node.test.right)) {
63
- report(node);
64
- }
65
- }
66
- },
67
- };
68
- },
69
- });
@@ -1,8 +0,0 @@
1
- /**
2
- * @fileoverview Disallow credentials in URL query parameters
3
- * @see https://owasp.org/www-project-mobile-top-10/
4
- * @see https://cwe.mitre.org/data/definitions/598.html
5
- */
6
- export interface Options {
7
- }
8
- export declare const noCredentialsInQueryParams: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
@@ -1,57 +0,0 @@
1
- "use strict";
2
- /**
3
- * @fileoverview Disallow credentials in URL query parameters
4
- * @see https://owasp.org/www-project-mobile-top-10/
5
- * @see https://cwe.mitre.org/data/definitions/598.html
6
- */
7
- Object.defineProperty(exports, "__esModule", { value: true });
8
- exports.noCredentialsInQueryParams = void 0;
9
- const eslint_devkit_1 = require("@interlace/eslint-devkit");
10
- exports.noCredentialsInQueryParams = (0, eslint_devkit_1.createRule)({
11
- name: 'no-credentials-in-query-params',
12
- meta: {
13
- type: 'problem',
14
- docs: {
15
- description: 'Disallow credentials in URL query parameters',
16
- },
17
- messages: {
18
- violationDetected: (0, eslint_devkit_1.formatLLMMessage)({
19
- icon: eslint_devkit_1.MessageIcons.SECURITY,
20
- issueName: 'Credentials in Query Parameters',
21
- cwe: 'CWE-798',
22
- description: 'Credentials detected in URL query parameters - this is a security risk',
23
- severity: 'CRITICAL',
24
- fix: 'Use secure methods: POST body, headers (Authorization), or secure cookies',
25
- documentationLink: 'https://cwe.mitre.org/data/definitions/798.html',
26
- })
27
- },
28
- schema: [],
29
- },
30
- defaultOptions: [],
31
- create(context) {
32
- const sourceCode = context.sourceCode;
33
- const sensitiveParams = ['password=', 'token=', 'apikey=', 'secret=', 'auth='];
34
- function report(node) {
35
- context.report({
36
- node,
37
- messageId: 'violationDetected',
38
- });
39
- }
40
- return {
41
- Literal(node) {
42
- if (typeof node.value === 'string') {
43
- const url = node.value.toLowerCase();
44
- if (sensitiveParams.some(param => url.includes('?' + param) || url.includes('&' + param))) {
45
- report(node);
46
- }
47
- }
48
- },
49
- TemplateLiteral(node) {
50
- const text = sourceCode.getText(node).toLowerCase();
51
- if (sensitiveParams.some(param => text.includes(param))) {
52
- report(node);
53
- }
54
- },
55
- };
56
- },
57
- });
@@ -1,6 +0,0 @@
1
- /**
2
- * @fileoverview Prevent sensitive data in temp directories
3
- */
4
- export interface Options {
5
- }
6
- export declare const noDataInTempStorage: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
@@ -1,64 +0,0 @@
1
- "use strict";
2
- /**
3
- * @fileoverview Prevent sensitive data in temp directories
4
- */
5
- Object.defineProperty(exports, "__esModule", { value: true });
6
- exports.noDataInTempStorage = void 0;
7
- const eslint_devkit_1 = require("@interlace/eslint-devkit");
8
- exports.noDataInTempStorage = (0, eslint_devkit_1.createRule)({
9
- name: 'no-data-in-temp-storage',
10
- meta: {
11
- type: 'problem',
12
- docs: {
13
- description: 'Prevent sensitive data in temp directories',
14
- },
15
- messages: {
16
- violationDetected: (0, eslint_devkit_1.formatLLMMessage)({
17
- icon: eslint_devkit_1.MessageIcons.SECURITY,
18
- issueName: 'Temp Storage Data',
19
- cwe: 'CWE-312',
20
- description: 'Sensitive data written to temp directory - not secure',
21
- severity: 'HIGH',
22
- fix: 'Use secure storage location or encrypt data before writing',
23
- documentationLink: 'https://cwe.mitre.org/data/definitions/312.html',
24
- })
25
- },
26
- schema: [],
27
- },
28
- defaultOptions: [],
29
- create(context) {
30
- function report(node) {
31
- context.report({ node, messageId: 'violationDetected' });
32
- }
33
- const tempPaths = ['/tmp', '/var/tmp', 'temp/', '/temp'];
34
- return {
35
- CallExpression(node) {
36
- // Detect fs.writeFileSync or fs.writeFile with temp path
37
- if (node.callee.type === 'MemberExpression' &&
38
- node.callee.object.type === 'Identifier' &&
39
- node.callee.object.name === 'fs' &&
40
- node.callee.property.type === 'Identifier' &&
41
- ['writeFileSync', 'writeFile'].includes(node.callee.property.name)) {
42
- const pathArg = node.arguments[0];
43
- if (pathArg && pathArg.type === 'Literal' && typeof pathArg.value === 'string') {
44
- if (tempPaths.some(tp => pathArg.value.includes(tp))) {
45
- report(node);
46
- }
47
- }
48
- }
49
- },
50
- Literal(node) {
51
- // Detect temp path literals
52
- if (typeof node.value === 'string') {
53
- if (tempPaths.some(tp => node.value.includes(tp))) {
54
- // Only flag if parent is assignment or variable declaration
55
- const parent = node.parent;
56
- if (parent?.type === 'VariableDeclarator' || parent?.type === 'AssignmentExpression') {
57
- report(node);
58
- }
59
- }
60
- }
61
- },
62
- };
63
- },
64
- });
@@ -1,8 +0,0 @@
1
- /**
2
- * @fileoverview Detect debug code in production
3
- * @see https://owasp.org/www-project-mobile-top-10/
4
- * @see https://cwe.mitre.org/data/definitions/489.html
5
- */
6
- export interface Options {
7
- }
8
- export declare const noDebugCodeInProduction: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;