From eb43166956bcd3e3189c3307fdd87c959325dfd4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Connor=20B=C3=A4r?= <connor-baer@users.noreply.github.com>
Date: Wed, 24 Jan 2024 09:40:29 +0000
Subject: [PATCH] Disable security/detect-object-injection rule (#928)

---
 .../eslint/__snapshots__/config.spec.ts.snap  | 28 +++++++++++++++++++
 src/configs/eslint/config.ts                  |  2 ++
 2 files changed, 30 insertions(+)

diff --git a/src/configs/eslint/__snapshots__/config.spec.ts.snap b/src/configs/eslint/__snapshots__/config.spec.ts.snap
index ef6a4f54..70ae6f95 100644
--- a/src/configs/eslint/__snapshots__/config.spec.ts.snap
+++ b/src/configs/eslint/__snapshots__/config.spec.ts.snap
@@ -2908,6 +2908,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -3124,6 +3125,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
         ],
       },
     ],
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -3346,6 +3348,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -3549,6 +3552,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -3761,6 +3765,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -3964,6 +3969,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -4183,6 +4189,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -4410,6 +4417,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -4589,6 +4597,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
       },
     ],
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -4774,6 +4783,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -4940,6 +4950,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -5115,6 +5126,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -5281,6 +5293,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -5463,6 +5476,7 @@ exports[`eslint > with options > should return a config for { language: 'JavaScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -10355,6 +10369,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -10702,6 +10717,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
         ],
       },
     ],
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -11055,6 +11071,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -11389,6 +11406,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -11732,6 +11750,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -12066,6 +12085,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -12416,6 +12436,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "object-curly-newline": "off",
     "operator-linebreak": "off",
     "quote-props": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -12774,6 +12795,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -13084,6 +13106,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
       },
     ],
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -13400,6 +13423,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -13697,6 +13721,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -14003,6 +14028,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -14300,6 +14326,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
@@ -14613,6 +14640,7 @@ exports[`eslint > with options > should return a config for { language: 'TypeScr
     "quote-props": "off",
     "react/display-name": "off",
     "react/react-in-jsx-scope": "off",
+    "security/detect-object-injection": "off",
   },
   "settings": {
     "import/resolver": {
diff --git a/src/configs/eslint/config.ts b/src/configs/eslint/config.ts
index b9c5cedf..74b0a0fa 100644
--- a/src/configs/eslint/config.ts
+++ b/src/configs/eslint/config.ts
@@ -265,6 +265,8 @@ function customizeEnvironments(environments?: Environment[]) {
         'node/no-missing-import': 'off',
         // This rule is already covered by similar ESLint rules.
         'node/no-extraneous-import': 'off',
+        // This rule produces too many false positives.
+        'security/detect-object-injection': 'off',
       },
       overrides: [
         {