Merge pull request #655 from sasjs/node-form-data-fix

fix: not to use NodeFormData function on web

src/job-execution/WebJobExecutor.ts

@@ -151,7 +151,7 @@ export class WebJobExecutor extends BaseJobExecutor {
 
     /* The NodeFormData object does not set the request header - so, set it */
     const contentType =
-      formData instanceof NodeFormData
+      formData instanceof NodeFormData && typeof FormData === 'undefined'
         ? `multipart/form-data; boundary=${formData.getBoundary()}`
         : undefined
 

src/utils/getValidJson.ts

@@ -12,6 +12,8 @@ export const getValidJson = (str: string | object): object => {
 
     if (typeof str === 'object') return str
 
+    if (str === '') return {}
+
     return JSON.parse(str)
   } catch (e) {
     if (e instanceof JsonParseArrayError) throw e