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,491 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
exports.noUnsafeDeserialization = 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.noUnsafeDeserialization = (0, eslint_devkit_1.createRule)({
|
|
8
|
-
name: 'no-unsafe-deserialization',
|
|
9
|
-
meta: {
|
|
10
|
-
type: 'problem',
|
|
11
|
-
docs: {
|
|
12
|
-
description: 'Detects unsafe deserialization of untrusted data',
|
|
13
|
-
},
|
|
14
|
-
fixable: 'code',
|
|
15
|
-
hasSuggestions: true,
|
|
16
|
-
messages: {
|
|
17
|
-
unsafeDeserialization: (0, eslint_devkit_2.formatLLMMessage)({
|
|
18
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
19
|
-
issueName: 'Unsafe Deserialization',
|
|
20
|
-
cwe: 'CWE-502',
|
|
21
|
-
description: 'Unsafe deserialization of untrusted data (incl. model/tool output)',
|
|
22
|
-
severity: '{{severity}}',
|
|
23
|
-
fix: '{{safeAlternative}} | validate model/tool output via schema and size limits',
|
|
24
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/502.html',
|
|
25
|
-
}),
|
|
26
|
-
dangerousEvalUsage: (0, eslint_devkit_2.formatLLMMessage)({
|
|
27
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
28
|
-
issueName: 'Dangerous eval() Usage',
|
|
29
|
-
cwe: 'CWE-502',
|
|
30
|
-
description: 'eval() used for deserialization (code execution vulnerability)',
|
|
31
|
-
severity: 'CRITICAL',
|
|
32
|
-
fix: 'Use JSON.parse() or safe deserialization libraries',
|
|
33
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/502.html',
|
|
34
|
-
}),
|
|
35
|
-
unsafeYamlParsing: (0, eslint_devkit_2.formatLLMMessage)({
|
|
36
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
37
|
-
issueName: 'Unsafe YAML Parsing',
|
|
38
|
-
cwe: 'CWE-502',
|
|
39
|
-
description: 'YAML parsing may execute code during deserialization',
|
|
40
|
-
severity: 'HIGH',
|
|
41
|
-
fix: 'Use yaml.safeLoad() or disable code execution',
|
|
42
|
-
documentationLink: 'https://www.npmjs.com/package/js-yaml#loadstr---options-',
|
|
43
|
-
}),
|
|
44
|
-
dangerousFunctionConstructor: (0, eslint_devkit_2.formatLLMMessage)({
|
|
45
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
46
|
-
issueName: 'Dangerous Function Constructor',
|
|
47
|
-
cwe: 'CWE-502',
|
|
48
|
-
description: 'Function constructor used with untrusted data',
|
|
49
|
-
severity: 'CRITICAL',
|
|
50
|
-
fix: 'Avoid Function constructor with user input',
|
|
51
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function',
|
|
52
|
-
}),
|
|
53
|
-
untrustedDeserializationInput: (0, eslint_devkit_2.formatLLMMessage)({
|
|
54
|
-
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
55
|
-
issueName: 'Untrusted Deserialization Input',
|
|
56
|
-
cwe: 'CWE-502',
|
|
57
|
-
description: 'Deserializing untrusted input (incl. LLM/MCP responses) without validation',
|
|
58
|
-
severity: 'HIGH',
|
|
59
|
-
fix: 'Schema-validate and size-cap before deserialization; reject unknown fields',
|
|
60
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/502.html',
|
|
61
|
-
}),
|
|
62
|
-
useSafeDeserializer: (0, eslint_devkit_2.formatLLMMessage)({
|
|
63
|
-
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
64
|
-
issueName: 'Use Safe Deserializer',
|
|
65
|
-
description: 'Use safe deserialization libraries',
|
|
66
|
-
severity: 'LOW',
|
|
67
|
-
fix: 'Use JSON.parse, safe-json-parse, or validated libraries',
|
|
68
|
-
documentationLink: 'https://www.npmjs.com/package/safe-json-parse',
|
|
69
|
-
}),
|
|
70
|
-
validateBeforeDeserialization: (0, eslint_devkit_2.formatLLMMessage)({
|
|
71
|
-
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
72
|
-
issueName: 'Validate Before Deserialization',
|
|
73
|
-
description: 'Validate input before deserialization',
|
|
74
|
-
severity: 'LOW',
|
|
75
|
-
fix: 'Implement input validation and length limits',
|
|
76
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/502.html',
|
|
77
|
-
}),
|
|
78
|
-
avoidEval: (0, eslint_devkit_2.formatLLMMessage)({
|
|
79
|
-
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
80
|
-
issueName: 'Avoid eval()',
|
|
81
|
-
description: 'Never use eval() for deserialization',
|
|
82
|
-
severity: 'LOW',
|
|
83
|
-
fix: 'Use JSON.parse() for data, vm.Script for code when absolutely necessary',
|
|
84
|
-
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/eval',
|
|
85
|
-
}),
|
|
86
|
-
strategySafeLibraries: (0, eslint_devkit_2.formatLLMMessage)({
|
|
87
|
-
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
88
|
-
issueName: 'Safe Libraries Strategy',
|
|
89
|
-
description: 'Use deserialization libraries with built-in safety',
|
|
90
|
-
severity: 'LOW',
|
|
91
|
-
fix: 'Use JSON.parse, js-yaml.safeLoad, or protobuf libraries',
|
|
92
|
-
documentationLink: 'https://www.npmjs.com/package/js-yaml',
|
|
93
|
-
}),
|
|
94
|
-
strategyInputValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
95
|
-
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
96
|
-
issueName: 'Input Validation Strategy',
|
|
97
|
-
description: 'Validate input before any deserialization',
|
|
98
|
-
severity: 'LOW',
|
|
99
|
-
fix: 'Implement schema validation and length limits',
|
|
100
|
-
documentationLink: 'https://cwe.mitre.org/data/definitions/502.html',
|
|
101
|
-
}),
|
|
102
|
-
strategySandboxing: (0, eslint_devkit_2.formatLLMMessage)({
|
|
103
|
-
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
104
|
-
issueName: 'Sandboxing Strategy',
|
|
105
|
-
description: 'Execute deserialization in sandboxed environment',
|
|
106
|
-
severity: 'LOW',
|
|
107
|
-
fix: 'Use vm module or worker threads for untrusted deserialization',
|
|
108
|
-
documentationLink: 'https://nodejs.org/api/vm.html',
|
|
109
|
-
})
|
|
110
|
-
},
|
|
111
|
-
schema: [
|
|
112
|
-
{
|
|
113
|
-
type: 'object',
|
|
114
|
-
properties: {
|
|
115
|
-
dangerousFunctions: {
|
|
116
|
-
type: 'array',
|
|
117
|
-
items: { type: 'string' },
|
|
118
|
-
default: ['eval', 'Function', 'setTimeout', 'setInterval', 'unserialize', 'deserialize', 'parseUnsafe'],
|
|
119
|
-
},
|
|
120
|
-
safeLibraries: {
|
|
121
|
-
type: 'array',
|
|
122
|
-
items: { type: 'string' },
|
|
123
|
-
default: ['JSON', 'safe-json-parse', 'js-yaml.safeLoad', 'protobuf', 'msgpack'],
|
|
124
|
-
},
|
|
125
|
-
validationFunctions: {
|
|
126
|
-
type: 'array',
|
|
127
|
-
items: { type: 'string' },
|
|
128
|
-
default: ['validateInput', 'sanitizeData', 'checkSchema', 'validateSchema'],
|
|
129
|
-
},
|
|
130
|
-
trustedSanitizers: {
|
|
131
|
-
type: 'array',
|
|
132
|
-
items: { type: 'string' },
|
|
133
|
-
default: [],
|
|
134
|
-
description: 'Additional function names to consider as safe deserializers',
|
|
135
|
-
},
|
|
136
|
-
trustedAnnotations: {
|
|
137
|
-
type: 'array',
|
|
138
|
-
items: { type: 'string' },
|
|
139
|
-
default: [],
|
|
140
|
-
description: 'Additional JSDoc annotations to consider as safe markers',
|
|
141
|
-
},
|
|
142
|
-
strictMode: {
|
|
143
|
-
type: 'boolean',
|
|
144
|
-
default: false,
|
|
145
|
-
description: 'Disable all false positive detection (strict mode)',
|
|
146
|
-
},
|
|
147
|
-
},
|
|
148
|
-
additionalProperties: false,
|
|
149
|
-
},
|
|
150
|
-
],
|
|
151
|
-
},
|
|
152
|
-
defaultOptions: [
|
|
153
|
-
{
|
|
154
|
-
dangerousFunctions: ['eval', 'Function', 'setTimeout', 'setInterval', 'unserialize', 'deserialize', 'parseUnsafe'],
|
|
155
|
-
safeLibraries: ['JSON', 'safe-json-parse', 'js-yaml.safeLoad', 'protobuf', 'msgpack'],
|
|
156
|
-
validationFunctions: ['validateInput', 'sanitizeData', 'checkSchema', 'validateSchema'],
|
|
157
|
-
trustedSanitizers: [],
|
|
158
|
-
trustedAnnotations: [],
|
|
159
|
-
strictMode: false,
|
|
160
|
-
},
|
|
161
|
-
],
|
|
162
|
-
create(context) {
|
|
163
|
-
const options = context.options[0] || {};
|
|
164
|
-
const { dangerousFunctions = ['eval', 'Function', 'setTimeout', 'setInterval', 'unserialize', 'deserialize', 'parseUnsafe'], validationFunctions = ['validateInput', 'sanitizeData', 'checkSchema', 'validateSchema'], trustedSanitizers = [], trustedAnnotations = [], strictMode = false, } = options;
|
|
165
|
-
const sourceCode = context.sourceCode || context.sourceCode;
|
|
166
|
-
const filename = context.filename || context.getFilename();
|
|
167
|
-
// Create safety checker for false positive detection
|
|
168
|
-
const safetyChecker = (0, eslint_devkit_3.createSafetyChecker)({
|
|
169
|
-
trustedSanitizers,
|
|
170
|
-
trustedAnnotations,
|
|
171
|
-
trustedOrmPatterns: [],
|
|
172
|
-
strictMode,
|
|
173
|
-
});
|
|
174
|
-
// Track variables that have been validated/sanitized
|
|
175
|
-
const validatedVariables = new Set();
|
|
176
|
-
// Track variables that contain untrusted data
|
|
177
|
-
const untrustedVariables = new Set();
|
|
178
|
-
/**
|
|
179
|
-
* Check if this is a dangerous deserialization function
|
|
180
|
-
*/
|
|
181
|
-
const isDangerousDeserialization = (node) => {
|
|
182
|
-
const callee = node.callee;
|
|
183
|
-
// Check for dangerous function calls
|
|
184
|
-
if (callee.type === 'Identifier' && dangerousFunctions.includes(callee.name)) {
|
|
185
|
-
return true;
|
|
186
|
-
}
|
|
187
|
-
// Check for member expressions like yaml.load, serialize.unserialize
|
|
188
|
-
if (callee.type === 'MemberExpression') {
|
|
189
|
-
const memberName = callee.property.type === 'Identifier' ? callee.property.name : '';
|
|
190
|
-
const objectName = callee.object.type === 'Identifier' ? callee.object.name : '';
|
|
191
|
-
// Check dangerous methods
|
|
192
|
-
if (dangerousFunctions.includes(memberName)) {
|
|
193
|
-
return true;
|
|
194
|
-
}
|
|
195
|
-
// Check dangerous libraries
|
|
196
|
-
if (['yaml', 'js-yaml', 'node-serialize', 'serialize-javascript'].includes(objectName.toLowerCase()) &&
|
|
197
|
-
['load', 'parse', 'unserialize', 'deserialize'].includes(memberName)) {
|
|
198
|
-
return true;
|
|
199
|
-
}
|
|
200
|
-
}
|
|
201
|
-
// Check for require() calls with dangerous libraries
|
|
202
|
-
if (callee.type === 'Identifier' && callee.name === 'require') {
|
|
203
|
-
const args = node.arguments;
|
|
204
|
-
if (args.length > 0 && args[0].type === 'Literal' && typeof args[0].value === 'string') {
|
|
205
|
-
const moduleName = args[0].value.toLowerCase();
|
|
206
|
-
if (['node-serialize', 'serialize-javascript', 'yaml', 'js-yaml'].includes(moduleName)) {
|
|
207
|
-
return true;
|
|
208
|
-
}
|
|
209
|
-
}
|
|
210
|
-
}
|
|
211
|
-
return false;
|
|
212
|
-
};
|
|
213
|
-
/**
|
|
214
|
-
* Check if input comes from untrusted source
|
|
215
|
-
*/
|
|
216
|
-
const isUntrustedInput = (inputNode) => {
|
|
217
|
-
// Check for MemberExpression patterns like req.body, req.query, etc.
|
|
218
|
-
if (inputNode.type === 'MemberExpression') {
|
|
219
|
-
if (inputNode.object.type === 'Identifier' && inputNode.object.name === 'req') {
|
|
220
|
-
return true;
|
|
221
|
-
}
|
|
222
|
-
if (inputNode.object.type === 'MemberExpression' &&
|
|
223
|
-
inputNode.object.object.type === 'Identifier' &&
|
|
224
|
-
inputNode.object.object.name === 'req') {
|
|
225
|
-
return true;
|
|
226
|
-
}
|
|
227
|
-
}
|
|
228
|
-
if (inputNode.type === 'Identifier') {
|
|
229
|
-
// Check if this variable has been marked as untrusted
|
|
230
|
-
if (untrustedVariables.has(inputNode.name)) {
|
|
231
|
-
return true;
|
|
232
|
-
}
|
|
233
|
-
// Only consider variables untrusted if they actually come from req.* patterns
|
|
234
|
-
// Don't flag generic variable names like 'input', 'data', etc.
|
|
235
|
-
// unless they have been explicitly marked as untrusted
|
|
236
|
-
// Check if it comes from function parameters (these are potentially untrusted)
|
|
237
|
-
let current = inputNode;
|
|
238
|
-
while (current) {
|
|
239
|
-
if (current.type === 'FunctionDeclaration' ||
|
|
240
|
-
current.type === 'FunctionExpression' ||
|
|
241
|
-
current.type === 'ArrowFunctionExpression') {
|
|
242
|
-
const func = current;
|
|
243
|
-
if (func.params.some((param) => {
|
|
244
|
-
if (param.type === 'Identifier') {
|
|
245
|
-
return param.name === inputNode.name;
|
|
246
|
-
}
|
|
247
|
-
return false;
|
|
248
|
-
})) {
|
|
249
|
-
return true; // Function parameters are untrusted
|
|
250
|
-
}
|
|
251
|
-
}
|
|
252
|
-
current = current.parent;
|
|
253
|
-
}
|
|
254
|
-
}
|
|
255
|
-
return false;
|
|
256
|
-
};
|
|
257
|
-
/**
|
|
258
|
-
* Check if input has been validated
|
|
259
|
-
*/
|
|
260
|
-
const isInputValidated = (inputNode) => {
|
|
261
|
-
let current = inputNode;
|
|
262
|
-
while (current) {
|
|
263
|
-
if (current.type === 'CallExpression' &&
|
|
264
|
-
current.callee.type === 'Identifier' &&
|
|
265
|
-
validationFunctions.includes(current.callee.name)) {
|
|
266
|
-
return true;
|
|
267
|
-
}
|
|
268
|
-
current = current.parent;
|
|
269
|
-
}
|
|
270
|
-
return false;
|
|
271
|
-
};
|
|
272
|
-
/**
|
|
273
|
-
* Check if this is a safe deserialization library
|
|
274
|
-
*/
|
|
275
|
-
const isSafeLibrary = (node) => {
|
|
276
|
-
const callee = node.callee;
|
|
277
|
-
if (callee.type === 'MemberExpression') {
|
|
278
|
-
const objectName = callee.object.type === 'Identifier' ? callee.object.name : '';
|
|
279
|
-
const memberName = callee.property.type === 'Identifier' ? callee.property.name : '';
|
|
280
|
-
// Check for safe patterns
|
|
281
|
-
if (objectName === 'JSON' && memberName === 'parse') {
|
|
282
|
-
return true;
|
|
283
|
-
}
|
|
284
|
-
if (objectName === 'yaml' && memberName === 'safeLoad') {
|
|
285
|
-
return true;
|
|
286
|
-
}
|
|
287
|
-
if (objectName === 'js-yaml' && memberName === 'safeLoad') {
|
|
288
|
-
return true;
|
|
289
|
-
}
|
|
290
|
-
}
|
|
291
|
-
return false;
|
|
292
|
-
};
|
|
293
|
-
const checkCallExpression = (node) => {
|
|
294
|
-
// 1. Check Function Constructor (NewExpression or CallExpression)
|
|
295
|
-
if ((node.type === 'NewExpression' || node.type === 'CallExpression') &&
|
|
296
|
-
node.callee.type === 'Identifier' &&
|
|
297
|
-
node.callee.name === 'Function') {
|
|
298
|
-
const args = node.arguments;
|
|
299
|
-
const hasUntrustedInput = args.some((arg) => isUntrustedInput(arg));
|
|
300
|
-
if (hasUntrustedInput) {
|
|
301
|
-
if (safetyChecker.isSafe(node, context))
|
|
302
|
-
return;
|
|
303
|
-
context.report({
|
|
304
|
-
node,
|
|
305
|
-
messageId: 'dangerousFunctionConstructor',
|
|
306
|
-
data: {
|
|
307
|
-
filePath: context.getFilename(),
|
|
308
|
-
line: String(node.loc?.start.line ?? 0),
|
|
309
|
-
severity: 'HIGH',
|
|
310
|
-
safeAlternative: 'Avoid dynamic function creation',
|
|
311
|
-
}
|
|
312
|
-
});
|
|
313
|
-
return;
|
|
314
|
-
}
|
|
315
|
-
}
|
|
316
|
-
// 2. Check CallExpressions (eval, unserialize, yaml, etc.)
|
|
317
|
-
if (isDangerousDeserialization(node)) {
|
|
318
|
-
const args = node.arguments;
|
|
319
|
-
const hasUntrustedInput = args.some((arg) => isUntrustedInput(arg));
|
|
320
|
-
// Check if explicit validation is present
|
|
321
|
-
const isSafe = isSafeLibrary(node);
|
|
322
|
-
const filename = context.getFilename();
|
|
323
|
-
if (!isSafe && hasUntrustedInput) {
|
|
324
|
-
// Basic safety check
|
|
325
|
-
const safe = safetyChecker.isSafe(node, context);
|
|
326
|
-
if (!safe) {
|
|
327
|
-
// Determine message ID
|
|
328
|
-
let messageId = 'unsafeDeserialization';
|
|
329
|
-
// Check specifically for YAML
|
|
330
|
-
const calleeText = sourceCode.getText(node.callee);
|
|
331
|
-
if (calleeText.includes('yaml') || calleeText.includes('YAML')) {
|
|
332
|
-
messageId = 'unsafeYamlParsing';
|
|
333
|
-
}
|
|
334
|
-
// Check for generic dangerous functions
|
|
335
|
-
if (node.callee.type === 'Identifier' && ['eval', 'setTimeout', 'setInterval'].includes(node.callee.name)) {
|
|
336
|
-
messageId = 'dangerousEvalUsage';
|
|
337
|
-
}
|
|
338
|
-
context.report({
|
|
339
|
-
node,
|
|
340
|
-
messageId,
|
|
341
|
-
data: {
|
|
342
|
-
library: calleeText,
|
|
343
|
-
filePath: filename,
|
|
344
|
-
line: String(node.loc?.start.line ?? 0),
|
|
345
|
-
severity: 'HIGH',
|
|
346
|
-
safeAlternative: 'Use JSON.parse() or validated safe deserialization libraries',
|
|
347
|
-
},
|
|
348
|
-
suggest: messageId === 'dangerousEvalUsage' ? [{
|
|
349
|
-
messageId: 'useSafeDeserializer',
|
|
350
|
-
fix: (fixer) => {
|
|
351
|
-
return fixer.replaceText(node, `JSON.parse(${sourceCode.getText(node.arguments[0])})`);
|
|
352
|
-
}
|
|
353
|
-
}] : undefined
|
|
354
|
-
});
|
|
355
|
-
}
|
|
356
|
-
}
|
|
357
|
-
}
|
|
358
|
-
// Check for untrusted input in potentially safe functions
|
|
359
|
-
if (isSafeLibrary(node)) {
|
|
360
|
-
const args = node.arguments;
|
|
361
|
-
const hasUntrustedInput = args.some((arg) => {
|
|
362
|
-
// Check if it's validated
|
|
363
|
-
if (arg.type === 'Identifier' && validatedVariables.has(arg.name)) {
|
|
364
|
-
return false;
|
|
365
|
-
}
|
|
366
|
-
return isUntrustedInput(arg) && !isInputValidated(arg);
|
|
367
|
-
});
|
|
368
|
-
if (hasUntrustedInput) {
|
|
369
|
-
// Even JSON.parse can be unsafe if used on complex objects that get eval'd later
|
|
370
|
-
/* c8 ignore start -- safetyChecker requires JSDoc annotations not testable via RuleTester */
|
|
371
|
-
if (safetyChecker.isSafe(node, context)) {
|
|
372
|
-
return;
|
|
373
|
-
}
|
|
374
|
-
/* c8 ignore stop */
|
|
375
|
-
context.report({
|
|
376
|
-
node,
|
|
377
|
-
messageId: 'untrustedDeserializationInput',
|
|
378
|
-
data: {
|
|
379
|
-
filePath: context.getFilename(),
|
|
380
|
-
line: String(node.loc?.start.line ?? 0),
|
|
381
|
-
},
|
|
382
|
-
suggest: [
|
|
383
|
-
{
|
|
384
|
-
messageId: 'validateBeforeDeserialization',
|
|
385
|
-
fix: () => null
|
|
386
|
-
},
|
|
387
|
-
],
|
|
388
|
-
});
|
|
389
|
-
}
|
|
390
|
-
}
|
|
391
|
-
};
|
|
392
|
-
return {
|
|
393
|
-
// Track variable assignments from untrusted sources
|
|
394
|
-
VariableDeclaration(node) {
|
|
395
|
-
for (const declarator of node.declarations) {
|
|
396
|
-
if (declarator.id.type === 'Identifier' && declarator.init) {
|
|
397
|
-
// Check if the initializer comes from an untrusted source
|
|
398
|
-
if (isUntrustedInput(declarator.init)) {
|
|
399
|
-
untrustedVariables.add(declarator.id.name);
|
|
400
|
-
}
|
|
401
|
-
// Check if it's assigned from fs operations or other untrusted sources
|
|
402
|
-
if (declarator.init.type === 'CallExpression') {
|
|
403
|
-
const callee = declarator.init.callee;
|
|
404
|
-
if (callee.type === 'MemberExpression' &&
|
|
405
|
-
callee.object.type === 'Identifier' &&
|
|
406
|
-
callee.object.name === 'fs' &&
|
|
407
|
-
callee.property.type === 'Identifier' &&
|
|
408
|
-
['readFile', 'readFileSync'].includes(callee.property.name)) {
|
|
409
|
-
untrustedVariables.add(declarator.id.name);
|
|
410
|
-
}
|
|
411
|
-
}
|
|
412
|
-
}
|
|
413
|
-
}
|
|
414
|
-
},
|
|
415
|
-
// Track assignment expressions
|
|
416
|
-
AssignmentExpression(node) {
|
|
417
|
-
if (node.left.type === 'Identifier' && isUntrustedInput(node.right)) {
|
|
418
|
-
untrustedVariables.add(node.left.name);
|
|
419
|
-
}
|
|
420
|
-
},
|
|
421
|
-
// Check dangerous function calls
|
|
422
|
-
CallExpression(node) {
|
|
423
|
-
checkCallExpression(node);
|
|
424
|
-
},
|
|
425
|
-
NewExpression(node) {
|
|
426
|
-
checkCallExpression(node);
|
|
427
|
-
},
|
|
428
|
-
// Check for dangerous require/import patterns
|
|
429
|
-
VariableDeclarator(node) {
|
|
430
|
-
if (!node.init) {
|
|
431
|
-
return;
|
|
432
|
-
}
|
|
433
|
-
// Track variables assigned from validation functions
|
|
434
|
-
if (node.id.type === 'Identifier' &&
|
|
435
|
-
node.init.type === 'CallExpression' &&
|
|
436
|
-
node.init.callee.type === 'Identifier' &&
|
|
437
|
-
(validationFunctions.includes(node.init.callee.name) || trustedSanitizers.includes(node.init.callee.name))) {
|
|
438
|
-
validatedVariables.add(node.id.name);
|
|
439
|
-
}
|
|
440
|
-
// Check for require/import of dangerous libraries
|
|
441
|
-
if (node.init.type === 'CallExpression' &&
|
|
442
|
-
node.init.callee.type === 'Identifier' &&
|
|
443
|
-
node.init.callee.name === 'require') {
|
|
444
|
-
const requireArg = node.init.arguments[0];
|
|
445
|
-
if (requireArg?.type === 'Literal' && typeof requireArg.value === 'string') {
|
|
446
|
-
const moduleName = requireArg.value;
|
|
447
|
-
if (['node-serialize', 'serialize-javascript', 'js-yaml', 'yaml'].includes(moduleName)) {
|
|
448
|
-
// Check if this variable is used unsafely later
|
|
449
|
-
if (node.id.type === 'Identifier') {
|
|
450
|
-
// Look ahead to see if this library is used dangerously
|
|
451
|
-
// This is a simplified check - in practice, we'd need more sophisticated analysis
|
|
452
|
-
const variables = sourceCode.getDeclaredVariables(node);
|
|
453
|
-
for (const variable of variables) {
|
|
454
|
-
for (const reference of variable.references) {
|
|
455
|
-
const refNode = reference.identifier;
|
|
456
|
-
// Check if reference is part of a call to dangerous method
|
|
457
|
-
// e.g. serialize.unserialize()
|
|
458
|
-
if (refNode.parent && refNode.parent.type === 'MemberExpression' &&
|
|
459
|
-
refNode.parent.object === refNode) {
|
|
460
|
-
const memberExpr = refNode.parent;
|
|
461
|
-
const propertyName = memberExpr.property.type === 'Identifier' ? memberExpr.property.name : '';
|
|
462
|
-
if (['unserialize', 'deserialize', 'load', 'parse'].includes(propertyName)) {
|
|
463
|
-
const callExpr = memberExpr.parent;
|
|
464
|
-
if (callExpr && callExpr.type === 'CallExpression' && callExpr.callee === memberExpr) {
|
|
465
|
-
// FALSE POSITIVE REDUCTION
|
|
466
|
-
if (safetyChecker.isSafe(callExpr, context)) {
|
|
467
|
-
continue;
|
|
468
|
-
}
|
|
469
|
-
context.report({
|
|
470
|
-
node: callExpr,
|
|
471
|
-
messageId: 'unsafeDeserialization',
|
|
472
|
-
data: {
|
|
473
|
-
filePath: filename,
|
|
474
|
-
line: String(callExpr.loc?.start.line ?? 0),
|
|
475
|
-
severity: 'CRITICAL',
|
|
476
|
-
safeAlternative: 'Avoid using this library or use safe alternatives',
|
|
477
|
-
},
|
|
478
|
-
});
|
|
479
|
-
}
|
|
480
|
-
}
|
|
481
|
-
}
|
|
482
|
-
}
|
|
483
|
-
}
|
|
484
|
-
}
|
|
485
|
-
}
|
|
486
|
-
}
|
|
487
|
-
}
|
|
488
|
-
}
|
|
489
|
-
};
|
|
490
|
-
},
|
|
491
|
-
});
|
|
@@ -1,106 +0,0 @@
|
|
|
1
|
-
"use strict";
|
|
2
|
-
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
-
exports.noUnsafeDynamicRequire = void 0;
|
|
4
|
-
const eslint_devkit_1 = require("@interlace/eslint-devkit");
|
|
5
|
-
const eslint_devkit_2 = require("@interlace/eslint-devkit");
|
|
6
|
-
exports.noUnsafeDynamicRequire = (0, eslint_devkit_2.createRule)({
|
|
7
|
-
name: 'no-unsafe-dynamic-require',
|
|
8
|
-
meta: {
|
|
9
|
-
type: 'problem',
|
|
10
|
-
docs: {
|
|
11
|
-
description: 'Prevent unsafe dynamic require() calls that could enable code injection',
|
|
12
|
-
},
|
|
13
|
-
fixable: 'code',
|
|
14
|
-
hasSuggestions: false,
|
|
15
|
-
messages: {
|
|
16
|
-
unsafeDynamicRequire: (0, eslint_devkit_1.formatLLMMessage)({
|
|
17
|
-
icon: eslint_devkit_1.MessageIcons.SECURITY,
|
|
18
|
-
issueName: 'Dynamic require()',
|
|
19
|
-
cwe: 'CWE-95',
|
|
20
|
-
description: 'Dynamic require() detected',
|
|
21
|
-
severity: 'CRITICAL',
|
|
22
|
-
fix: 'Use allowlist: const ALLOWED = ["mod1", "mod2"]; if (!ALLOWED.includes(name)) throw Error("Not allowed")',
|
|
23
|
-
documentationLink: 'https://owasp.org/www-community/attacks/Code_Injection',
|
|
24
|
-
}),
|
|
25
|
-
},
|
|
26
|
-
schema: [
|
|
27
|
-
{
|
|
28
|
-
type: 'object',
|
|
29
|
-
properties: {
|
|
30
|
-
allowDynamicImport: {
|
|
31
|
-
type: 'boolean',
|
|
32
|
-
default: false,
|
|
33
|
-
},
|
|
34
|
-
},
|
|
35
|
-
additionalProperties: false,
|
|
36
|
-
},
|
|
37
|
-
],
|
|
38
|
-
},
|
|
39
|
-
defaultOptions: [
|
|
40
|
-
{
|
|
41
|
-
allowDynamicImport: false,
|
|
42
|
-
},
|
|
43
|
-
],
|
|
44
|
-
create(context) {
|
|
45
|
-
/**
|
|
46
|
-
* Track variables that reference require
|
|
47
|
-
* Maps variable name to the node where it was assigned
|
|
48
|
-
*/
|
|
49
|
-
const requireVariables = new Set();
|
|
50
|
-
/**
|
|
51
|
-
* Check if a node is a reference to require
|
|
52
|
-
*/
|
|
53
|
-
const isRequireReference = (node) => {
|
|
54
|
-
if (node.type === 'Identifier' && node.name === 'require') {
|
|
55
|
-
return true;
|
|
56
|
-
}
|
|
57
|
-
if (node.type === 'Identifier' && requireVariables.has(node.name)) {
|
|
58
|
-
return true;
|
|
59
|
-
}
|
|
60
|
-
return false;
|
|
61
|
-
};
|
|
62
|
-
/**
|
|
63
|
-
* Check if argument is dynamic (not a literal)
|
|
64
|
-
*/
|
|
65
|
-
const isDynamicArgument = (arg) => {
|
|
66
|
-
if (arg.type === 'Literal')
|
|
67
|
-
return false;
|
|
68
|
-
if (arg.type === 'TemplateLiteral' && arg.expressions.length === 0)
|
|
69
|
-
return false;
|
|
70
|
-
return true;
|
|
71
|
-
};
|
|
72
|
-
return {
|
|
73
|
-
VariableDeclarator(node) {
|
|
74
|
-
// Track when require is assigned to a variable
|
|
75
|
-
if (node.id.type === 'Identifier' && node.init) {
|
|
76
|
-
if (node.init.type === 'Identifier' && node.init.name === 'require') {
|
|
77
|
-
requireVariables.add(node.id.name);
|
|
78
|
-
}
|
|
79
|
-
}
|
|
80
|
-
},
|
|
81
|
-
CallExpression(node) {
|
|
82
|
-
// Check for require() calls (direct or via variable)
|
|
83
|
-
if (node.callee.type !== 'Identifier') {
|
|
84
|
-
return;
|
|
85
|
-
}
|
|
86
|
-
// Check if callee is require or a variable that references require
|
|
87
|
-
if (!isRequireReference(node.callee)) {
|
|
88
|
-
return;
|
|
89
|
-
}
|
|
90
|
-
// Must have at least one argument
|
|
91
|
-
if (node.arguments.length === 0)
|
|
92
|
-
return;
|
|
93
|
-
const firstArg = node.arguments[0];
|
|
94
|
-
if (firstArg.type === 'SpreadElement')
|
|
95
|
-
return;
|
|
96
|
-
// Check if dynamic
|
|
97
|
-
if (!isDynamicArgument(firstArg))
|
|
98
|
-
return;
|
|
99
|
-
context.report({
|
|
100
|
-
node,
|
|
101
|
-
messageId: 'unsafeDynamicRequire',
|
|
102
|
-
});
|
|
103
|
-
},
|
|
104
|
-
};
|
|
105
|
-
},
|
|
106
|
-
});
|
|
@@ -1,9 +0,0 @@
|
|
|
1
|
-
export interface Options {
|
|
2
|
-
/** Allow literal string patterns. Default: false */
|
|
3
|
-
allowLiterals?: boolean;
|
|
4
|
-
/** Trusted functions that escape input. Default: ['escapeRegex', 'escape', 'sanitize'] */
|
|
5
|
-
trustedEscapingFunctions?: string[];
|
|
6
|
-
/** Maximum pattern length for dynamic regex. Default: 100 */
|
|
7
|
-
maxPatternLength?: number;
|
|
8
|
-
}
|
|
9
|
-
export declare const noUnsafeRegexConstruction: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|