eslint-plugin-secure-coding 1.0.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/AGENTS.md +196 -0
- package/CHANGELOG.md +105 -0
- package/LICENSE +23 -0
- package/README.md +377 -0
- package/package.json +80 -0
- package/src/index.d.ts +32 -0
- package/src/index.js +345 -0
- package/src/index.js.map +1 -0
- package/src/rules/security/database-injection.d.ts +13 -0
- package/src/rules/security/database-injection.js +407 -0
- package/src/rules/security/database-injection.js.map +1 -0
- package/src/rules/security/detect-child-process.d.ts +11 -0
- package/src/rules/security/detect-child-process.js +460 -0
- package/src/rules/security/detect-child-process.js.map +1 -0
- package/src/rules/security/detect-eval-with-expression.d.ts +9 -0
- package/src/rules/security/detect-eval-with-expression.js +393 -0
- package/src/rules/security/detect-eval-with-expression.js.map +1 -0
- package/src/rules/security/detect-non-literal-fs-filename.d.ts +7 -0
- package/src/rules/security/detect-non-literal-fs-filename.js +322 -0
- package/src/rules/security/detect-non-literal-fs-filename.js.map +1 -0
- package/src/rules/security/detect-non-literal-regexp.d.ts +9 -0
- package/src/rules/security/detect-non-literal-regexp.js +387 -0
- package/src/rules/security/detect-non-literal-regexp.js.map +1 -0
- package/src/rules/security/detect-object-injection.d.ts +11 -0
- package/src/rules/security/detect-object-injection.js +411 -0
- package/src/rules/security/detect-object-injection.js.map +1 -0
- package/src/rules/security/no-buffer-overread.d.ts +14 -0
- package/src/rules/security/no-buffer-overread.js +519 -0
- package/src/rules/security/no-buffer-overread.js.map +1 -0
- package/src/rules/security/no-clickjacking.d.ts +10 -0
- package/src/rules/security/no-clickjacking.js +381 -0
- package/src/rules/security/no-clickjacking.js.map +1 -0
- package/src/rules/security/no-directive-injection.d.ts +12 -0
- package/src/rules/security/no-directive-injection.js +446 -0
- package/src/rules/security/no-directive-injection.js.map +1 -0
- package/src/rules/security/no-document-cookie.d.ts +5 -0
- package/src/rules/security/no-document-cookie.js +90 -0
- package/src/rules/security/no-document-cookie.js.map +1 -0
- package/src/rules/security/no-electron-security-issues.d.ts +10 -0
- package/src/rules/security/no-electron-security-issues.js +421 -0
- package/src/rules/security/no-electron-security-issues.js.map +1 -0
- package/src/rules/security/no-exposed-sensitive-data.d.ts +11 -0
- package/src/rules/security/no-exposed-sensitive-data.js +341 -0
- package/src/rules/security/no-exposed-sensitive-data.js.map +1 -0
- package/src/rules/security/no-format-string-injection.d.ts +17 -0
- package/src/rules/security/no-format-string-injection.js +653 -0
- package/src/rules/security/no-format-string-injection.js.map +1 -0
- package/src/rules/security/no-graphql-injection.d.ts +12 -0
- package/src/rules/security/no-graphql-injection.js +410 -0
- package/src/rules/security/no-graphql-injection.js.map +1 -0
- package/src/rules/security/no-hardcoded-credentials.d.ts +26 -0
- package/src/rules/security/no-hardcoded-credentials.js +377 -0
- package/src/rules/security/no-hardcoded-credentials.js.map +1 -0
- package/src/rules/security/no-improper-sanitization.d.ts +12 -0
- package/src/rules/security/no-improper-sanitization.js +408 -0
- package/src/rules/security/no-improper-sanitization.js.map +1 -0
- package/src/rules/security/no-improper-type-validation.d.ts +10 -0
- package/src/rules/security/no-improper-type-validation.js +420 -0
- package/src/rules/security/no-improper-type-validation.js.map +1 -0
- package/src/rules/security/no-insecure-comparison.d.ts +7 -0
- package/src/rules/security/no-insecure-comparison.js +125 -0
- package/src/rules/security/no-insecure-comparison.js.map +1 -0
- package/src/rules/security/no-insecure-cookie-settings.d.ts +9 -0
- package/src/rules/security/no-insecure-cookie-settings.js +305 -0
- package/src/rules/security/no-insecure-cookie-settings.js.map +1 -0
- package/src/rules/security/no-insecure-jwt.d.ts +10 -0
- package/src/rules/security/no-insecure-jwt.js +338 -0
- package/src/rules/security/no-insecure-jwt.js.map +1 -0
- package/src/rules/security/no-insecure-redirects.d.ts +7 -0
- package/src/rules/security/no-insecure-redirects.js +215 -0
- package/src/rules/security/no-insecure-redirects.js.map +1 -0
- package/src/rules/security/no-insufficient-postmessage-validation.d.ts +14 -0
- package/src/rules/security/no-insufficient-postmessage-validation.js +390 -0
- package/src/rules/security/no-insufficient-postmessage-validation.js.map +1 -0
- package/src/rules/security/no-insufficient-random.d.ts +9 -0
- package/src/rules/security/no-insufficient-random.js +207 -0
- package/src/rules/security/no-insufficient-random.js.map +1 -0
- package/src/rules/security/no-ldap-injection.d.ts +10 -0
- package/src/rules/security/no-ldap-injection.js +449 -0
- package/src/rules/security/no-ldap-injection.js.map +1 -0
- package/src/rules/security/no-missing-authentication.d.ts +13 -0
- package/src/rules/security/no-missing-authentication.js +322 -0
- package/src/rules/security/no-missing-authentication.js.map +1 -0
- package/src/rules/security/no-missing-cors-check.d.ts +9 -0
- package/src/rules/security/no-missing-cors-check.js +449 -0
- package/src/rules/security/no-missing-cors-check.js.map +1 -0
- package/src/rules/security/no-missing-csrf-protection.d.ts +11 -0
- package/src/rules/security/no-missing-csrf-protection.js +183 -0
- package/src/rules/security/no-missing-csrf-protection.js.map +1 -0
- package/src/rules/security/no-missing-security-headers.d.ts +7 -0
- package/src/rules/security/no-missing-security-headers.js +217 -0
- package/src/rules/security/no-missing-security-headers.js.map +1 -0
- package/src/rules/security/no-privilege-escalation.d.ts +13 -0
- package/src/rules/security/no-privilege-escalation.js +321 -0
- package/src/rules/security/no-privilege-escalation.js.map +1 -0
- package/src/rules/security/no-redos-vulnerable-regex.d.ts +7 -0
- package/src/rules/security/no-redos-vulnerable-regex.js +307 -0
- package/src/rules/security/no-redos-vulnerable-regex.js.map +1 -0
- package/src/rules/security/no-sensitive-data-exposure.d.ts +11 -0
- package/src/rules/security/no-sensitive-data-exposure.js +251 -0
- package/src/rules/security/no-sensitive-data-exposure.js.map +1 -0
- package/src/rules/security/no-sql-injection.d.ts +10 -0
- package/src/rules/security/no-sql-injection.js +332 -0
- package/src/rules/security/no-sql-injection.js.map +1 -0
- package/src/rules/security/no-timing-attack.d.ts +10 -0
- package/src/rules/security/no-timing-attack.js +358 -0
- package/src/rules/security/no-timing-attack.js.map +1 -0
- package/src/rules/security/no-toctou-vulnerability.d.ts +7 -0
- package/src/rules/security/no-toctou-vulnerability.js +165 -0
- package/src/rules/security/no-toctou-vulnerability.js.map +1 -0
- package/src/rules/security/no-unchecked-loop-condition.d.ts +12 -0
- package/src/rules/security/no-unchecked-loop-condition.js +635 -0
- package/src/rules/security/no-unchecked-loop-condition.js.map +1 -0
- package/src/rules/security/no-unencrypted-transmission.d.ts +11 -0
- package/src/rules/security/no-unencrypted-transmission.js +237 -0
- package/src/rules/security/no-unencrypted-transmission.js.map +1 -0
- package/src/rules/security/no-unescaped-url-parameter.d.ts +9 -0
- package/src/rules/security/no-unescaped-url-parameter.js +266 -0
- package/src/rules/security/no-unescaped-url-parameter.js.map +1 -0
- package/src/rules/security/no-unlimited-resource-allocation.d.ts +12 -0
- package/src/rules/security/no-unlimited-resource-allocation.js +659 -0
- package/src/rules/security/no-unlimited-resource-allocation.js.map +1 -0
- package/src/rules/security/no-unsafe-deserialization.d.ts +10 -0
- package/src/rules/security/no-unsafe-deserialization.js +501 -0
- package/src/rules/security/no-unsafe-deserialization.js.map +1 -0
- package/src/rules/security/no-unsafe-dynamic-require.d.ts +5 -0
- package/src/rules/security/no-unsafe-dynamic-require.js +107 -0
- package/src/rules/security/no-unsafe-dynamic-require.js.map +1 -0
- package/src/rules/security/no-unsafe-regex-construction.d.ts +9 -0
- package/src/rules/security/no-unsafe-regex-construction.js +292 -0
- package/src/rules/security/no-unsafe-regex-construction.js.map +1 -0
- package/src/rules/security/no-unsanitized-html.d.ts +9 -0
- package/src/rules/security/no-unsanitized-html.js +347 -0
- package/src/rules/security/no-unsanitized-html.js.map +1 -0
- package/src/rules/security/no-unvalidated-user-input.d.ts +9 -0
- package/src/rules/security/no-unvalidated-user-input.js +418 -0
- package/src/rules/security/no-unvalidated-user-input.js.map +1 -0
- package/src/rules/security/no-weak-crypto.d.ts +11 -0
- package/src/rules/security/no-weak-crypto.js +350 -0
- package/src/rules/security/no-weak-crypto.js.map +1 -0
- package/src/rules/security/no-weak-password-recovery.d.ts +12 -0
- package/src/rules/security/no-weak-password-recovery.js +401 -0
- package/src/rules/security/no-weak-password-recovery.js.map +1 -0
- package/src/rules/security/no-xpath-injection.d.ts +10 -0
- package/src/rules/security/no-xpath-injection.js +487 -0
- package/src/rules/security/no-xpath-injection.js.map +1 -0
- package/src/rules/security/no-xxe-injection.d.ts +7 -0
- package/src/rules/security/no-xxe-injection.js +270 -0
- package/src/rules/security/no-xxe-injection.js.map +1 -0
- package/src/rules/security/no-zip-slip.d.ts +9 -0
- package/src/rules/security/no-zip-slip.js +446 -0
- package/src/rules/security/no-zip-slip.js.map +1 -0
- package/src/types/index.d.ts +131 -0
- package/src/types/index.js +18 -0
- package/src/types/index.js.map +1 -0
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-insecure-jwt.js","sourceRoot":"","sources":["../../../../../../packages/eslint-plugin-secure-coding/src/rules/security/no-insecure-jwt.ts"],"names":[],"mappings":";;;AAaA,4DAAsD;AACtD,4DAA0E;AAC1E,4DAIkC;AA2BrB,QAAA,aAAa,GAAG,IAAA,0BAAU,EAA0B;IAC/D,IAAI,EAAE,iBAAiB;IACvB,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,oEAAoE;SAClF;QACD,OAAO,EAAE,MAAM;QACf,cAAc,EAAE,IAAI;QACpB,QAAQ,EAAE;YACR,oBAAoB,EAAE,IAAA,gCAAgB,EAAC;gBACrC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,wBAAwB;gBACnC,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,uCAAuC;gBACpD,QAAQ,EAAE,UAAU;gBACpB,GAAG,EAAE,4DAA4D;gBACjE,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,4BAA4B,EAAE,IAAA,gCAAgB,EAAC;gBAC7C,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,oCAAoC;gBAC/C,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,2CAA2C;gBACxD,QAAQ,EAAE,UAAU;gBACpB,GAAG,EAAE,0CAA0C;gBAC/C,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,aAAa,EAAE,IAAA,gCAAgB,EAAC;gBAC9B,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,iBAAiB;gBAC5B,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,0CAA0C;gBACvD,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,6CAA6C;gBAClD,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,oBAAoB,EAAE,IAAA,gCAAgB,EAAC;gBACrC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,wBAAwB;gBACnC,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,oCAAoC;gBACjD,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,8CAA8C;gBACnD,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,oBAAoB;gBAC/B,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,qCAAqC;gBAClD,QAAQ,EAAE,QAAQ;gBAClB,GAAG,EAAE,uDAAuD;gBAC5D,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,mBAAmB,EAAE,IAAA,gCAAgB,EAAC;gBACpC,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,wBAAwB;gBACnC,WAAW,EAAE,kCAAkC;gBAC/C,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,8CAA8C;gBACnD,iBAAiB,EAAE,4CAA4C;aAChE,CAAC;YACF,iBAAiB,EAAE,IAAA,gCAAgB,EAAC;gBAClC,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,qBAAqB;gBAChC,WAAW,EAAE,kDAAkD;gBAC/D,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,qCAAqC;gBAC1C,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,0BAA0B,EAAE,IAAA,gCAAgB,EAAC;gBAC3C,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,2BAA2B;gBACtC,WAAW,EAAE,iCAAiC;gBAC9C,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,0CAA0C;gBAC/C,iBAAiB,EAAE,4CAA4C;aAChE,CAAC;YACF,yBAAyB,EAAE,IAAA,gCAAgB,EAAC;gBAC1C,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,+BAA+B;gBAC1C,WAAW,EAAE,gCAAgC;gBAC7C,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,kDAAkD;gBACvD,iBAAiB,EAAE,qCAAqC;aACzD,CAAC;YACF,qBAAqB,EAAE,IAAA,gCAAgB,EAAC;gBACtC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,yBAAyB;gBACpC,WAAW,EAAE,sCAAsC;gBACnD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,kDAAkD;gBACvD,iBAAiB,EAAE,oCAAoC;aACxD,CAAC;SACH;QACD,MAAM,EAAE;YACN;gBACE,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACV,yBAAyB,EAAE;wBACzB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;qBACZ;oBACD,eAAe,EAAE;wBACf,IAAI,EAAE,QAAQ;wBACd,OAAO,EAAE,EAAE;wBACX,OAAO,EAAE,EAAE;qBACZ;oBACD,mBAAmB,EAAE;wBACnB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,CAAC,cAAc,EAAE,MAAM,EAAE,KAAK,CAAC;qBACzC;oBACD,iBAAiB,EAAE;wBACjB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,yDAAyD;qBACvE;oBACD,kBAAkB,EAAE;wBAClB,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;wBACX,WAAW,EAAE,0DAA0D;qBACxE;oBACD,UAAU,EAAE;wBACV,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,KAAK;wBACd,WAAW,EAAE,oDAAoD;qBAClE;iBACF;gBACD,oBAAoB,EAAE,KAAK;aAC5B;SACF;KACF;IACD,cAAc,EAAE;QACd;YACE,yBAAyB,EAAE,EAAE;YAC7B,eAAe,EAAE,EAAE;YACnB,mBAAmB,EAAE,CAAC,cAAc,EAAE,MAAM,EAAE,KAAK,CAAC;YACpD,iBAAiB,EAAE,EAAE;YACrB,kBAAkB,EAAE,EAAE;YACtB,UAAU,EAAE,KAAK;SAClB;KACF;IACD,MAAM,CAAC,OAAsD;QAC3D,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACzC,MAAM,EACJ,eAAe,GAAG,EAAE,EACpB,mBAAmB,GAAG,CAAC,cAAc,EAAE,MAAM,EAAE,KAAK,CAAC,EACrD,iBAAiB,GAAG,EAAE,EACtB,kBAAkB,GAAG,EAAE,EACvB,UAAU,GAAG,KAAK,GACnB,GAAY,OAAO,CAAC;QAErB,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;QAC5D,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,IAAI,OAAO,CAAC,WAAW,EAAE,CAAC;QAE3D,qDAAqD;QACrD,MAAM,aAAa,GAAG,IAAA,mCAAmB,EAAC;YACxC,iBAAiB;YACjB,kBAAkB;YAClB,kBAAkB,EAAE,EAAE;YACtB,UAAU;SACX,CAAC,CAAC;QAEH;;WAEG;QACH,MAAM,YAAY,GAAG,CAAC,UAAyB,EAAW,EAAE;YAC1D,IAAI,UAAU,CAAC,IAAI,KAAK,SAAS,IAAI,OAAO,UAAU,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;gBAC1E,OAAO,UAAU,CAAC,KAAK,CAAC,MAAM,GAAG,eAAe,CAAC;YACnD,CAAC;YACD,OAAO,KAAK,CAAC,CAAC,kDAAkD;QAClE,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,wBAAwB,GAAG,CAAC,OAAgC,EAAW,EAAE;YAC7E,kCAAkC;YAClC,IACE,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB;gBAC1C,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY;gBAC7C,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,QAAQ,EACzC,CAAC;gBACD,OAAO,IAAI,CAAC;YACd,CAAC;YAED,iCAAiC;YACjC,OAAO,IAAA,iCAAiB,EAAC,OAAO,EAAE,OAAO,EAAE,kBAAkB,CAAC,CAAC;QACjE,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,mBAAmB,GAAG,CAAC,IAA6B,EAAW,EAAE;YACrE,0DAA0D;YAC1D,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB,EAAE,CAAC;gBAC5C,OAAO,KAAK,CAAC;YACf,CAAC;YAED,uCAAuC;YACvC,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC;YAClC,IAAI,MAAM,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gBACjC,OAAO,mBAAmB,CAAC,QAAQ,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,CAAC;YACjE,CAAC;YAED,OAAO,KAAK,CAAC;QACf,CAAC,CAAC;QAEF;;WAEG;QACH,MAAM,cAAc,GAAG,CAAC,IAA6B,EAAE,EAAE;YACvD,MAAM,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;YAE5C,oCAAoC;YACpC,MAAM,gBAAgB,GAAG,0BAA0B,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAErE,8BAA8B;YAC9B,MAAM,gBAAgB,GAAG,iDAAiD,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAC5F,MAAM,gBAAgB,GAAG,gEAAgE,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAE3G,OAAO;gBACL,UAAU;gBACV,gBAAgB;gBAChB,gBAAgB;gBAChB,gBAAgB;gBAChB,YAAY,EAAE,aAAa,CAAC,IAAI,CAAC,UAAU,CAAC;gBAC5C,YAAY,EAAE,aAAa,CAAC,IAAI,CAAC,UAAU,CAAC;aAC7C,CAAC;QACJ,CAAC,CAAC;QAEF,OAAO;YACL,iCAAiC;YACjC,cAAc,CAAC,IAA6B;gBAC1C,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC/B,OAAO;gBACT,CAAC;gBAED,MAAM,OAAO,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC;gBAErC,qDAAqD;gBACrD,IAAI,OAAO,CAAC,gBAAgB,EAAE,CAAC;oBAC7B,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI;wBACJ,SAAS,EAAE,sBAAsB;wBACjC,IAAI,EAAE;4BACJ,QAAQ,EAAE,QAAQ;4BAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;yBACxC;qBACF,CAAC,CAAC;oBACH,OAAO;gBACT,CAAC;gBAED,qDAAqD;gBACrD,IAAI,OAAO,CAAC,gBAAgB,IAAI,IAAI,CAAC,SAAS,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;oBAC3D,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;oBACpC,IAAI,YAAY,CAAC,SAAS,CAAC,EAAE,CAAC;wBAC5B,OAAO,CAAC,MAAM,CAAC;4BACb,IAAI;4BACJ,SAAS,EAAE,eAAe;4BAC1B,IAAI,EAAE;gCACJ,QAAQ,EAAE,QAAQ;gCAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;6BACxC;yBACF,CAAC,CAAC;wBACH,OAAO;oBACT,CAAC;gBACH,CAAC;gBAED,uDAAuD;gBACvD,IAAI,OAAO,CAAC,YAAY,IAAI,CAAC,OAAO,CAAC,YAAY,EAAE,CAAC;oBAClD,sDAAsD;oBACtD,IAAI,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,OAAO,CAAC,EAAE,CAAC;wBACxC,OAAO;oBACT,CAAC;oBAED,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI;wBACJ,SAAS,EAAE,8BAA8B;wBACzC,IAAI,EAAE;4BACJ,QAAQ,EAAE,QAAQ;4BAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;yBACxC;wBACD,OAAO,EAAE;4BACP;gCACE,SAAS,EAAE,mBAAmB;gCAC9B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI,CAAC,+BAA+B;6BAChD;yBACF;qBACF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;YAED,8DAA8D;YAC9D,kBAAkB,CAAC,IAAiC;gBAClD,IAAI,CAAC,IAAI,CAAC,IAAI,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,gBAAgB,EAAE,CAAC;oBACtD,OAAO;gBACT,CAAC;gBAED,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC;gBAC3B,IAAI,CAAC,mBAAmB,CAAC,QAAQ,CAAC,EAAE,CAAC;oBACnC,OAAO;gBACT,CAAC;gBAED,MAAM,OAAO,GAAG,cAAc,CAAC,QAAQ,CAAC,CAAC;gBAEzC,6CAA6C;gBAC7C,IAAI,OAAO,CAAC,YAAY,IAAI,CAAC,OAAO,CAAC,YAAY,EAAE,CAAC;oBAClD,2BAA2B;oBAC3B,IAAI,aAAa,CAAC,MAAM,CAAC,QAAQ,EAAE,OAAO,CAAC,EAAE,CAAC;wBAC5C,OAAO;oBACT,CAAC;oBAED,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI;wBACJ,SAAS,EAAE,sBAAsB;wBACjC,IAAI,EAAE;4BACJ,QAAQ,EAAE,QAAQ;4BAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;yBACxC;qBACF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;YAED,iDAAiD;YACjD,OAAO,CAAC,IAAsB;gBAC5B,IAAI,OAAO,IAAI,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;oBACnC,OAAO;gBACT,CAAC;gBAED,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC;gBAEzB,mCAAmC;gBACnC,IAAI,KAAK,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC,CAAC,gBAAgB;oBAC5E,qCAAqC;oBACrC,IAAI,OAAO,GAA8B,IAAI,CAAC;oBAC9C,IAAI,UAAU,GAAG,KAAK,CAAC;oBAEvB,6DAA6D;oBAC7D,OAAO,OAAO,IAAI,CAAC,UAAU,EAAE,CAAC;wBAC9B,IAAI,OAAO,CAAC,IAAI,KAAK,gBAAgB,IAAI,wBAAwB,CAAC,OAAO,CAAC,EAAE,CAAC;4BAC3E,UAAU,GAAG,IAAI,CAAC;4BAClB,MAAM;wBACR,CAAC;wBACD,OAAO,GAAG,OAAO,CAAC,MAAuB,CAAC;oBAC5C,CAAC;oBAED,IAAI,CAAC,UAAU,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,OAAO,CAAC,EAAE,CAAC;wBACxD,OAAO,CAAC,MAAM,CAAC;4BACb,IAAI;4BACJ,SAAS,EAAE,kBAAkB;4BAC7B,IAAI,EAAE;gCACJ,QAAQ,EAAE,QAAQ;gCAClB,IAAI,EAAE,MAAM,CAAC,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,CAAC;6BACxC;yBACF,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}
|
|
@@ -0,0 +1,7 @@
|
|
|
1
|
+
export interface Options {
|
|
2
|
+
/** Ignore in test files. Default: true */
|
|
3
|
+
ignoreInTests?: boolean;
|
|
4
|
+
/** Allowed redirect domains. Default: [] */
|
|
5
|
+
allowedDomains?: string[];
|
|
6
|
+
}
|
|
7
|
+
export declare const noInsecureRedirects: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|
|
@@ -0,0 +1,215 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.noInsecureRedirects = void 0;
|
|
4
|
+
const eslint_devkit_1 = require("@interlace/eslint-devkit");
|
|
5
|
+
const eslint_devkit_2 = require("@interlace/eslint-devkit");
|
|
6
|
+
/**
|
|
7
|
+
* Check if redirect URL is validated
|
|
8
|
+
*/
|
|
9
|
+
function isRedirectValidated(node, sourceCode) {
|
|
10
|
+
const callText = sourceCode.getText(node);
|
|
11
|
+
// Check if URL is from user input (req.query, req.body, req.params)
|
|
12
|
+
const userInputPattern = /\b(req\.(query|body|params)|window\.location|document\.location)\b/;
|
|
13
|
+
if (!userInputPattern.test(callText)) {
|
|
14
|
+
// Not from user input, assume safe
|
|
15
|
+
return true;
|
|
16
|
+
}
|
|
17
|
+
// Look for validation patterns in the surrounding code
|
|
18
|
+
// This is a simplified static analysis - in practice, would need data flow analysis
|
|
19
|
+
const program = sourceCode.ast;
|
|
20
|
+
const nodeStart = node.loc?.start;
|
|
21
|
+
const nodeEnd = node.loc?.end;
|
|
22
|
+
if (!nodeStart || !nodeEnd || !program) {
|
|
23
|
+
return false;
|
|
24
|
+
}
|
|
25
|
+
// Check for validation function calls before this redirect
|
|
26
|
+
const validationPatterns = [
|
|
27
|
+
/\b(validateUrl|validateRedirect|isValidUrl|isSafeUrl)\s*\(/,
|
|
28
|
+
/\b(whitelist|allowedDomains|permittedUrls)\s*\./,
|
|
29
|
+
/\b(url\.hostname|url\.host)\s*===/,
|
|
30
|
+
/\ballowedDomains\.includes\s*\(/,
|
|
31
|
+
/\bstartsWith\s*\(\s*['"]/,
|
|
32
|
+
/\bendsWith\s*\(\s*['"]/,
|
|
33
|
+
/\bindexOf\s*\(\s*['"]/,
|
|
34
|
+
/\bmatch\s*\(\s*\//,
|
|
35
|
+
];
|
|
36
|
+
// Look for validation in the same function scope
|
|
37
|
+
let current = node;
|
|
38
|
+
let depth = 0;
|
|
39
|
+
const maxDepth = 20;
|
|
40
|
+
while (current && depth < maxDepth) {
|
|
41
|
+
// Check siblings before this node
|
|
42
|
+
const parent = current.parent;
|
|
43
|
+
if (!parent)
|
|
44
|
+
break;
|
|
45
|
+
if (parent.type === 'BlockStatement') {
|
|
46
|
+
const body = parent.body;
|
|
47
|
+
const currentIndex = body.indexOf(current);
|
|
48
|
+
// Check previous statements in the same block
|
|
49
|
+
for (let i = currentIndex - 1; i >= 0 && i >= currentIndex - 5; i--) {
|
|
50
|
+
const stmt = body[i];
|
|
51
|
+
const stmtText = sourceCode.getText(stmt);
|
|
52
|
+
if (validationPatterns.some(pattern => pattern.test(stmtText))) {
|
|
53
|
+
return true; // Found validation
|
|
54
|
+
}
|
|
55
|
+
}
|
|
56
|
+
}
|
|
57
|
+
current = parent;
|
|
58
|
+
depth++;
|
|
59
|
+
}
|
|
60
|
+
// No validation found
|
|
61
|
+
return false;
|
|
62
|
+
}
|
|
63
|
+
exports.noInsecureRedirects = (0, eslint_devkit_2.createRule)({
|
|
64
|
+
name: 'no-insecure-redirects',
|
|
65
|
+
meta: {
|
|
66
|
+
type: 'problem',
|
|
67
|
+
docs: {
|
|
68
|
+
description: 'Detects open redirect vulnerabilities',
|
|
69
|
+
},
|
|
70
|
+
hasSuggestions: true,
|
|
71
|
+
messages: {
|
|
72
|
+
insecureRedirect: (0, eslint_devkit_1.formatLLMMessage)({
|
|
73
|
+
icon: eslint_devkit_1.MessageIcons.SECURITY,
|
|
74
|
+
issueName: 'Open redirect',
|
|
75
|
+
cwe: 'CWE-601',
|
|
76
|
+
description: 'Unvalidated redirect detected - user-controlled URL',
|
|
77
|
+
severity: 'HIGH',
|
|
78
|
+
fix: 'Whitelist allowed domains or validate redirect target',
|
|
79
|
+
documentationLink: 'https://owasp.org/www-community/vulnerabilities/Unvalidated_Redirects_and_Forwards',
|
|
80
|
+
}),
|
|
81
|
+
whitelistDomains: (0, eslint_devkit_1.formatLLMMessage)({
|
|
82
|
+
icon: eslint_devkit_1.MessageIcons.INFO,
|
|
83
|
+
issueName: 'Whitelist Domains',
|
|
84
|
+
description: 'Whitelist allowed redirect domains',
|
|
85
|
+
severity: 'LOW',
|
|
86
|
+
fix: 'if (allowedDomains.includes(url.hostname)) redirect(url)',
|
|
87
|
+
documentationLink: 'https://owasp.org/www-community/vulnerabilities/Unvalidated_Redirects_and_Forwards',
|
|
88
|
+
}),
|
|
89
|
+
validateRedirect: (0, eslint_devkit_1.formatLLMMessage)({
|
|
90
|
+
icon: eslint_devkit_1.MessageIcons.INFO,
|
|
91
|
+
issueName: 'Validate Redirect',
|
|
92
|
+
description: 'Validate redirect URL before use',
|
|
93
|
+
severity: 'LOW',
|
|
94
|
+
fix: 'validateRedirectUrl(userInput) before redirect',
|
|
95
|
+
documentationLink: 'https://owasp.org/www-community/vulnerabilities/Unvalidated_Redirects_and_Forwards',
|
|
96
|
+
}),
|
|
97
|
+
useRelativeUrl: (0, eslint_devkit_1.formatLLMMessage)({
|
|
98
|
+
icon: eslint_devkit_1.MessageIcons.INFO,
|
|
99
|
+
issueName: 'Use Relative URL',
|
|
100
|
+
description: 'Use relative URLs for internal redirects',
|
|
101
|
+
severity: 'LOW',
|
|
102
|
+
fix: 'redirect("/internal/path") instead of absolute URLs',
|
|
103
|
+
documentationLink: 'https://owasp.org/www-community/vulnerabilities/Unvalidated_Redirects_and_Forwards',
|
|
104
|
+
}),
|
|
105
|
+
},
|
|
106
|
+
schema: [
|
|
107
|
+
{
|
|
108
|
+
type: 'object',
|
|
109
|
+
properties: {
|
|
110
|
+
ignoreInTests: {
|
|
111
|
+
type: 'boolean',
|
|
112
|
+
default: true,
|
|
113
|
+
},
|
|
114
|
+
allowedDomains: {
|
|
115
|
+
type: 'array',
|
|
116
|
+
items: { type: 'string' },
|
|
117
|
+
default: [],
|
|
118
|
+
},
|
|
119
|
+
},
|
|
120
|
+
additionalProperties: false,
|
|
121
|
+
},
|
|
122
|
+
],
|
|
123
|
+
},
|
|
124
|
+
defaultOptions: [
|
|
125
|
+
{
|
|
126
|
+
ignoreInTests: true,
|
|
127
|
+
allowedDomains: [],
|
|
128
|
+
},
|
|
129
|
+
],
|
|
130
|
+
create(context, [options = {}]) {
|
|
131
|
+
const { ignoreInTests = true } = options || {};
|
|
132
|
+
const filename = context.getFilename();
|
|
133
|
+
const isTestFile = ignoreInTests && /\.(test|spec)\.(ts|tsx|js|jsx)$/.test(filename);
|
|
134
|
+
if (isTestFile) {
|
|
135
|
+
return {};
|
|
136
|
+
}
|
|
137
|
+
const sourceCode = context.sourceCode || context.sourceCode;
|
|
138
|
+
/**
|
|
139
|
+
* Check redirect calls and assignments
|
|
140
|
+
*/
|
|
141
|
+
function checkCallExpression(node) {
|
|
142
|
+
// Check for res.redirect, window.location, etc.
|
|
143
|
+
if (node.callee.type === 'MemberExpression' &&
|
|
144
|
+
node.callee.property.type === 'Identifier') {
|
|
145
|
+
const methodName = node.callee.property.name;
|
|
146
|
+
if (['redirect', 'replace', 'assign'].includes(methodName)) {
|
|
147
|
+
// Check if redirect URL is validated
|
|
148
|
+
if (!isRedirectValidated(node, sourceCode)) {
|
|
149
|
+
context.report({
|
|
150
|
+
node,
|
|
151
|
+
messageId: 'insecureRedirect',
|
|
152
|
+
suggest: [
|
|
153
|
+
{
|
|
154
|
+
messageId: 'whitelistDomains',
|
|
155
|
+
fix: () => null,
|
|
156
|
+
},
|
|
157
|
+
{
|
|
158
|
+
messageId: 'validateRedirect',
|
|
159
|
+
fix: () => null,
|
|
160
|
+
},
|
|
161
|
+
{
|
|
162
|
+
messageId: 'useRelativeUrl',
|
|
163
|
+
fix: () => null,
|
|
164
|
+
},
|
|
165
|
+
],
|
|
166
|
+
});
|
|
167
|
+
}
|
|
168
|
+
}
|
|
169
|
+
}
|
|
170
|
+
}
|
|
171
|
+
/**
|
|
172
|
+
* Check assignment expressions like window.location.href = ...
|
|
173
|
+
*/
|
|
174
|
+
function checkAssignmentExpression(node) {
|
|
175
|
+
// Check for window.location.href assignments
|
|
176
|
+
if (node.left.type === 'MemberExpression' &&
|
|
177
|
+
node.left.object.type === 'MemberExpression' &&
|
|
178
|
+
node.left.object.object.type === 'Identifier' &&
|
|
179
|
+
node.left.object.object.name === 'window' &&
|
|
180
|
+
node.left.object.property.type === 'Identifier' &&
|
|
181
|
+
node.left.object.property.name === 'location' &&
|
|
182
|
+
node.left.property.type === 'Identifier' &&
|
|
183
|
+
['href', 'replace', 'assign'].includes(node.left.property.name)) {
|
|
184
|
+
// Check if assignment value comes from user input
|
|
185
|
+
const rightText = sourceCode.getText(node.right);
|
|
186
|
+
const userInputPattern = /\b(req\.(query|body|params)|window\.location|document\.location)\b/;
|
|
187
|
+
if (userInputPattern.test(rightText)) {
|
|
188
|
+
context.report({
|
|
189
|
+
node,
|
|
190
|
+
messageId: 'insecureRedirect',
|
|
191
|
+
suggest: [
|
|
192
|
+
{
|
|
193
|
+
messageId: 'whitelistDomains',
|
|
194
|
+
fix: () => null,
|
|
195
|
+
},
|
|
196
|
+
{
|
|
197
|
+
messageId: 'validateRedirect',
|
|
198
|
+
fix: () => null,
|
|
199
|
+
},
|
|
200
|
+
{
|
|
201
|
+
messageId: 'useRelativeUrl',
|
|
202
|
+
fix: () => null,
|
|
203
|
+
},
|
|
204
|
+
],
|
|
205
|
+
});
|
|
206
|
+
}
|
|
207
|
+
}
|
|
208
|
+
}
|
|
209
|
+
return {
|
|
210
|
+
CallExpression: checkCallExpression,
|
|
211
|
+
AssignmentExpression: checkAssignmentExpression,
|
|
212
|
+
};
|
|
213
|
+
},
|
|
214
|
+
});
|
|
215
|
+
//# sourceMappingURL=no-insecure-redirects.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-insecure-redirects.js","sourceRoot":"","sources":["../../../../../../packages/eslint-plugin-secure-coding/src/rules/security/no-insecure-redirects.ts"],"names":[],"mappings":";;;AASA,4DAA0E;AAC1E,4DAAsD;AAkBtD;;GAEG;AACH,SAAS,mBAAmB,CAC1B,IAA6B,EAC7B,UAA+B;IAE/B,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IAE1C,oEAAoE;IACpE,MAAM,gBAAgB,GAAG,oEAAoE,CAAC;IAE9F,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;QACrC,mCAAmC;QACnC,OAAO,IAAI,CAAC;IACd,CAAC;IAED,uDAAuD;IACvD,oFAAoF;IAEpF,MAAM,OAAO,GAAG,UAAU,CAAC,GAAG,CAAC;IAC/B,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,EAAE,KAAK,CAAC;IAClC,MAAM,OAAO,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC;IAE9B,IAAI,CAAC,SAAS,IAAI,CAAC,OAAO,IAAI,CAAC,OAAO,EAAE,CAAC;QACvC,OAAO,KAAK,CAAC;IACf,CAAC;IAED,2DAA2D;IAC3D,MAAM,kBAAkB,GAAG;QACzB,4DAA4D;QAC5D,iDAAiD;QACjD,mCAAmC;QACnC,iCAAiC;QACjC,0BAA0B;QAC1B,wBAAwB;QACxB,uBAAuB;QACvB,mBAAmB;KACpB,CAAC;IAEF,iDAAiD;IACjD,IAAI,OAAO,GAAyB,IAAI,CAAC;IACzC,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,MAAM,QAAQ,GAAG,EAAE,CAAC;IAEpB,OAAO,OAAO,IAAI,KAAK,GAAG,QAAQ,EAAE,CAAC;QACnC,kCAAkC;QAClC,MAAM,MAAM,GAA+B,OAAsD,CAAC,MAAM,CAAC;QACzG,IAAI,CAAC,MAAM;YAAE,MAAM;QAEnB,IAAI,MAAM,CAAC,IAAI,KAAK,gBAAgB,EAAE,CAAC;YACrC,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC;YACzB,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAAC,OAA6B,CAAC,CAAC;YAEjE,8CAA8C;YAC9C,KAAK,IAAI,CAAC,GAAG,YAAY,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,YAAY,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;gBACpE,MAAM,IAAI,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;gBACrB,MAAM,QAAQ,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;gBAE1C,IAAI,kBAAkB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,EAAE,CAAC;oBAC/D,OAAO,IAAI,CAAC,CAAC,mBAAmB;gBAClC,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,GAAG,MAAM,CAAC;QACjB,KAAK,EAAE,CAAC;IACV,CAAC;IAED,sBAAsB;IACtB,OAAO,KAAK,CAAC;AACf,CAAC;AAEY,QAAA,mBAAmB,GAAG,IAAA,0BAAU,EAA0B;IACrE,IAAI,EAAE,uBAAuB;IAC7B,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,uCAAuC;SACrD;QACD,cAAc,EAAE,IAAI;QACpB,QAAQ,EAAE;YACR,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,QAAQ;gBAC3B,SAAS,EAAE,eAAe;gBAC1B,GAAG,EAAE,SAAS;gBACd,WAAW,EAAE,qDAAqD;gBAClE,QAAQ,EAAE,MAAM;gBAChB,GAAG,EAAE,uDAAuD;gBAC5D,iBAAiB,EAAE,oFAAoF;aACxG,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,mBAAmB;gBAC9B,WAAW,EAAE,oCAAoC;gBACjD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,0DAA0D;gBAC/D,iBAAiB,EAAE,oFAAoF;aACxG,CAAC;YACF,gBAAgB,EAAE,IAAA,gCAAgB,EAAC;gBACjC,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,mBAAmB;gBAC9B,WAAW,EAAE,kCAAkC;gBAC/C,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,gDAAgD;gBACrD,iBAAiB,EAAE,oFAAoF;aACxG,CAAC;YACF,cAAc,EAAE,IAAA,gCAAgB,EAAC;gBAC/B,IAAI,EAAE,4BAAY,CAAC,IAAI;gBACvB,SAAS,EAAE,kBAAkB;gBAC7B,WAAW,EAAE,0CAA0C;gBACvD,QAAQ,EAAE,KAAK;gBACf,GAAG,EAAE,qDAAqD;gBAC1D,iBAAiB,EAAE,oFAAoF;aACxG,CAAC;SACH;QACD,MAAM,EAAE;YACN;gBACE,IAAI,EAAE,QAAQ;gBACd,UAAU,EAAE;oBACV,aAAa,EAAE;wBACb,IAAI,EAAE,SAAS;wBACf,OAAO,EAAE,IAAI;qBACd;oBACD,cAAc,EAAE;wBACd,IAAI,EAAE,OAAO;wBACb,KAAK,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE;wBACzB,OAAO,EAAE,EAAE;qBACZ;iBACF;gBACD,oBAAoB,EAAE,KAAK;aAC5B;SACF;KACF;IACD,cAAc,EAAE;QACd;YACE,aAAa,EAAE,IAAI;YACnB,cAAc,EAAE,EAAE;SACnB;KACF;IACD,MAAM,CAAC,OAAsD,EAAE,CAAC,OAAO,GAAG,EAAE,CAAC;QAC3E,MAAM,EACV,aAAa,GAAG,IAAI,EACnB,GAAY,OAAO,IAAI,EAAE,CAAC;QAEvB,MAAM,QAAQ,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;QACvC,MAAM,UAAU,GAAG,aAAa,IAAI,iCAAiC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAErF,IAAI,UAAU,EAAE,CAAC;YACf,OAAO,EAAE,CAAC;QACZ,CAAC;QAED,MAAM,UAAU,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;QAE5D;;WAEG;QACH,SAAS,mBAAmB,CAAC,IAA6B;YACxD,gDAAgD;YAChD,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB;gBACvC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gBAC/C,MAAM,UAAU,GAAG,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAE7C,IAAI,CAAC,UAAU,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;oBAC3D,qCAAqC;oBACrC,IAAI,CAAC,mBAAmB,CAAC,IAAI,EAAE,UAAU,CAAC,EAAE,CAAC;wBAC3C,OAAO,CAAC,MAAM,CAAC;4BACb,IAAI;4BACJ,SAAS,EAAE,kBAAkB;4BAC7B,OAAO,EAAE;gCACP;oCACE,SAAS,EAAE,kBAAkB;oCAC7B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI;iCAChB;gCACD;oCACE,SAAS,EAAE,kBAAkB;oCAC7B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI;iCAChB;gCACD;oCACE,SAAS,EAAE,gBAAgB;oCAC3B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI;iCAChB;6BACF;yBACF,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;QACH,CAAC;QAED;;WAEG;QACH,SAAS,yBAAyB,CAAC,IAAmC;YACpE,6CAA6C;YAC7C,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,kBAAkB;gBACrC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,kBAAkB;gBAC5C,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,YAAY;gBAC7C,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,QAAQ;gBACzC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY;gBAC/C,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,UAAU;gBAC7C,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,KAAK,YAAY;gBACxC,CAAC,MAAM,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;gBAEpE,kDAAkD;gBAClD,MAAM,SAAS,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;gBACjD,MAAM,gBAAgB,GAAG,oEAAoE,CAAC;gBAE9F,IAAI,gBAAgB,CAAC,IAAI,CAAC,SAAS,CAAC,EAAE,CAAC;oBACrC,OAAO,CAAC,MAAM,CAAC;wBACb,IAAI;wBACJ,SAAS,EAAE,kBAAkB;wBAC7B,OAAO,EAAE;4BACP;gCACE,SAAS,EAAE,kBAAkB;gCAC7B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI;6BAChB;4BACD;gCACE,SAAS,EAAE,kBAAkB;gCAC7B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI;6BAChB;4BACD;gCACE,SAAS,EAAE,gBAAgB;gCAC3B,GAAG,EAAE,GAAG,EAAE,CAAC,IAAI;6BAChB;yBACF;qBACF,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO;YACL,cAAc,EAAE,mBAAmB;YACnC,oBAAoB,EAAE,yBAAyB;SAChD,CAAC;IACJ,CAAC;CACF,CAAC,CAAC"}
|
|
@@ -0,0 +1,14 @@
|
|
|
1
|
+
import { type SecurityRuleOptions } from '@interlace/eslint-devkit';
|
|
2
|
+
export interface Options extends SecurityRuleOptions {
|
|
3
|
+
/** Allowed origins for postMessage communication */
|
|
4
|
+
allowedOrigins?: string[];
|
|
5
|
+
/** Whether to allow wildcard origins in development */
|
|
6
|
+
allowWildcardInDev?: boolean;
|
|
7
|
+
/** Message event handler patterns to check */
|
|
8
|
+
messageHandlerPatterns?: string[];
|
|
9
|
+
/** Origins considered safe (e.g., localhost, development domains) */
|
|
10
|
+
safeOrigins?: string[];
|
|
11
|
+
/** Whether to perform deep origin validation analysis */
|
|
12
|
+
deepValidation?: boolean;
|
|
13
|
+
}
|
|
14
|
+
export declare const noInsufficientPostmessageValidation: ESLintUtils.RuleModule<MessageIds, Options, unknown, ESLintUtils.RuleListener>;
|
|
@@ -0,0 +1,390 @@
|
|
|
1
|
+
"use strict";
|
|
2
|
+
Object.defineProperty(exports, "__esModule", { value: true });
|
|
3
|
+
exports.noInsufficientPostmessageValidation = void 0;
|
|
4
|
+
const eslint_devkit_1 = require("@interlace/eslint-devkit");
|
|
5
|
+
const eslint_devkit_2 = require("@interlace/eslint-devkit");
|
|
6
|
+
const eslint_devkit_3 = require("@interlace/eslint-devkit");
|
|
7
|
+
exports.noInsufficientPostmessageValidation = (0, eslint_devkit_1.createRule)({
|
|
8
|
+
name: 'no-insufficient-postmessage-validation',
|
|
9
|
+
meta: {
|
|
10
|
+
type: 'problem',
|
|
11
|
+
docs: {
|
|
12
|
+
description: 'Detects insufficient postMessage origin validation',
|
|
13
|
+
},
|
|
14
|
+
fixable: 'code',
|
|
15
|
+
hasSuggestions: true,
|
|
16
|
+
messages: {
|
|
17
|
+
insufficientPostmessageValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
18
|
+
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
19
|
+
issueName: 'Insufficient postMessage Validation',
|
|
20
|
+
cwe: 'CWE-20',
|
|
21
|
+
description: 'postMessage origin validation is insufficient',
|
|
22
|
+
severity: '{{severity}}',
|
|
23
|
+
fix: '{{safeAlternative}}',
|
|
24
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
25
|
+
}),
|
|
26
|
+
wildcardOrigin: (0, eslint_devkit_2.formatLLMMessage)({
|
|
27
|
+
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
28
|
+
issueName: 'Wildcard postMessage Origin',
|
|
29
|
+
cwe: 'CWE-20',
|
|
30
|
+
description: 'postMessage uses wildcard origin (*)',
|
|
31
|
+
severity: 'CRITICAL',
|
|
32
|
+
fix: 'Use specific trusted origins instead of "*"',
|
|
33
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
34
|
+
}),
|
|
35
|
+
missingOriginCheck: (0, eslint_devkit_2.formatLLMMessage)({
|
|
36
|
+
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
37
|
+
issueName: 'Missing Origin Check',
|
|
38
|
+
cwe: 'CWE-20',
|
|
39
|
+
description: 'Message handler missing origin validation',
|
|
40
|
+
severity: 'HIGH',
|
|
41
|
+
fix: 'Check event.origin against trusted origins',
|
|
42
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
43
|
+
}),
|
|
44
|
+
unsafeMessageHandler: (0, eslint_devkit_2.formatLLMMessage)({
|
|
45
|
+
icon: eslint_devkit_2.MessageIcons.SECURITY,
|
|
46
|
+
issueName: 'Unsafe Message Handler',
|
|
47
|
+
cwe: 'CWE-20',
|
|
48
|
+
description: 'Message event handler processes data without validation',
|
|
49
|
+
severity: 'MEDIUM',
|
|
50
|
+
fix: 'Validate event.origin and event.data before processing',
|
|
51
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
52
|
+
}),
|
|
53
|
+
useSpecificOrigins: (0, eslint_devkit_2.formatLLMMessage)({
|
|
54
|
+
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
55
|
+
issueName: 'Use Specific Origins',
|
|
56
|
+
description: 'Specify trusted origins explicitly',
|
|
57
|
+
severity: 'LOW',
|
|
58
|
+
fix: 'postMessage(data, "https://trusted-domain.com")',
|
|
59
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
60
|
+
}),
|
|
61
|
+
validateMessageOrigin: (0, eslint_devkit_2.formatLLMMessage)({
|
|
62
|
+
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
63
|
+
issueName: 'Validate Message Origin',
|
|
64
|
+
description: 'Validate message origin before processing',
|
|
65
|
+
severity: 'LOW',
|
|
66
|
+
fix: 'if (event.origin === "https://trusted.com") { /* process */ }',
|
|
67
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
68
|
+
}),
|
|
69
|
+
implementOriginWhitelist: (0, eslint_devkit_2.formatLLMMessage)({
|
|
70
|
+
icon: eslint_devkit_2.MessageIcons.INFO,
|
|
71
|
+
issueName: 'Implement Origin Whitelist',
|
|
72
|
+
description: 'Use whitelist of allowed origins',
|
|
73
|
+
severity: 'LOW',
|
|
74
|
+
fix: 'const allowedOrigins = ["https://app1.com", "https://app2.com"];',
|
|
75
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
76
|
+
}),
|
|
77
|
+
strategyOriginValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
78
|
+
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
79
|
+
issueName: 'Origin Validation Strategy',
|
|
80
|
+
description: 'Validate message origins against trusted list',
|
|
81
|
+
severity: 'LOW',
|
|
82
|
+
fix: 'Check event.origin against predefined allowed origins',
|
|
83
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
84
|
+
}),
|
|
85
|
+
strategyContentValidation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
86
|
+
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
87
|
+
issueName: 'Content Validation Strategy',
|
|
88
|
+
description: 'Validate message content in addition to origin',
|
|
89
|
+
severity: 'LOW',
|
|
90
|
+
fix: 'Validate both event.origin and event.data structure',
|
|
91
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
92
|
+
}),
|
|
93
|
+
strategyEnvironmentSeparation: (0, eslint_devkit_2.formatLLMMessage)({
|
|
94
|
+
icon: eslint_devkit_2.MessageIcons.STRATEGY,
|
|
95
|
+
issueName: 'Environment Separation Strategy',
|
|
96
|
+
description: 'Use different origins for different environments',
|
|
97
|
+
severity: 'LOW',
|
|
98
|
+
fix: 'Use environment-specific origin validation',
|
|
99
|
+
documentationLink: 'https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage',
|
|
100
|
+
})
|
|
101
|
+
},
|
|
102
|
+
schema: [
|
|
103
|
+
{
|
|
104
|
+
type: 'object',
|
|
105
|
+
properties: {
|
|
106
|
+
allowedOrigins: {
|
|
107
|
+
type: 'array',
|
|
108
|
+
items: { type: 'string' },
|
|
109
|
+
default: [],
|
|
110
|
+
},
|
|
111
|
+
allowWildcardInDev: {
|
|
112
|
+
type: 'boolean',
|
|
113
|
+
default: false,
|
|
114
|
+
description: 'Allow wildcard origins in development environment'
|
|
115
|
+
},
|
|
116
|
+
messageHandlerPatterns: {
|
|
117
|
+
type: 'array',
|
|
118
|
+
items: { type: 'string' },
|
|
119
|
+
default: ['addEventListener', 'onmessage', 'postMessage'],
|
|
120
|
+
},
|
|
121
|
+
trustedSanitizers: {
|
|
122
|
+
type: 'array',
|
|
123
|
+
items: { type: 'string' },
|
|
124
|
+
default: [],
|
|
125
|
+
description: 'Additional function names to consider as origin validators',
|
|
126
|
+
},
|
|
127
|
+
trustedAnnotations: {
|
|
128
|
+
type: 'array',
|
|
129
|
+
items: { type: 'string' },
|
|
130
|
+
default: [],
|
|
131
|
+
description: 'Additional JSDoc annotations to consider as safe markers',
|
|
132
|
+
},
|
|
133
|
+
strictMode: {
|
|
134
|
+
type: 'boolean',
|
|
135
|
+
default: false,
|
|
136
|
+
description: 'Disable all false positive detection (strict mode)',
|
|
137
|
+
},
|
|
138
|
+
safeOrigins: {
|
|
139
|
+
type: 'array',
|
|
140
|
+
items: { type: 'string' },
|
|
141
|
+
default: ['localhost', '127.0.0.1', '0.0.0.0'],
|
|
142
|
+
description: 'Origins considered safe for postMessage communication',
|
|
143
|
+
},
|
|
144
|
+
deepValidation: {
|
|
145
|
+
type: 'boolean',
|
|
146
|
+
default: false,
|
|
147
|
+
description: 'Enable deep AST analysis for origin validation detection',
|
|
148
|
+
},
|
|
149
|
+
},
|
|
150
|
+
additionalProperties: false,
|
|
151
|
+
},
|
|
152
|
+
],
|
|
153
|
+
},
|
|
154
|
+
defaultOptions: [
|
|
155
|
+
{
|
|
156
|
+
allowedOrigins: [],
|
|
157
|
+
allowWildcardInDev: false,
|
|
158
|
+
messageHandlerPatterns: ['addEventListener', 'onmessage', 'postMessage'],
|
|
159
|
+
trustedSanitizers: [],
|
|
160
|
+
trustedAnnotations: [],
|
|
161
|
+
strictMode: false,
|
|
162
|
+
safeOrigins: ['localhost', '127.0.0.1', '0.0.0.0'],
|
|
163
|
+
deepValidation: false,
|
|
164
|
+
},
|
|
165
|
+
],
|
|
166
|
+
create(context) {
|
|
167
|
+
const options = context.options[0] || {};
|
|
168
|
+
const { allowedOrigins = [], allowWildcardInDev = false, trustedSanitizers = [], trustedAnnotations = [], strictMode = false, deepValidation = false, } = options;
|
|
169
|
+
const sourceCode = context.sourceCode || context.sourceCode;
|
|
170
|
+
const filename = context.filename || context.getFilename();
|
|
171
|
+
// Create safety checker for false positive detection
|
|
172
|
+
const safetyChecker = (0, eslint_devkit_3.createSafetyChecker)({
|
|
173
|
+
trustedSanitizers,
|
|
174
|
+
trustedAnnotations,
|
|
175
|
+
trustedOrmPatterns: [],
|
|
176
|
+
strictMode,
|
|
177
|
+
});
|
|
178
|
+
/**
|
|
179
|
+
* Check if origin validation is present in a message handler
|
|
180
|
+
*
|
|
181
|
+
* ⚠️ STATIC ANALYSIS LIMITATION:
|
|
182
|
+
* This can only detect simple patterns. Complex validation logic
|
|
183
|
+
* (external functions, runtime config, dynamic validation) cannot
|
|
184
|
+
* be detected statically.
|
|
185
|
+
*/
|
|
186
|
+
const hasOriginValidation = (functionNode) => {
|
|
187
|
+
if (deepValidation) {
|
|
188
|
+
// Deep AST analysis (more thorough but slower)
|
|
189
|
+
try {
|
|
190
|
+
let hasOriginCheck = false;
|
|
191
|
+
const checkNode = (node) => {
|
|
192
|
+
// Check for event.origin comparisons
|
|
193
|
+
if (node.type === 'BinaryExpression' &&
|
|
194
|
+
((node.left.type === 'MemberExpression' &&
|
|
195
|
+
node.left.property.type === 'Identifier' &&
|
|
196
|
+
node.left.property.name === 'origin' &&
|
|
197
|
+
node.left.object.type === 'Identifier') ||
|
|
198
|
+
(node.right.type === 'MemberExpression' &&
|
|
199
|
+
node.right.property.type === 'Identifier' &&
|
|
200
|
+
node.right.property.name === 'origin' &&
|
|
201
|
+
node.right.object.type === 'Identifier'))) {
|
|
202
|
+
hasOriginCheck = true;
|
|
203
|
+
return;
|
|
204
|
+
}
|
|
205
|
+
// Check for includes() calls on origin arrays
|
|
206
|
+
if (node.type === 'CallExpression' &&
|
|
207
|
+
node.callee.type === 'MemberExpression' &&
|
|
208
|
+
node.callee.property.type === 'Identifier' &&
|
|
209
|
+
node.callee.property.name === 'includes') {
|
|
210
|
+
const objectText = sourceCode.getText(node.callee.object).toLowerCase();
|
|
211
|
+
if (objectText.includes('origin') || objectText.includes('allowed')) {
|
|
212
|
+
hasOriginCheck = true;
|
|
213
|
+
return;
|
|
214
|
+
}
|
|
215
|
+
}
|
|
216
|
+
// Recursively check child nodes
|
|
217
|
+
for (const key in node) {
|
|
218
|
+
if (key === 'parent' || key === 'range' || key === 'loc')
|
|
219
|
+
continue;
|
|
220
|
+
const child = node[key];
|
|
221
|
+
if (child && typeof child === 'object' && 'type' in child) {
|
|
222
|
+
checkNode(child);
|
|
223
|
+
}
|
|
224
|
+
else if (Array.isArray(child)) {
|
|
225
|
+
child.forEach(item => {
|
|
226
|
+
if (item && typeof item === 'object' && 'type' in item) {
|
|
227
|
+
checkNode(item);
|
|
228
|
+
}
|
|
229
|
+
});
|
|
230
|
+
}
|
|
231
|
+
}
|
|
232
|
+
};
|
|
233
|
+
checkNode(functionNode);
|
|
234
|
+
return hasOriginCheck;
|
|
235
|
+
}
|
|
236
|
+
catch {
|
|
237
|
+
// If AST traversal fails, fall back to simple text matching
|
|
238
|
+
return false;
|
|
239
|
+
}
|
|
240
|
+
}
|
|
241
|
+
else {
|
|
242
|
+
// Simple text-based check (faster, less accurate)
|
|
243
|
+
const functionText = sourceCode.getText(functionNode).toLowerCase();
|
|
244
|
+
return functionText.includes('event.origin') ||
|
|
245
|
+
(functionText.includes('origin') && functionText.includes('event'));
|
|
246
|
+
}
|
|
247
|
+
};
|
|
248
|
+
return {
|
|
249
|
+
// Check postMessage calls with wildcard origins
|
|
250
|
+
CallExpression(node) {
|
|
251
|
+
const callee = node.callee;
|
|
252
|
+
// Check for any postMessage() calls
|
|
253
|
+
if (callee.type === 'MemberExpression' &&
|
|
254
|
+
callee.property.type === 'Identifier' &&
|
|
255
|
+
callee.property.name === 'postMessage') {
|
|
256
|
+
// Check allowed origins for non-wildcard origins
|
|
257
|
+
const args = node.arguments;
|
|
258
|
+
if (args.length >= 2) {
|
|
259
|
+
const targetOrigin = args[1];
|
|
260
|
+
if (targetOrigin.type === 'Literal' &&
|
|
261
|
+
typeof targetOrigin.value === 'string' &&
|
|
262
|
+
targetOrigin.value !== '*') {
|
|
263
|
+
// Check if origin is explicitly allowed
|
|
264
|
+
const originValue = targetOrigin.value;
|
|
265
|
+
const isAllowed = allowedOrigins.includes(originValue) ||
|
|
266
|
+
allowedOrigins.includes('*') ||
|
|
267
|
+
allowedOrigins.length === 0; // If no restrictions, allow all specific origins
|
|
268
|
+
if (!isAllowed) {
|
|
269
|
+
// FALSE POSITIVE REDUCTION
|
|
270
|
+
if (safetyChecker.isSafe(node, context)) {
|
|
271
|
+
return;
|
|
272
|
+
}
|
|
273
|
+
context.report({
|
|
274
|
+
node: targetOrigin,
|
|
275
|
+
messageId: 'useSpecificOrigins',
|
|
276
|
+
data: {
|
|
277
|
+
filePath: filename,
|
|
278
|
+
line: String(node.loc?.start.line ?? 0),
|
|
279
|
+
origin: originValue,
|
|
280
|
+
},
|
|
281
|
+
});
|
|
282
|
+
}
|
|
283
|
+
}
|
|
284
|
+
// Check for wildcard origin
|
|
285
|
+
if (targetOrigin.type === 'Literal' &&
|
|
286
|
+
typeof targetOrigin.value === 'string' &&
|
|
287
|
+
targetOrigin.value === '*') {
|
|
288
|
+
// Check if origin is explicitly allowed
|
|
289
|
+
const originValue = targetOrigin.value;
|
|
290
|
+
const isAllowed = allowedOrigins.includes(originValue) || allowedOrigins.includes('*');
|
|
291
|
+
// Allow if explicitly allowed OR in development if configured
|
|
292
|
+
if (!isAllowed && !allowWildcardInDev) {
|
|
293
|
+
// TEMP: bypass safety checker
|
|
294
|
+
// if (safetyChecker.isSafe(node, context)) {
|
|
295
|
+
// return;
|
|
296
|
+
// }
|
|
297
|
+
context.report({
|
|
298
|
+
node: targetOrigin,
|
|
299
|
+
messageId: 'wildcardOrigin',
|
|
300
|
+
data: {
|
|
301
|
+
filePath: filename,
|
|
302
|
+
line: String(node.loc?.start.line ?? 0),
|
|
303
|
+
},
|
|
304
|
+
});
|
|
305
|
+
}
|
|
306
|
+
}
|
|
307
|
+
}
|
|
308
|
+
}
|
|
309
|
+
// Check for addEventListener('message', ...) and IPC calls
|
|
310
|
+
// Check for addEventListener('message', ...)
|
|
311
|
+
if (callee.type === 'MemberExpression' &&
|
|
312
|
+
callee.property.type === 'Identifier' &&
|
|
313
|
+
callee.property.name === 'addEventListener') {
|
|
314
|
+
const args = node.arguments;
|
|
315
|
+
if (args.length >= 2) {
|
|
316
|
+
const eventType = args[0];
|
|
317
|
+
const handler = args[1];
|
|
318
|
+
// Check if it's a message event
|
|
319
|
+
if (eventType.type === 'Literal' &&
|
|
320
|
+
typeof eventType.value === 'string' &&
|
|
321
|
+
eventType.value === 'message') {
|
|
322
|
+
// Check if handler is a function and lacks origin validation
|
|
323
|
+
if (handler.type === 'FunctionExpression' ||
|
|
324
|
+
handler.type === 'ArrowFunctionExpression') {
|
|
325
|
+
if (!hasOriginValidation(handler)) {
|
|
326
|
+
// FALSE POSITIVE REDUCTION
|
|
327
|
+
if (safetyChecker.isSafe(node, context)) {
|
|
328
|
+
return;
|
|
329
|
+
}
|
|
330
|
+
context.report({
|
|
331
|
+
node: handler,
|
|
332
|
+
messageId: 'missingOriginCheck',
|
|
333
|
+
data: {
|
|
334
|
+
filePath: filename,
|
|
335
|
+
line: String(node.loc?.start.line ?? 0),
|
|
336
|
+
},
|
|
337
|
+
});
|
|
338
|
+
}
|
|
339
|
+
}
|
|
340
|
+
}
|
|
341
|
+
}
|
|
342
|
+
}
|
|
343
|
+
},
|
|
344
|
+
// Check for window.onmessage assignments
|
|
345
|
+
AssignmentExpression(node) {
|
|
346
|
+
const left = node.left;
|
|
347
|
+
if (left.type === 'MemberExpression' &&
|
|
348
|
+
left.property.type === 'Identifier' &&
|
|
349
|
+
left.property.name === 'onmessage' &&
|
|
350
|
+
left.object.type === 'Identifier' &&
|
|
351
|
+
left.object.name === 'window') {
|
|
352
|
+
const handler = node.right;
|
|
353
|
+
if (handler && (handler.type === 'FunctionExpression' ||
|
|
354
|
+
handler.type === 'ArrowFunctionExpression')) {
|
|
355
|
+
if (!hasOriginValidation(handler)) {
|
|
356
|
+
// FALSE POSITIVE REDUCTION
|
|
357
|
+
if (safetyChecker.isSafe(node, context)) {
|
|
358
|
+
return;
|
|
359
|
+
}
|
|
360
|
+
context.report({
|
|
361
|
+
node: handler,
|
|
362
|
+
messageId: 'missingOriginCheck',
|
|
363
|
+
data: {
|
|
364
|
+
filePath: filename,
|
|
365
|
+
line: String(node.loc?.start.line ?? 0),
|
|
366
|
+
},
|
|
367
|
+
});
|
|
368
|
+
}
|
|
369
|
+
}
|
|
370
|
+
}
|
|
371
|
+
},
|
|
372
|
+
// Check for event.data usage without origin validation
|
|
373
|
+
MemberExpression(node) {
|
|
374
|
+
// Look for event.data usage
|
|
375
|
+
if (node.property.type === 'Identifier' &&
|
|
376
|
+
node.property.name === 'data' &&
|
|
377
|
+
node.object.type === 'Identifier' &&
|
|
378
|
+
node.object.name === 'event') {
|
|
379
|
+
// For now, simplify: only flag if we're clearly in an unsafe context
|
|
380
|
+
// The addEventListener check above should handle most cases
|
|
381
|
+
// This is a simplified version to avoid complex AST traversal issues
|
|
382
|
+
// Only flag if this is clearly in a message handler without validation
|
|
383
|
+
// We'll rely on the addEventListener check for the main logic
|
|
384
|
+
return;
|
|
385
|
+
}
|
|
386
|
+
},
|
|
387
|
+
};
|
|
388
|
+
},
|
|
389
|
+
});
|
|
390
|
+
//# sourceMappingURL=no-insufficient-postmessage-validation.js.map
|