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,14 +0,0 @@
|
|
|
1
|
-
import { type SecurityRuleOptions } from '@interlace/eslint-devkit';
|
|
2
|
-
export interface Options extends SecurityRuleOptions {
|
|
3
|
-
/** Allowed origins for postMessage communication */
|
|
4
|
-
allowedOrigins?: string[];
|
|
5
|
-
/** Whether to allow wildcard origins in development */
|
|
6
|
-
allowWildcardInDev?: boolean;
|
|
7
|
-
/** Message event handler patterns to check */
|
|
8
|
-
messageHandlerPatterns?: string[];
|
|
9
|
-
/** Origins considered safe (e.g., localhost, development domains) */
|
|
10
|
-
safeOrigins?: string[];
|
|
11
|
-
/** Whether to perform deep origin validation analysis */
|
|
12
|
-
deepValidation?: boolean;
|
|
13
|
-
}
|
|
14
|
-
export declare const noInsufficientPostmessageValidation: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|
|
@@ -1,392 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
exports.noInsufficientPostmessageValidation = 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.noInsufficientPostmessageValidation = (0, eslint_devkit_1.createRule)({
|
|
8
|
-
name: 'no-insufficient-postmessage-validation',
|
|
9
|
-
meta: {
|
|
10
|
-
type: 'problem',
|
|
11
|
-
docs: {
|
|
12
|
-
description: 'Detects insufficient postMessage origin validation',
|
|
13
|
-
},
|
|
14
|
-
fixable: 'code',
|
|
15
|
-
hasSuggestions: true,
|
|
16
|
-
messages: {
|
|
17
|
-
insufficientPostmessageValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
18
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
19
|
-
issueName: 'Insufficient postMessage Validation',
|
|
20
|
-
cwe: 'CWE-20',
|
|
21
|
-
description: 'postMessage origin validation is insufficient',
|
|
22
|
-
severity: '{{severity}}',
|
|
23
|
-
fix: '{{safeAlternative}}',
|
|
24
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
25
|
-
}),
|
|
26
|
-
wildcardOrigin: (0, eslint_devkit_2.formatLLMMessage)({
|
|
27
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
28
|
-
issueName: 'Wildcard postMessage Origin',
|
|
29
|
-
cwe: 'CWE-20',
|
|
30
|
-
description: 'postMessage uses wildcard origin (*)',
|
|
31
|
-
severity: 'CRITICAL',
|
|
32
|
-
fix: 'Use specific trusted origins instead of "*"',
|
|
33
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
34
|
-
}),
|
|
35
|
-
missingOriginCheck: (0, eslint_devkit_2.formatLLMMessage)({
|
|
36
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
37
|
-
issueName: 'Missing Origin Check',
|
|
38
|
-
cwe: 'CWE-20',
|
|
39
|
-
description: 'Message handler missing origin validation',
|
|
40
|
-
severity: 'HIGH',
|
|
41
|
-
fix: 'Check event.origin against trusted origins',
|
|
42
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
43
|
-
}),
|
|
44
|
-
unsafeMessageHandler: (0, eslint_devkit_2.formatLLMMessage)({
|
|
45
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
46
|
-
issueName: 'Unsafe Message Handler',
|
|
47
|
-
cwe: 'CWE-20',
|
|
48
|
-
description: 'Message event handler processes data without validation',
|
|
49
|
-
severity: 'MEDIUM',
|
|
50
|
-
fix: 'Validate event.origin and event.data before processing',
|
|
51
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
52
|
-
}),
|
|
53
|
-
useSpecificOrigins: (0, eslint_devkit_2.formatLLMMessage)({
|
|
54
|
-
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
55
|
-
issueName: 'Use Specific Origins',
|
|
56
|
-
description: 'Specify trusted origins explicitly',
|
|
57
|
-
severity: 'LOW',
|
|
58
|
-
fix: 'postMessage(data, "https://trusted-domain.com")',
|
|
59
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
60
|
-
}),
|
|
61
|
-
validateMessageOrigin: (0, eslint_devkit_2.formatLLMMessage)({
|
|
62
|
-
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
63
|
-
issueName: 'Validate Message Origin',
|
|
64
|
-
description: 'Validate message origin before processing',
|
|
65
|
-
severity: 'LOW',
|
|
66
|
-
fix: 'if (event.origin === "https://trusted.com") { /* process */ }',
|
|
67
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
68
|
-
}),
|
|
69
|
-
implementOriginWhitelist: (0, eslint_devkit_2.formatLLMMessage)({
|
|
70
|
-
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
71
|
-
issueName: 'Implement Origin Whitelist',
|
|
72
|
-
description: 'Use whitelist of allowed origins',
|
|
73
|
-
severity: 'LOW',
|
|
74
|
-
fix: 'const allowedOrigins = ["https://app1.com", "https://app2.com"];',
|
|
75
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
76
|
-
}),
|
|
77
|
-
strategyOriginValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
78
|
-
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
79
|
-
issueName: 'Origin Validation Strategy',
|
|
80
|
-
description: 'Validate message origins against trusted list',
|
|
81
|
-
severity: 'LOW',
|
|
82
|
-
fix: 'Check event.origin against predefined allowed origins',
|
|
83
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
84
|
-
}),
|
|
85
|
-
strategyContentValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
86
|
-
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
87
|
-
issueName: 'Content Validation Strategy',
|
|
88
|
-
description: 'Validate message content in addition to origin',
|
|
89
|
-
severity: 'LOW',
|
|
90
|
-
fix: 'Validate both event.origin and event.data structure',
|
|
91
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
92
|
-
}),
|
|
93
|
-
strategyEnvironmentSeparation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
94
|
-
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
95
|
-
issueName: 'Environment Separation Strategy',
|
|
96
|
-
description: 'Use different origins for different environments',
|
|
97
|
-
severity: 'LOW',
|
|
98
|
-
fix: 'Use environment-specific origin validation',
|
|
99
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
100
|
-
})
|
|
101
|
-
},
|
|
102
|
-
schema: [
|
|
103
|
-
{
|
|
104
|
-
type: 'object',
|
|
105
|
-
properties: {
|
|
106
|
-
allowedOrigins: {
|
|
107
|
-
type: 'array',
|
|
108
|
-
items: { type: 'string' },
|
|
109
|
-
default: [],
|
|
110
|
-
},
|
|
111
|
-
allowWildcardInDev: {
|
|
112
|
-
type: 'boolean',
|
|
113
|
-
default: false,
|
|
114
|
-
description: 'Allow wildcard origins in development environment'
|
|
115
|
-
},
|
|
116
|
-
messageHandlerPatterns: {
|
|
117
|
-
type: 'array',
|
|
118
|
-
items: { type: 'string' },
|
|
119
|
-
default: ['addEventListener', 'onmessage', 'postMessage'],
|
|
120
|
-
},
|
|
121
|
-
trustedSanitizers: {
|
|
122
|
-
type: 'array',
|
|
123
|
-
items: { type: 'string' },
|
|
124
|
-
default: [],
|
|
125
|
-
description: 'Additional function names to consider as origin validators',
|
|
126
|
-
},
|
|
127
|
-
trustedAnnotations: {
|
|
128
|
-
type: 'array',
|
|
129
|
-
items: { type: 'string' },
|
|
130
|
-
default: [],
|
|
131
|
-
description: 'Additional JSDoc annotations to consider as safe markers',
|
|
132
|
-
},
|
|
133
|
-
strictMode: {
|
|
134
|
-
type: 'boolean',
|
|
135
|
-
default: false,
|
|
136
|
-
description: 'Disable all false positive detection (strict mode)',
|
|
137
|
-
},
|
|
138
|
-
safeOrigins: {
|
|
139
|
-
type: 'array',
|
|
140
|
-
items: { type: 'string' },
|
|
141
|
-
default: ['localhost', '127.0.0.1', '0.0.0.0'],
|
|
142
|
-
description: 'Origins considered safe for postMessage communication',
|
|
143
|
-
},
|
|
144
|
-
deepValidation: {
|
|
145
|
-
type: 'boolean',
|
|
146
|
-
default: false,
|
|
147
|
-
description: 'Enable deep AST analysis for origin validation detection',
|
|
148
|
-
},
|
|
149
|
-
},
|
|
150
|
-
additionalProperties: false,
|
|
151
|
-
},
|
|
152
|
-
],
|
|
153
|
-
},
|
|
154
|
-
defaultOptions: [
|
|
155
|
-
{
|
|
156
|
-
allowedOrigins: [],
|
|
157
|
-
allowWildcardInDev: false,
|
|
158
|
-
messageHandlerPatterns: ['addEventListener', 'onmessage', 'postMessage'],
|
|
159
|
-
trustedSanitizers: [],
|
|
160
|
-
trustedAnnotations: [],
|
|
161
|
-
strictMode: false,
|
|
162
|
-
safeOrigins: ['localhost', '127.0.0.1', '0.0.0.0'],
|
|
163
|
-
deepValidation: false,
|
|
164
|
-
},
|
|
165
|
-
],
|
|
166
|
-
create(context) {
|
|
167
|
-
const options = context.options[0] || {};
|
|
168
|
-
const { allowedOrigins = [], allowWildcardInDev = false, trustedSanitizers = [], trustedAnnotations = [], strictMode = false, deepValidation = false, } = options;
|
|
169
|
-
const sourceCode = context.sourceCode || context.sourceCode;
|
|
170
|
-
const filename = context.filename || context.getFilename();
|
|
171
|
-
// Create safety checker for false positive detection
|
|
172
|
-
const safetyChecker = (0, eslint_devkit_3.createSafetyChecker)({
|
|
173
|
-
trustedSanitizers,
|
|
174
|
-
trustedAnnotations,
|
|
175
|
-
trustedOrmPatterns: [],
|
|
176
|
-
strictMode,
|
|
177
|
-
});
|
|
178
|
-
/**
|
|
179
|
-
* Check if origin validation is present in a message handler
|
|
180
|
-
*
|
|
181
|
-
* ⚠️ STATIC ANALYSIS LIMITATION:
|
|
182
|
-
* This can only detect simple patterns. Complex validation logic
|
|
183
|
-
* (external functions, runtime config, dynamic validation) cannot
|
|
184
|
-
* be detected statically.
|
|
185
|
-
*/
|
|
186
|
-
const hasOriginValidation = (functionNode) => {
|
|
187
|
-
if (deepValidation) {
|
|
188
|
-
// Deep AST analysis (more thorough but slower)
|
|
189
|
-
try {
|
|
190
|
-
let hasOriginCheck = false;
|
|
191
|
-
const checkNode = (node) => {
|
|
192
|
-
// Check for event.origin comparisons
|
|
193
|
-
if (node.type === 'BinaryExpression' &&
|
|
194
|
-
((node.left.type === 'MemberExpression' &&
|
|
195
|
-
node.left.property.type === 'Identifier' &&
|
|
196
|
-
node.left.property.name === 'origin' &&
|
|
197
|
-
node.left.object.type === 'Identifier') ||
|
|
198
|
-
(node.right.type === 'MemberExpression' &&
|
|
199
|
-
node.right.property.type === 'Identifier' &&
|
|
200
|
-
node.right.property.name === 'origin' &&
|
|
201
|
-
node.right.object.type === 'Identifier'))) {
|
|
202
|
-
hasOriginCheck = true;
|
|
203
|
-
return;
|
|
204
|
-
}
|
|
205
|
-
// Check for includes() calls on origin arrays
|
|
206
|
-
if (node.type === 'CallExpression' &&
|
|
207
|
-
node.callee.type === 'MemberExpression' &&
|
|
208
|
-
node.callee.property.type === 'Identifier' &&
|
|
209
|
-
node.callee.property.name === 'includes') {
|
|
210
|
-
const objectText = sourceCode.getText(node.callee.object).toLowerCase();
|
|
211
|
-
if (objectText.includes('origin') || objectText.includes('allowed')) {
|
|
212
|
-
hasOriginCheck = true;
|
|
213
|
-
return;
|
|
214
|
-
}
|
|
215
|
-
}
|
|
216
|
-
// Recursively check child nodes
|
|
217
|
-
for (const key in node) {
|
|
218
|
-
if (key === 'parent' || key === 'range' || key === 'loc')
|
|
219
|
-
continue;
|
|
220
|
-
const child = node[key];
|
|
221
|
-
if (child && typeof child === 'object' && 'type' in child) {
|
|
222
|
-
checkNode(child);
|
|
223
|
-
}
|
|
224
|
-
else if (Array.isArray(child)) {
|
|
225
|
-
child.forEach(item => {
|
|
226
|
-
if (item && typeof item === 'object' && 'type' in item) {
|
|
227
|
-
checkNode(item);
|
|
228
|
-
}
|
|
229
|
-
});
|
|
230
|
-
}
|
|
231
|
-
}
|
|
232
|
-
};
|
|
233
|
-
checkNode(functionNode);
|
|
234
|
-
return hasOriginCheck;
|
|
235
|
-
}
|
|
236
|
-
catch {
|
|
237
|
-
// If AST traversal fails, fall back to simple text matching
|
|
238
|
-
return false;
|
|
239
|
-
}
|
|
240
|
-
}
|
|
241
|
-
else {
|
|
242
|
-
// Simple text-based check (faster, less accurate)
|
|
243
|
-
const functionText = sourceCode.getText(functionNode).toLowerCase();
|
|
244
|
-
return functionText.includes('event.origin') ||
|
|
245
|
-
(functionText.includes('origin') && functionText.includes('event'));
|
|
246
|
-
}
|
|
247
|
-
};
|
|
248
|
-
return {
|
|
249
|
-
// Check postMessage calls with wildcard origins
|
|
250
|
-
CallExpression(node) {
|
|
251
|
-
const callee = node.callee;
|
|
252
|
-
// Check for any postMessage() calls
|
|
253
|
-
if (callee.type === 'MemberExpression' &&
|
|
254
|
-
callee.property.type === 'Identifier' &&
|
|
255
|
-
callee.property.name === 'postMessage') {
|
|
256
|
-
// Check allowed origins for non-wildcard origins
|
|
257
|
-
const args = node.arguments;
|
|
258
|
-
if (args.length >= 2) {
|
|
259
|
-
const targetOrigin = args[1];
|
|
260
|
-
if (targetOrigin.type === 'Literal' &&
|
|
261
|
-
typeof targetOrigin.value === 'string' &&
|
|
262
|
-
targetOrigin.value !== '*') {
|
|
263
|
-
// Check if origin is explicitly allowed
|
|
264
|
-
const originValue = targetOrigin.value;
|
|
265
|
-
const isAllowed = allowedOrigins.includes(originValue) ||
|
|
266
|
-
allowedOrigins.includes('*') ||
|
|
267
|
-
allowedOrigins.length === 0; // If no restrictions, allow all specific origins
|
|
268
|
-
if (!isAllowed) {
|
|
269
|
-
/* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
|
|
270
|
-
if (safetyChecker.isSafe(node, context)) {
|
|
271
|
-
return;
|
|
272
|
-
}
|
|
273
|
-
/* c8 ignore stop */
|
|
274
|
-
context.report({
|
|
275
|
-
node: targetOrigin,
|
|
276
|
-
messageId: 'useSpecificOrigins',
|
|
277
|
-
data: {
|
|
278
|
-
filePath: filename,
|
|
279
|
-
line: String(node.loc?.start.line ?? 0),
|
|
280
|
-
origin: originValue,
|
|
281
|
-
},
|
|
282
|
-
});
|
|
283
|
-
}
|
|
284
|
-
}
|
|
285
|
-
// Check for wildcard origin
|
|
286
|
-
if (targetOrigin.type === 'Literal' &&
|
|
287
|
-
typeof targetOrigin.value === 'string' &&
|
|
288
|
-
targetOrigin.value === '*') {
|
|
289
|
-
// Check if origin is explicitly allowed
|
|
290
|
-
const originValue = targetOrigin.value;
|
|
291
|
-
const isAllowed = allowedOrigins.includes(originValue) || allowedOrigins.includes('*');
|
|
292
|
-
// Allow if explicitly allowed OR in development if configured
|
|
293
|
-
if (!isAllowed && !allowWildcardInDev) {
|
|
294
|
-
// TEMP: bypass safety checker
|
|
295
|
-
// if (safetyChecker.isSafe(node, context)) {
|
|
296
|
-
// return;
|
|
297
|
-
// }
|
|
298
|
-
context.report({
|
|
299
|
-
node: targetOrigin,
|
|
300
|
-
messageId: 'wildcardOrigin',
|
|
301
|
-
data: {
|
|
302
|
-
filePath: filename,
|
|
303
|
-
line: String(node.loc?.start.line ?? 0),
|
|
304
|
-
},
|
|
305
|
-
});
|
|
306
|
-
}
|
|
307
|
-
}
|
|
308
|
-
}
|
|
309
|
-
}
|
|
310
|
-
// Check for addEventListener('message', ...) and IPC calls
|
|
311
|
-
// Check for addEventListener('message', ...)
|
|
312
|
-
if (callee.type === 'MemberExpression' &&
|
|
313
|
-
callee.property.type === 'Identifier' &&
|
|
314
|
-
callee.property.name === 'addEventListener') {
|
|
315
|
-
const args = node.arguments;
|
|
316
|
-
if (args.length >= 2) {
|
|
317
|
-
const eventType = args[0];
|
|
318
|
-
const handler = args[1];
|
|
319
|
-
// Check if it's a message event
|
|
320
|
-
if (eventType.type === 'Literal' &&
|
|
321
|
-
typeof eventType.value === 'string' &&
|
|
322
|
-
eventType.value === 'message') {
|
|
323
|
-
// Check if handler is a function and lacks origin validation
|
|
324
|
-
if (handler.type === 'FunctionExpression' ||
|
|
325
|
-
handler.type === 'ArrowFunctionExpression') {
|
|
326
|
-
if (!hasOriginValidation(handler)) {
|
|
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: handler,
|
|
334
|
-
messageId: 'missingOriginCheck',
|
|
335
|
-
data: {
|
|
336
|
-
filePath: filename,
|
|
337
|
-
line: String(node.loc?.start.line ?? 0),
|
|
338
|
-
},
|
|
339
|
-
});
|
|
340
|
-
}
|
|
341
|
-
}
|
|
342
|
-
}
|
|
343
|
-
}
|
|
344
|
-
}
|
|
345
|
-
},
|
|
346
|
-
// Check for window.onmessage assignments
|
|
347
|
-
AssignmentExpression(node) {
|
|
348
|
-
const left = node.left;
|
|
349
|
-
if (left.type === 'MemberExpression' &&
|
|
350
|
-
left.property.type === 'Identifier' &&
|
|
351
|
-
left.property.name === 'onmessage' &&
|
|
352
|
-
left.object.type === 'Identifier' &&
|
|
353
|
-
left.object.name === 'window') {
|
|
354
|
-
const handler = node.right;
|
|
355
|
-
if (handler && (handler.type === 'FunctionExpression' ||
|
|
356
|
-
handler.type === 'ArrowFunctionExpression')) {
|
|
357
|
-
if (!hasOriginValidation(handler)) {
|
|
358
|
-
/* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
|
|
359
|
-
if (safetyChecker.isSafe(node, context)) {
|
|
360
|
-
return;
|
|
361
|
-
}
|
|
362
|
-
/* c8 ignore stop */
|
|
363
|
-
context.report({
|
|
364
|
-
node: handler,
|
|
365
|
-
messageId: 'missingOriginCheck',
|
|
366
|
-
data: {
|
|
367
|
-
filePath: filename,
|
|
368
|
-
line: String(node.loc?.start.line ?? 0),
|
|
369
|
-
},
|
|
370
|
-
});
|
|
371
|
-
}
|
|
372
|
-
}
|
|
373
|
-
}
|
|
374
|
-
},
|
|
375
|
-
// Check for event.data usage without origin validation
|
|
376
|
-
MemberExpression(node) {
|
|
377
|
-
// Look for event.data usage
|
|
378
|
-
if (node.property.type === 'Identifier' &&
|
|
379
|
-
node.property.name === 'data' &&
|
|
380
|
-
node.object.type === 'Identifier' &&
|
|
381
|
-
node.object.name === 'event') {
|
|
382
|
-
// For now, simplify: only flag if we're clearly in an unsafe context
|
|
383
|
-
// The addEventListener check above should handle most cases
|
|
384
|
-
// This is a simplified version to avoid complex AST traversal issues
|
|
385
|
-
// Only flag if this is clearly in a message handler without validation
|
|
386
|
-
// We'll rely on the addEventListener check for the main logic
|
|
387
|
-
return;
|
|
388
|
-
}
|
|
389
|
-
},
|
|
390
|
-
};
|
|
391
|
-
},
|
|
392
|
-
});
|
|
@@ -1,9 +0,0 @@
|
|
|
1
|
-
export interface Options {
|
|
2
|
-
/** Allow Math.random() in test files. Default: false */
|
|
3
|
-
allowInTests?: boolean;
|
|
4
|
-
/** Additional weak PRNG patterns to detect. Default: [] */
|
|
5
|
-
additionalWeakPatterns?: string[];
|
|
6
|
-
/** Trusted random libraries. Default: ['crypto'] */
|
|
7
|
-
trustedLibraries?: string[];
|
|
8
|
-
}
|
|
9
|
-
export declare const noInsufficientRandom: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|
|
@@ -1,208 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
exports.noInsufficientRandom = void 0;
|
|
4
|
-
const eslint_devkit_1 = require("@interlace/eslint-devkit");
|
|
5
|
-
const eslint_devkit_2 = require("@interlace/eslint-devkit");
|
|
6
|
-
const WEAK_RANDOM_PATTERNS = [
|
|
7
|
-
{
|
|
8
|
-
pattern: /\bMath\.random\b/i,
|
|
9
|
-
name: 'Math.random()',
|
|
10
|
-
category: 'math',
|
|
11
|
-
alternatives: ['crypto.getRandomValues()', 'crypto.randomBytes()'],
|
|
12
|
-
example: {
|
|
13
|
-
bad: 'const random = Math.random();',
|
|
14
|
-
good: 'const random = crypto.getRandomValues(new Uint32Array(1))[0] / (0xFFFFFFFF + 1);'
|
|
15
|
-
},
|
|
16
|
-
effort: '5 minutes'
|
|
17
|
-
},
|
|
18
|
-
{
|
|
19
|
-
pattern: /\brandom\(\)/i,
|
|
20
|
-
name: 'random()',
|
|
21
|
-
category: 'library',
|
|
22
|
-
alternatives: ['crypto.getRandomValues()'],
|
|
23
|
-
example: {
|
|
24
|
-
bad: 'const random = random();',
|
|
25
|
-
good: 'const random = crypto.getRandomValues(new Uint32Array(1))[0] / (0xFFFFFFFF + 1);'
|
|
26
|
-
},
|
|
27
|
-
effort: '5 minutes'
|
|
28
|
-
}
|
|
29
|
-
];
|
|
30
|
-
/**
|
|
31
|
-
* Check if a string contains a weak random pattern
|
|
32
|
-
*/
|
|
33
|
-
function containsWeakRandom(value, additionalPatterns) {
|
|
34
|
-
// Check standard patterns
|
|
35
|
-
for (const pattern of WEAK_RANDOM_PATTERNS) {
|
|
36
|
-
if (pattern.pattern.test(value)) {
|
|
37
|
-
return pattern;
|
|
38
|
-
}
|
|
39
|
-
}
|
|
40
|
-
// Check additional patterns
|
|
41
|
-
for (const additionalPattern of additionalPatterns) {
|
|
42
|
-
const regex = new RegExp(`\\b${additionalPattern}\\b`, 'i');
|
|
43
|
-
if (regex.test(value)) {
|
|
44
|
-
return {
|
|
45
|
-
pattern: regex,
|
|
46
|
-
name: additionalPattern,
|
|
47
|
-
category: 'custom',
|
|
48
|
-
alternatives: ['crypto.getRandomValues()'],
|
|
49
|
-
example: {
|
|
50
|
-
bad: `${additionalPattern}()`,
|
|
51
|
-
good: 'crypto.getRandomValues(new Uint32Array(1))[0] / (0xFFFFFFFF + 1)'
|
|
52
|
-
},
|
|
53
|
-
effort: '10 minutes'
|
|
54
|
-
};
|
|
55
|
-
}
|
|
56
|
-
}
|
|
57
|
-
return null;
|
|
58
|
-
}
|
|
59
|
-
/**
|
|
60
|
-
* Check if a MemberExpression is Math.random()
|
|
61
|
-
*/
|
|
62
|
-
function isMathRandom(node) {
|
|
63
|
-
return (node.object.type === 'Identifier' &&
|
|
64
|
-
node.object.name === 'Math' &&
|
|
65
|
-
node.property.type === 'Identifier' &&
|
|
66
|
-
node.property.name === 'random');
|
|
67
|
-
}
|
|
68
|
-
/**
|
|
69
|
-
* Generate refactoring suggestions
|
|
70
|
-
*/
|
|
71
|
-
function generateSuggestions(pattern) {
|
|
72
|
-
const suggestions = [];
|
|
73
|
-
if (pattern.category === 'math') {
|
|
74
|
-
suggestions.push({
|
|
75
|
-
messageId: 'useCryptoRandomValues',
|
|
76
|
-
fix: 'Use crypto.getRandomValues(): const random = crypto.getRandomValues(new Uint32Array(1))[0] / (0xFFFFFFFF + 1);'
|
|
77
|
-
});
|
|
78
|
-
}
|
|
79
|
-
else {
|
|
80
|
-
suggestions.push({
|
|
81
|
-
messageId: 'useSecureRandom',
|
|
82
|
-
fix: 'Use crypto.getRandomValues() or crypto.randomBytes() for secure random number generation'
|
|
83
|
-
});
|
|
84
|
-
}
|
|
85
|
-
return suggestions;
|
|
86
|
-
}
|
|
87
|
-
exports.noInsufficientRandom = (0, eslint_devkit_2.createRule)({
|
|
88
|
-
name: 'no-insufficient-random',
|
|
89
|
-
meta: {
|
|
90
|
-
type: 'problem',
|
|
91
|
-
deprecated: true,
|
|
92
|
-
replacedBy: ['@see eslint-plugin-crypto/no-math-random-crypto'],
|
|
93
|
-
docs: {
|
|
94
|
-
description: 'Detects weak random number generation (Math.random(), weak PRNG)',
|
|
95
|
-
},
|
|
96
|
-
hasSuggestions: true,
|
|
97
|
-
messages: {
|
|
98
|
-
insufficientRandom: (0, eslint_devkit_1.formatLLMMessage)({
|
|
99
|
-
icon: eslint_devkit_1.MessageIcons.SECURITY,
|
|
100
|
-
issueName: 'Weak random number generation',
|
|
101
|
-
cwe: 'CWE-338',
|
|
102
|
-
description: 'Use of weak pseudo-random number generator: {{pattern}}',
|
|
103
|
-
severity: 'HIGH',
|
|
104
|
-
fix: '{{safeAlternative}}',
|
|
105
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/338.html',
|
|
106
|
-
}),
|
|
107
|
-
useCryptoRandomValues: (0, eslint_devkit_1.formatLLMMessage)({
|
|
108
|
-
icon: eslint_devkit_1.MessageIcons.INFO,
|
|
109
|
-
issueName: 'Use Crypto Random',
|
|
110
|
-
description: 'Use crypto.getRandomValues()',
|
|
111
|
-
severity: 'LOW',
|
|
112
|
-
fix: 'crypto.getRandomValues(new Uint32Array(1))[0] / (0xFFFFFFFF + 1)',
|
|
113
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Crypto/getRandomValues',
|
|
114
|
-
}),
|
|
115
|
-
useSecureRandom: (0, eslint_devkit_1.formatLLMMessage)({
|
|
116
|
-
icon: eslint_devkit_1.MessageIcons.INFO,
|
|
117
|
-
issueName: 'Use Secure Random',
|
|
118
|
-
description: 'Use crypto for secure random generation',
|
|
119
|
-
severity: 'LOW',
|
|
120
|
-
fix: 'crypto.randomBytes() or crypto.getRandomValues()',
|
|
121
|
-
documentationLink: 'https://nodejs.org/api/crypto.html#cryptorandombytessize-callback',
|
|
122
|
-
}),
|
|
123
|
-
},
|
|
124
|
-
schema: [
|
|
125
|
-
{
|
|
126
|
-
type: 'object',
|
|
127
|
-
properties: {
|
|
128
|
-
allowInTests: {
|
|
129
|
-
type: 'boolean',
|
|
130
|
-
default: false,
|
|
131
|
-
description: 'Allow Math.random() in test files',
|
|
132
|
-
},
|
|
133
|
-
additionalWeakPatterns: {
|
|
134
|
-
type: 'array',
|
|
135
|
-
items: { type: 'string' },
|
|
136
|
-
default: [],
|
|
137
|
-
description: 'Additional weak PRNG patterns to detect',
|
|
138
|
-
},
|
|
139
|
-
},
|
|
140
|
-
additionalProperties: false,
|
|
141
|
-
},
|
|
142
|
-
],
|
|
143
|
-
},
|
|
144
|
-
defaultOptions: [
|
|
145
|
-
{
|
|
146
|
-
allowInTests: false,
|
|
147
|
-
additionalWeakPatterns: [],
|
|
148
|
-
},
|
|
149
|
-
],
|
|
150
|
-
create(context, [options = {}]) {
|
|
151
|
-
const { allowInTests = false, additionalWeakPatterns = [], } = options;
|
|
152
|
-
const filename = context.getFilename();
|
|
153
|
-
const isTestFile = allowInTests && /\.(test|spec)\.(ts|tsx|js|jsx)$/.test(filename);
|
|
154
|
-
/**
|
|
155
|
-
* Check if a call expression uses weak random generation
|
|
156
|
-
*/
|
|
157
|
-
function checkCallExpression(node) {
|
|
158
|
-
if (isTestFile) {
|
|
159
|
-
return;
|
|
160
|
-
}
|
|
161
|
-
// Check for Math.random() calls
|
|
162
|
-
if (node.callee.type === 'MemberExpression' && isMathRandom(node.callee)) {
|
|
163
|
-
const pattern = containsWeakRandom('Math.random()', additionalWeakPatterns);
|
|
164
|
-
if (pattern) {
|
|
165
|
-
const safeAlternative = pattern.alternatives[0];
|
|
166
|
-
const suggestions = generateSuggestions(pattern);
|
|
167
|
-
context.report({
|
|
168
|
-
node: node.callee,
|
|
169
|
-
messageId: 'insufficientRandom',
|
|
170
|
-
data: {
|
|
171
|
-
pattern: pattern.name,
|
|
172
|
-
safeAlternative: `Use ${safeAlternative}: ${pattern.example.good}`,
|
|
173
|
-
},
|
|
174
|
-
suggest: suggestions.map(step => ({
|
|
175
|
-
messageId: step.messageId,
|
|
176
|
-
fix: (fixer) => {
|
|
177
|
-
// Replace Math.random() with crypto.getRandomValues()
|
|
178
|
-
return fixer.replaceText(node, 'crypto.getRandomValues(new Uint32Array(1))[0] / (0xFFFFFFFF + 1)');
|
|
179
|
-
},
|
|
180
|
-
})),
|
|
181
|
-
});
|
|
182
|
-
}
|
|
183
|
-
return;
|
|
184
|
-
}
|
|
185
|
-
// Check for standalone random() function calls (if configured)
|
|
186
|
-
if (node.callee.type === 'Identifier' && additionalWeakPatterns.length > 0) {
|
|
187
|
-
const sourceCode = context.sourceCode || context.sourceCode;
|
|
188
|
-
const callText = sourceCode.getText(node);
|
|
189
|
-
const pattern = containsWeakRandom(callText, additionalWeakPatterns);
|
|
190
|
-
if (pattern) {
|
|
191
|
-
const safeAlternative = pattern.alternatives[0];
|
|
192
|
-
context.report({
|
|
193
|
-
node: node.callee,
|
|
194
|
-
messageId: 'insufficientRandom',
|
|
195
|
-
data: {
|
|
196
|
-
pattern: pattern.name,
|
|
197
|
-
safeAlternative: `Use ${safeAlternative}: ${pattern.example.good}`,
|
|
198
|
-
},
|
|
199
|
-
// Custom patterns cannot be safely auto-fixed - error message provides guidance
|
|
200
|
-
});
|
|
201
|
-
}
|
|
202
|
-
}
|
|
203
|
-
}
|
|
204
|
-
return {
|
|
205
|
-
CallExpression: checkCallExpression,
|
|
206
|
-
};
|
|
207
|
-
},
|
|
208
|
-
});
|
|
@@ -1,10 +0,0 @@
|
|
|
1
|
-
import { type SecurityRuleOptions } from '@interlace/eslint-devkit';
|
|
2
|
-
export interface Options extends SecurityRuleOptions {
|
|
3
|
-
/** LDAP-related function names to check */
|
|
4
|
-
ldapFunctions?: string[];
|
|
5
|
-
/** Functions that safely escape LDAP input */
|
|
6
|
-
ldapEscapeFunctions?: string[];
|
|
7
|
-
/** Functions that validate LDAP input */
|
|
8
|
-
ldapValidationFunctions?: string[];
|
|
9
|
-
}
|
|
10
|
-
export declare const noLdapInjection: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|