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