From f5895872d4bf81cd9a81b391400a9d55b6af3ac8 Mon Sep 17 00:00:00 2001
From: Nidhi <nidhi@appsmith.com>
Date: Wed, 18 Aug 2021 16:55:33 +0530
Subject: [PATCH] Allow extra fields in multipart file data (#6689)

* Allow extra fields in multipart file data

* Remove extra annotation

* Reuse object mapper

* Reuse object mapper
---
 .../src/main/java/com/external/helpers/DataUtils.java       | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/server/appsmith-plugins/restApiPlugin/src/main/java/com/external/helpers/DataUtils.java b/app/server/appsmith-plugins/restApiPlugin/src/main/java/com/external/helpers/DataUtils.java
index 3dbfc2b456..2fdb484e94 100644
--- a/app/server/appsmith-plugins/restApiPlugin/src/main/java/com/external/helpers/DataUtils.java
+++ b/app/server/appsmith-plugins/restApiPlugin/src/main/java/com/external/helpers/DataUtils.java
@@ -5,6 +5,7 @@ import com.appsmith.external.exceptions.pluginExceptions.AppsmithPluginException
 import com.appsmith.external.models.Property;
 import com.external.dtos.MultipartFormDataDTO;
 import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.databind.DeserializationFeature;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.google.gson.JsonSyntaxException;
 import net.minidev.json.JSONArray;
@@ -33,6 +34,7 @@ import java.util.stream.Collectors;
 public class DataUtils {
 
     private static DataUtils dataUtils;
+    private final ObjectMapper objectMapper;
 
     public enum MultipartFormDataType {
         TEXT,
@@ -40,6 +42,8 @@ public class DataUtils {
     }
 
     private DataUtils() {
+        this.objectMapper = new ObjectMapper();
+        this.objectMapper.disable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES);
     }
 
     public static DataUtils getInstance() {
@@ -146,7 +150,7 @@ public class DataUtils {
                         if (MultipartFormDataType.TEXT.equals(multipartFormDataType)) {
                             bodyBuilder.part(key, property.getValue());
                         } else if (MultipartFormDataType.FILE.equals(multipartFormDataType)) {
-                            ObjectMapper objectMapper = new ObjectMapper();
+
                             MultipartFormDataDTO multipartFormDataDTO = null;
                             try {
                                 multipartFormDataDTO = objectMapper.readValue(