eslint-plugin-secure-coding 2.3.2 → 2.3.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +1 -0
- package/package.json +3 -10
- package/src/index.ts +605 -0
- package/src/rules/__tests__/integration-demo.test.ts +290 -0
- package/src/rules/__tests__/integration-llm.test.ts +89 -0
- package/src/rules/database-injection/database-injection.test.ts +456 -0
- package/src/rules/database-injection/index.ts +488 -0
- package/src/rules/detect-child-process/detect-child-process.test.ts +207 -0
- package/src/rules/detect-child-process/index.ts +634 -0
- package/src/rules/detect-eval-with-expression/detect-eval-with-expression.test.ts +416 -0
- package/src/rules/detect-eval-with-expression/index.ts +463 -0
- package/src/rules/detect-mixed-content/detect-mixed-content.test.ts +28 -0
- package/src/rules/detect-mixed-content/index.ts +52 -0
- package/src/rules/detect-non-literal-fs-filename/detect-non-literal-fs-filename.test.ts +269 -0
- package/src/rules/detect-non-literal-fs-filename/index.ts +551 -0
- package/src/rules/detect-non-literal-regexp/detect-non-literal-regexp.test.ts +189 -0
- package/src/rules/detect-non-literal-regexp/index.ts +490 -0
- package/src/rules/detect-object-injection/detect-object-injection.test.ts +440 -0
- package/src/rules/detect-object-injection/index.ts +674 -0
- package/src/rules/detect-suspicious-dependencies/detect-suspicious-dependencies.test.ts +32 -0
- package/src/rules/detect-suspicious-dependencies/index.ts +84 -0
- package/src/rules/detect-weak-password-validation/detect-weak-password-validation.test.ts +31 -0
- package/src/rules/detect-weak-password-validation/index.ts +68 -0
- package/src/rules/no-allow-arbitrary-loads/index.ts +54 -0
- package/src/rules/no-allow-arbitrary-loads/no-allow-arbitrary-loads.test.ts +28 -0
- package/src/rules/no-arbitrary-file-access/index.ts +238 -0
- package/src/rules/no-arbitrary-file-access/no-arbitrary-file-access.test.ts +119 -0
- package/src/rules/no-buffer-overread/index.ts +724 -0
- package/src/rules/no-buffer-overread/no-buffer-overread.test.ts +313 -0
- package/src/rules/no-clickjacking/index.ts +481 -0
- package/src/rules/no-clickjacking/no-clickjacking.test.ts +253 -0
- package/src/rules/no-client-side-auth-logic/index.ts +81 -0
- package/src/rules/no-client-side-auth-logic/no-client-side-auth-logic.test.ts +33 -0
- package/src/rules/no-credentials-in-query-params/index.ts +69 -0
- package/src/rules/no-credentials-in-query-params/no-credentials-in-query-params.test.ts +33 -0
- package/src/rules/no-credentials-in-storage-api/index.ts +64 -0
- package/src/rules/no-credentials-in-storage-api/no-credentials-in-storage-api.test.ts +31 -0
- package/src/rules/no-data-in-temp-storage/index.ts +75 -0
- package/src/rules/no-data-in-temp-storage/no-data-in-temp-storage.test.ts +33 -0
- package/src/rules/no-debug-code-in-production/index.ts +59 -0
- package/src/rules/no-debug-code-in-production/no-debug-code-in-production.test.ts +26 -0
- package/src/rules/no-directive-injection/index.ts +551 -0
- package/src/rules/no-directive-injection/no-directive-injection.test.ts +305 -0
- package/src/rules/no-disabled-certificate-validation/index.ts +72 -0
- package/src/rules/no-disabled-certificate-validation/no-disabled-certificate-validation.test.ts +33 -0
- package/src/rules/no-document-cookie/index.ts +113 -0
- package/src/rules/no-document-cookie/no-document-cookie.test.ts +382 -0
- package/src/rules/no-dynamic-dependency-loading/index.ts +60 -0
- package/src/rules/no-dynamic-dependency-loading/no-dynamic-dependency-loading.test.ts +27 -0
- package/src/rules/no-electron-security-issues/index.ts +504 -0
- package/src/rules/no-electron-security-issues/no-electron-security-issues.test.ts +324 -0
- package/src/rules/no-exposed-debug-endpoints/index.ts +73 -0
- package/src/rules/no-exposed-debug-endpoints/no-exposed-debug-endpoints.test.ts +40 -0
- package/src/rules/no-exposed-sensitive-data/index.ts +428 -0
- package/src/rules/no-exposed-sensitive-data/no-exposed-sensitive-data.test.ts +75 -0
- package/src/rules/no-format-string-injection/index.ts +801 -0
- package/src/rules/no-format-string-injection/no-format-string-injection.test.ts +437 -0
- package/src/rules/no-graphql-injection/index.ts +508 -0
- package/src/rules/no-graphql-injection/no-graphql-injection.test.ts +371 -0
- package/src/rules/no-hardcoded-credentials/index.ts +478 -0
- package/src/rules/no-hardcoded-credentials/no-hardcoded-credentials.test.ts +639 -0
- package/src/rules/no-hardcoded-session-tokens/index.ts +69 -0
- package/src/rules/no-hardcoded-session-tokens/no-hardcoded-session-tokens.test.ts +42 -0
- package/src/rules/no-http-urls/index.ts +131 -0
- package/src/rules/no-http-urls/no-http-urls.test.ts +60 -0
- package/src/rules/no-improper-sanitization/index.ts +502 -0
- package/src/rules/no-improper-sanitization/no-improper-sanitization.test.ts +156 -0
- package/src/rules/no-improper-type-validation/index.ts +572 -0
- package/src/rules/no-improper-type-validation/no-improper-type-validation.test.ts +372 -0
- package/src/rules/no-insecure-comparison/index.ts +232 -0
- package/src/rules/no-insecure-comparison/no-insecure-comparison.test.ts +218 -0
- package/src/rules/no-insecure-cookie-settings/index.ts +391 -0
- package/src/rules/no-insecure-cookie-settings/no-insecure-cookie-settings.test.ts +409 -0
- package/src/rules/no-insecure-jwt/index.ts +467 -0
- package/src/rules/no-insecure-jwt/no-insecure-jwt.test.ts +259 -0
- package/src/rules/no-insecure-redirects/index.ts +267 -0
- package/src/rules/no-insecure-redirects/no-insecure-redirects.test.ts +108 -0
- package/src/rules/no-insecure-websocket/index.ts +72 -0
- package/src/rules/no-insecure-websocket/no-insecure-websocket.test.ts +42 -0
- package/src/rules/no-insufficient-postmessage-validation/index.ts +497 -0
- package/src/rules/no-insufficient-postmessage-validation/no-insufficient-postmessage-validation.test.ts +360 -0
- package/src/rules/no-insufficient-random/index.ts +288 -0
- package/src/rules/no-insufficient-random/no-insufficient-random.test.ts +246 -0
- package/src/rules/no-ldap-injection/index.ts +547 -0
- package/src/rules/no-ldap-injection/no-ldap-injection.test.ts +317 -0
- package/src/rules/no-missing-authentication/index.ts +408 -0
- package/src/rules/no-missing-authentication/no-missing-authentication.test.ts +350 -0
- package/src/rules/no-missing-cors-check/index.ts +453 -0
- package/src/rules/no-missing-cors-check/no-missing-cors-check.test.ts +392 -0
- package/src/rules/no-missing-csrf-protection/index.ts +229 -0
- package/src/rules/no-missing-csrf-protection/no-missing-csrf-protection.test.ts +222 -0
- package/src/rules/no-missing-security-headers/index.ts +266 -0
- package/src/rules/no-missing-security-headers/no-missing-security-headers.test.ts +98 -0
- package/src/rules/no-password-in-url/index.ts +64 -0
- package/src/rules/no-password-in-url/no-password-in-url.test.ts +27 -0
- package/src/rules/no-permissive-cors/index.ts +78 -0
- package/src/rules/no-permissive-cors/no-permissive-cors.test.ts +28 -0
- package/src/rules/no-pii-in-logs/index.ts +83 -0
- package/src/rules/no-pii-in-logs/no-pii-in-logs.test.ts +26 -0
- package/src/rules/no-postmessage-origin-wildcard/index.ts +67 -0
- package/src/rules/no-postmessage-origin-wildcard/no-postmessage-origin-wildcard.test.ts +27 -0
- package/src/rules/no-privilege-escalation/index.ts +403 -0
- package/src/rules/no-privilege-escalation/no-privilege-escalation.test.ts +306 -0
- package/src/rules/no-redos-vulnerable-regex/index.ts +379 -0
- package/src/rules/no-redos-vulnerable-regex/no-redos-vulnerable-regex.test.ts +83 -0
- package/src/rules/no-sensitive-data-exposure/index.ts +294 -0
- package/src/rules/no-sensitive-data-exposure/no-sensitive-data-exposure.test.ts +262 -0
- package/src/rules/no-sensitive-data-in-analytics/index.ts +73 -0
- package/src/rules/no-sensitive-data-in-analytics/no-sensitive-data-in-analytics.test.ts +42 -0
- package/src/rules/no-sensitive-data-in-cache/index.ts +59 -0
- package/src/rules/no-sensitive-data-in-cache/no-sensitive-data-in-cache.test.ts +32 -0
- package/src/rules/no-sql-injection/index.ts +424 -0
- package/src/rules/no-sql-injection/no-sql-injection.test.ts +303 -0
- package/src/rules/no-timing-attack/index.ts +552 -0
- package/src/rules/no-timing-attack/no-timing-attack.test.ts +348 -0
- package/src/rules/no-toctou-vulnerability/index.ts +250 -0
- package/src/rules/no-toctou-vulnerability/no-toctou-vulnerability.test.ts +60 -0
- package/src/rules/no-tracking-without-consent/index.ts +78 -0
- package/src/rules/no-tracking-without-consent/no-tracking-without-consent.test.ts +34 -0
- package/src/rules/no-unchecked-loop-condition/index.ts +781 -0
- package/src/rules/no-unchecked-loop-condition/no-unchecked-loop-condition.test.ts +459 -0
- package/src/rules/no-unencrypted-local-storage/index.ts +73 -0
- package/src/rules/no-unencrypted-local-storage/no-unencrypted-local-storage.test.ts +41 -0
- package/src/rules/no-unencrypted-transmission/index.ts +296 -0
- package/src/rules/no-unencrypted-transmission/no-unencrypted-transmission.test.ts +287 -0
- package/src/rules/no-unescaped-url-parameter/index.ts +424 -0
- package/src/rules/no-unescaped-url-parameter/no-unescaped-url-parameter.test.ts +263 -0
- package/src/rules/no-unlimited-resource-allocation/index.ts +767 -0
- package/src/rules/no-unlimited-resource-allocation/no-unlimited-resource-allocation.test.ts +544 -0
- package/src/rules/no-unsafe-deserialization/index.ts +593 -0
- package/src/rules/no-unsafe-deserialization/no-unsafe-deserialization.test.ts +310 -0
- package/src/rules/no-unsafe-dynamic-require/index.ts +125 -0
- package/src/rules/no-unsafe-dynamic-require/no-unsafe-dynamic-require.test.ts +151 -0
- package/src/rules/no-unsafe-regex-construction/index.ts +370 -0
- package/src/rules/no-unsafe-regex-construction/no-unsafe-regex-construction.test.ts +181 -0
- package/src/rules/no-unsanitized-html/index.ts +400 -0
- package/src/rules/no-unsanitized-html/no-unsanitized-html.test.ts +488 -0
- package/src/rules/no-unvalidated-deeplinks/index.ts +73 -0
- package/src/rules/no-unvalidated-deeplinks/no-unvalidated-deeplinks.test.ts +29 -0
- package/src/rules/no-unvalidated-user-input/index.ts +498 -0
- package/src/rules/no-unvalidated-user-input/no-unvalidated-user-input.test.ts +463 -0
- package/src/rules/no-verbose-error-messages/index.ts +83 -0
- package/src/rules/no-verbose-error-messages/no-verbose-error-messages.test.ts +34 -0
- package/src/rules/no-weak-crypto/index.ts +447 -0
- package/src/rules/no-weak-crypto/no-weak-crypto.test.ts +297 -0
- package/src/rules/no-weak-password-recovery/index.ts +509 -0
- package/src/rules/no-weak-password-recovery/no-weak-password-recovery.test.ts +184 -0
- package/src/rules/no-xpath-injection/index.ts +596 -0
- package/src/rules/no-xpath-injection/no-xpath-injection.test.ts +405 -0
- package/src/rules/no-xxe-injection/index.ts +342 -0
- package/src/rules/no-xxe-injection/no-xxe-injection.test.ts +122 -0
- package/src/rules/no-zip-slip/index.ts +526 -0
- package/src/rules/no-zip-slip/no-zip-slip.test.ts +305 -0
- package/src/rules/require-backend-authorization/index.ts +71 -0
- package/src/rules/require-backend-authorization/require-backend-authorization.test.ts +31 -0
- package/src/rules/require-code-minification/index.ts +54 -0
- package/src/rules/require-code-minification/require-code-minification.test.ts +30 -0
- package/src/rules/require-csp-headers/index.ts +74 -0
- package/src/rules/require-csp-headers/require-csp-headers.test.ts +34 -0
- package/src/rules/require-data-minimization/index.ts +65 -0
- package/src/rules/require-data-minimization/require-data-minimization.test.ts +31 -0
- package/src/rules/require-dependency-integrity/index.ts +78 -0
- package/src/rules/require-dependency-integrity/require-dependency-integrity.test.ts +44 -0
- package/src/rules/require-https-only/index.ts +75 -0
- package/src/rules/require-https-only/require-https-only.test.ts +26 -0
- package/src/rules/require-mime-type-validation/index.ts +77 -0
- package/src/rules/require-mime-type-validation/require-mime-type-validation.test.ts +32 -0
- package/src/rules/require-network-timeout/index.ts +58 -0
- package/src/rules/require-network-timeout/require-network-timeout.test.ts +26 -0
- package/src/rules/require-package-lock/index.ts +75 -0
- package/src/rules/require-package-lock/require-package-lock.test.ts +27 -0
- package/src/rules/require-secure-credential-storage/index.ts +60 -0
- package/src/rules/require-secure-credential-storage/require-secure-credential-storage.test.ts +26 -0
- package/src/rules/require-secure-defaults/index.ts +54 -0
- package/src/rules/require-secure-defaults/require-secure-defaults.test.ts +26 -0
- package/src/rules/require-secure-deletion/index.ts +52 -0
- package/src/rules/require-secure-deletion/require-secure-deletion.test.ts +29 -0
- package/src/rules/require-storage-encryption/index.ts +60 -0
- package/src/rules/require-storage-encryption/require-storage-encryption.test.ts +26 -0
- package/src/rules/require-url-validation/index.ts +85 -0
- package/src/rules/require-url-validation/require-url-validation.test.ts +32 -0
- package/src/types/{index.d.ts → index.ts} +157 -53
- package/src/index.d.ts +0 -32
- package/src/index.js +0 -465
- package/src/rules/database-injection/index.d.ts +0 -13
- package/src/rules/database-injection/index.js +0 -406
- package/src/rules/detect-child-process/index.d.ts +0 -11
- package/src/rules/detect-child-process/index.js +0 -529
- package/src/rules/detect-eval-with-expression/index.d.ts +0 -9
- package/src/rules/detect-eval-with-expression/index.js +0 -392
- package/src/rules/detect-mixed-content/index.d.ts +0 -8
- package/src/rules/detect-mixed-content/index.js +0 -44
- package/src/rules/detect-non-literal-fs-filename/index.d.ts +0 -7
- package/src/rules/detect-non-literal-fs-filename/index.js +0 -454
- package/src/rules/detect-non-literal-regexp/index.d.ts +0 -9
- package/src/rules/detect-non-literal-regexp/index.js +0 -403
- package/src/rules/detect-object-injection/index.d.ts +0 -11
- package/src/rules/detect-object-injection/index.js +0 -560
- package/src/rules/detect-suspicious-dependencies/index.d.ts +0 -8
- package/src/rules/detect-suspicious-dependencies/index.js +0 -71
- package/src/rules/detect-weak-password-validation/index.d.ts +0 -6
- package/src/rules/detect-weak-password-validation/index.js +0 -58
- package/src/rules/no-allow-arbitrary-loads/index.d.ts +0 -8
- package/src/rules/no-allow-arbitrary-loads/index.js +0 -47
- package/src/rules/no-arbitrary-file-access/index.d.ts +0 -13
- package/src/rules/no-arbitrary-file-access/index.js +0 -195
- package/src/rules/no-buffer-overread/index.d.ts +0 -29
- package/src/rules/no-buffer-overread/index.js +0 -606
- package/src/rules/no-clickjacking/index.d.ts +0 -10
- package/src/rules/no-clickjacking/index.js +0 -396
- package/src/rules/no-client-side-auth-logic/index.d.ts +0 -6
- package/src/rules/no-client-side-auth-logic/index.js +0 -69
- package/src/rules/no-credentials-in-query-params/index.d.ts +0 -8
- package/src/rules/no-credentials-in-query-params/index.js +0 -57
- package/src/rules/no-credentials-in-storage-api/index.d.ts +0 -6
- package/src/rules/no-credentials-in-storage-api/index.js +0 -54
- package/src/rules/no-data-in-temp-storage/index.d.ts +0 -6
- package/src/rules/no-data-in-temp-storage/index.js +0 -64
- package/src/rules/no-debug-code-in-production/index.d.ts +0 -8
- package/src/rules/no-debug-code-in-production/index.js +0 -51
- package/src/rules/no-directive-injection/index.d.ts +0 -12
- package/src/rules/no-directive-injection/index.js +0 -457
- package/src/rules/no-disabled-certificate-validation/index.d.ts +0 -6
- package/src/rules/no-disabled-certificate-validation/index.js +0 -61
- package/src/rules/no-document-cookie/index.d.ts +0 -5
- package/src/rules/no-document-cookie/index.js +0 -89
- package/src/rules/no-dynamic-dependency-loading/index.d.ts +0 -8
- package/src/rules/no-dynamic-dependency-loading/index.js +0 -51
- package/src/rules/no-electron-security-issues/index.d.ts +0 -10
- package/src/rules/no-electron-security-issues/index.js +0 -423
- package/src/rules/no-exposed-debug-endpoints/index.d.ts +0 -6
- package/src/rules/no-exposed-debug-endpoints/index.js +0 -62
- package/src/rules/no-exposed-sensitive-data/index.d.ts +0 -11
- package/src/rules/no-exposed-sensitive-data/index.js +0 -340
- package/src/rules/no-format-string-injection/index.d.ts +0 -17
- package/src/rules/no-format-string-injection/index.js +0 -660
- package/src/rules/no-graphql-injection/index.d.ts +0 -12
- package/src/rules/no-graphql-injection/index.js +0 -411
- package/src/rules/no-hardcoded-credentials/index.d.ts +0 -26
- package/src/rules/no-hardcoded-credentials/index.js +0 -376
- package/src/rules/no-hardcoded-session-tokens/index.d.ts +0 -6
- package/src/rules/no-hardcoded-session-tokens/index.js +0 -59
- package/src/rules/no-http-urls/index.d.ts +0 -12
- package/src/rules/no-http-urls/index.js +0 -114
- package/src/rules/no-improper-sanitization/index.d.ts +0 -12
- package/src/rules/no-improper-sanitization/index.js +0 -411
- package/src/rules/no-improper-type-validation/index.d.ts +0 -10
- package/src/rules/no-improper-type-validation/index.js +0 -475
- package/src/rules/no-insecure-comparison/index.d.ts +0 -7
- package/src/rules/no-insecure-comparison/index.js +0 -193
- package/src/rules/no-insecure-cookie-settings/index.d.ts +0 -9
- package/src/rules/no-insecure-cookie-settings/index.js +0 -306
- package/src/rules/no-insecure-jwt/index.d.ts +0 -10
- package/src/rules/no-insecure-jwt/index.js +0 -380
- package/src/rules/no-insecure-redirects/index.d.ts +0 -7
- package/src/rules/no-insecure-redirects/index.js +0 -216
- package/src/rules/no-insecure-websocket/index.d.ts +0 -6
- package/src/rules/no-insecure-websocket/index.js +0 -61
- package/src/rules/no-insufficient-postmessage-validation/index.d.ts +0 -14
- package/src/rules/no-insufficient-postmessage-validation/index.js +0 -392
- package/src/rules/no-insufficient-random/index.d.ts +0 -9
- package/src/rules/no-insufficient-random/index.js +0 -208
- package/src/rules/no-ldap-injection/index.d.ts +0 -10
- package/src/rules/no-ldap-injection/index.js +0 -455
- package/src/rules/no-missing-authentication/index.d.ts +0 -13
- package/src/rules/no-missing-authentication/index.js +0 -333
- package/src/rules/no-missing-cors-check/index.d.ts +0 -9
- package/src/rules/no-missing-cors-check/index.js +0 -399
- package/src/rules/no-missing-csrf-protection/index.d.ts +0 -11
- package/src/rules/no-missing-csrf-protection/index.js +0 -180
- package/src/rules/no-missing-security-headers/index.d.ts +0 -7
- package/src/rules/no-missing-security-headers/index.js +0 -218
- package/src/rules/no-password-in-url/index.d.ts +0 -8
- package/src/rules/no-password-in-url/index.js +0 -54
- package/src/rules/no-permissive-cors/index.d.ts +0 -8
- package/src/rules/no-permissive-cors/index.js +0 -65
- package/src/rules/no-pii-in-logs/index.d.ts +0 -8
- package/src/rules/no-pii-in-logs/index.js +0 -70
- package/src/rules/no-postmessage-origin-wildcard/index.d.ts +0 -8
- package/src/rules/no-postmessage-origin-wildcard/index.js +0 -56
- package/src/rules/no-privilege-escalation/index.d.ts +0 -13
- package/src/rules/no-privilege-escalation/index.js +0 -321
- package/src/rules/no-redos-vulnerable-regex/index.d.ts +0 -7
- package/src/rules/no-redos-vulnerable-regex/index.js +0 -306
- package/src/rules/no-sensitive-data-exposure/index.d.ts +0 -11
- package/src/rules/no-sensitive-data-exposure/index.js +0 -250
- package/src/rules/no-sensitive-data-in-analytics/index.d.ts +0 -8
- package/src/rules/no-sensitive-data-in-analytics/index.js +0 -62
- package/src/rules/no-sensitive-data-in-cache/index.d.ts +0 -8
- package/src/rules/no-sensitive-data-in-cache/index.js +0 -52
- package/src/rules/no-sql-injection/index.d.ts +0 -10
- package/src/rules/no-sql-injection/index.js +0 -335
- package/src/rules/no-timing-attack/index.d.ts +0 -10
- package/src/rules/no-timing-attack/index.js +0 -447
- package/src/rules/no-toctou-vulnerability/index.d.ts +0 -7
- package/src/rules/no-toctou-vulnerability/index.js +0 -208
- package/src/rules/no-tracking-without-consent/index.d.ts +0 -6
- package/src/rules/no-tracking-without-consent/index.js +0 -67
- package/src/rules/no-unchecked-loop-condition/index.d.ts +0 -12
- package/src/rules/no-unchecked-loop-condition/index.js +0 -646
- package/src/rules/no-unencrypted-local-storage/index.d.ts +0 -8
- package/src/rules/no-unencrypted-local-storage/index.js +0 -61
- package/src/rules/no-unencrypted-transmission/index.d.ts +0 -11
- package/src/rules/no-unencrypted-transmission/index.js +0 -236
- package/src/rules/no-unescaped-url-parameter/index.d.ts +0 -9
- package/src/rules/no-unescaped-url-parameter/index.js +0 -355
- package/src/rules/no-unlimited-resource-allocation/index.d.ts +0 -12
- package/src/rules/no-unlimited-resource-allocation/index.js +0 -643
- package/src/rules/no-unsafe-deserialization/index.d.ts +0 -10
- package/src/rules/no-unsafe-deserialization/index.js +0 -491
- package/src/rules/no-unsafe-dynamic-require/index.d.ts +0 -5
- package/src/rules/no-unsafe-dynamic-require/index.js +0 -106
- package/src/rules/no-unsafe-regex-construction/index.d.ts +0 -9
- package/src/rules/no-unsafe-regex-construction/index.js +0 -291
- package/src/rules/no-unsanitized-html/index.d.ts +0 -9
- package/src/rules/no-unsanitized-html/index.js +0 -335
- package/src/rules/no-unvalidated-deeplinks/index.d.ts +0 -6
- package/src/rules/no-unvalidated-deeplinks/index.js +0 -62
- package/src/rules/no-unvalidated-user-input/index.d.ts +0 -9
- package/src/rules/no-unvalidated-user-input/index.js +0 -420
- package/src/rules/no-verbose-error-messages/index.d.ts +0 -8
- package/src/rules/no-verbose-error-messages/index.js +0 -68
- package/src/rules/no-weak-crypto/index.d.ts +0 -11
- package/src/rules/no-weak-crypto/index.js +0 -351
- package/src/rules/no-weak-password-recovery/index.d.ts +0 -12
- package/src/rules/no-weak-password-recovery/index.js +0 -424
- package/src/rules/no-xpath-injection/index.d.ts +0 -10
- package/src/rules/no-xpath-injection/index.js +0 -487
- package/src/rules/no-xxe-injection/index.d.ts +0 -7
- package/src/rules/no-xxe-injection/index.js +0 -266
- package/src/rules/no-zip-slip/index.d.ts +0 -9
- package/src/rules/no-zip-slip/index.js +0 -445
- package/src/rules/require-backend-authorization/index.d.ts +0 -6
- package/src/rules/require-backend-authorization/index.js +0 -60
- package/src/rules/require-code-minification/index.d.ts +0 -8
- package/src/rules/require-code-minification/index.js +0 -47
- package/src/rules/require-csp-headers/index.d.ts +0 -6
- package/src/rules/require-csp-headers/index.js +0 -64
- package/src/rules/require-data-minimization/index.d.ts +0 -8
- package/src/rules/require-data-minimization/index.js +0 -53
- package/src/rules/require-dependency-integrity/index.d.ts +0 -6
- package/src/rules/require-dependency-integrity/index.js +0 -64
- package/src/rules/require-https-only/index.d.ts +0 -8
- package/src/rules/require-https-only/index.js +0 -62
- package/src/rules/require-mime-type-validation/index.d.ts +0 -6
- package/src/rules/require-mime-type-validation/index.js +0 -66
- package/src/rules/require-network-timeout/index.d.ts +0 -8
- package/src/rules/require-network-timeout/index.js +0 -50
- package/src/rules/require-package-lock/index.d.ts +0 -8
- package/src/rules/require-package-lock/index.js +0 -63
- package/src/rules/require-secure-credential-storage/index.d.ts +0 -8
- package/src/rules/require-secure-credential-storage/index.js +0 -50
- package/src/rules/require-secure-defaults/index.d.ts +0 -8
- package/src/rules/require-secure-defaults/index.js +0 -47
- package/src/rules/require-secure-deletion/index.d.ts +0 -8
- package/src/rules/require-secure-deletion/index.js +0 -44
- package/src/rules/require-storage-encryption/index.d.ts +0 -8
- package/src/rules/require-storage-encryption/index.js +0 -50
- package/src/rules/require-url-validation/index.d.ts +0 -6
- package/src/rules/require-url-validation/index.js +0 -72
- package/src/types/index.js +0 -17
|
@@ -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,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,54 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
/**
|
|
3
|
-
* @fileoverview Disallow storing credentials in browser/mobile storage APIs
|
|
4
|
-
*/
|
|
5
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
6
|
-
exports.noCredentialsInStorageApi = void 0;
|
|
7
|
-
const eslint_devkit_1 = require("@interlace/eslint-devkit");
|
|
8
|
-
exports.noCredentialsInStorageApi = (0, eslint_devkit_1.createRule)({
|
|
9
|
-
name: 'no-credentials-in-storage-api',
|
|
10
|
-
meta: {
|
|
11
|
-
type: 'problem',
|
|
12
|
-
docs: {
|
|
13
|
-
description: 'Disallow storing credentials in browser/mobile storage APIs',
|
|
14
|
-
},
|
|
15
|
-
messages: {
|
|
16
|
-
violationDetected: (0, eslint_devkit_1.formatLLMMessage)({
|
|
17
|
-
icon: eslint_devkit_1.MessageIcons.SECURITY,
|
|
18
|
-
issueName: 'Credentials in Storage',
|
|
19
|
-
cwe: 'CWE-522',
|
|
20
|
-
description: 'Credentials stored in insecure browser/mobile storage',
|
|
21
|
-
severity: 'CRITICAL',
|
|
22
|
-
fix: 'Use secure storage like Keychain, SecureStore, or encrypted storage',
|
|
23
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/522.html',
|
|
24
|
-
})
|
|
25
|
-
},
|
|
26
|
-
schema: [],
|
|
27
|
-
},
|
|
28
|
-
defaultOptions: [],
|
|
29
|
-
create(context) {
|
|
30
|
-
function report(node) {
|
|
31
|
-
context.report({ node, messageId: 'violationDetected' });
|
|
32
|
-
}
|
|
33
|
-
const sensitiveKeys = ['password', 'token', 'apikey', 'secret', 'credential', 'auth', 'key'];
|
|
34
|
-
const storageObjects = ['localStorage', 'sessionStorage', 'AsyncStorage'];
|
|
35
|
-
return {
|
|
36
|
-
CallExpression(node) {
|
|
37
|
-
// Check localStorage.setItem/sessionStorage.setItem/AsyncStorage.setItem
|
|
38
|
-
if (node.callee.type === 'MemberExpression' &&
|
|
39
|
-
node.callee.object.type === 'Identifier' &&
|
|
40
|
-
storageObjects.includes(node.callee.object.name) &&
|
|
41
|
-
node.callee.property.type === 'Identifier' &&
|
|
42
|
-
node.callee.property.name === 'setItem') {
|
|
43
|
-
const keyArg = node.arguments[0];
|
|
44
|
-
if (keyArg && keyArg.type === 'Literal' && typeof keyArg.value === 'string') {
|
|
45
|
-
const key = keyArg.value.toLowerCase();
|
|
46
|
-
if (sensitiveKeys.some(k => key.includes(k))) {
|
|
47
|
-
report(node);
|
|
48
|
-
}
|
|
49
|
-
}
|
|
50
|
-
}
|
|
51
|
-
},
|
|
52
|
-
};
|
|
53
|
-
},
|
|
54
|
-
});
|