Fix the simulate rest API when using tags (#1872)

When using tags, the simulate option does not merge the provided tags with the extracted document, resulting in a wrong simulation.
dadoonet · May 3, 2024 · 5dd56e7 · 5dd56e7
1 parent d3918bb
commit 5dd56e7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/rest/src/main/java/fr/pilato/elasticsearch/crawler/fs/rest/DocumentApi.java b/rest/src/main/java/fr/pilato/elasticsearch/crawler/fs/rest/DocumentApi.java
@@ -208,11 +208,11 @@ private UploadResponse uploadToDocumentService(
         // Elasticsearch entity coordinates (we use the first node address)
         ServerUrl node = settings.getElasticsearch().getNodes().get(0);
         String url = node.getUrl() + "/" + index + "/_doc/" + id;
+        final Doc mergedDoc = this.getMergedJsonDoc(doc, tags);
         if (Boolean.parseBoolean(simulate)) {
             logger.debug("Simulate mode is on, so we skip sending document [{}] to elasticsearch at [{}].", filename, url);
         } else {
             logger.debug("Sending document [{}] to elasticsearch.", filename);
-            final Doc mergedDoc = this.getMergedJsonDoc(doc, tags);
             documentService.index(
                     index,
                     id,
@@ -227,7 +227,7 @@ private UploadResponse uploadToDocumentService(
 
         if (logger.isDebugEnabled() || Boolean.parseBoolean(debug)) {
             // We send the content back if debug is on or if we got in the query explicitly a debug command
-            response.setDoc(doc);
+            response.setDoc(mergedDoc);
         }
 
         return response;