Fix issues 406 errors

We set content-type null on UJS as a flag to remove it later down the stack. I'm not sure why I did it this way, but it missed a case when the method type on the form was a GET. This resulted in content-type of "null" being sent to the server. This isn't the best fix and I have to find a better way of achieving the same thing.
thoughtbot · Jun 4, 2024 · 091c076 · 091c076
1 parent 318b66d
commit 091c076
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/superglue/lib/utils/request.js b/superglue/lib/utils/request.js
@@ -60,12 +60,17 @@ export function argsForFetch(
     'x-superglue-request': true,
   }
 
+  // This needs to be done better. This is saying to
+  // remove the content-type header from UJS form
+  // submissions.
+  const fromUJSForm = headers['content-type'] === null
+
   if (method != 'GET' && method != 'HEAD') {
-    if (headers['content-type'] === null) {
-      delete headers['content-type']
-    } else {
-      headers['content-type'] = 'application/json'
-    }
+    headers['content-type'] = 'application/json'
+  }
+
+  if (fromUJSForm) {
+    delete headers['content-type']
   }
 
   if (currentState.csrfToken) {