Merge pull request #2007 from skjolber/emptyObfuscateJsonBodyFields

Return 'none' body filter if list of to-be-filtered JSON field-names is empty

logbook-spring-boot-autoconfigure/src/main/java/org/zalando/logbook/autoconfigure/LogbookAutoConfiguration.java

@@ -260,7 +260,9 @@ public BodyFilter bodyFilter() {
     public BodyFilter jsonBodyFieldsFilter() {
         final LogbookProperties.Obfuscate obfuscate = properties.getObfuscate();
         final List<String> jsonBodyFields = obfuscate.getJsonBodyFields();
-
+        if (jsonBodyFields.isEmpty()) {
+            return BodyFilter.none();
+        }
         return new JacksonJsonFieldBodyFilter(jsonBodyFields, obfuscate.getReplacement());
     }
 

logbook-spring-boot-autoconfigure/src/test/java/org/zalando/logbook/autoconfigure/ObfuscateBodyDefaultTest.java

@@ -0,0 +1,25 @@
+package org.zalando.logbook.autoconfigure;
+
+import org.json.JSONException;
+import org.junit.jupiter.api.Test;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.beans.factory.annotation.Qualifier;
+import org.zalando.logbook.BodyFilter;
+
+import java.io.IOException;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+@LogbookTest
+class ObfuscateBodyDefaultTest {
+
+    @Autowired
+    @Qualifier("jsonBodyFieldsFilter")
+    private BodyFilter jsonBodyFieldsFilter;
+
+    @Test
+    void shouldNotFilterJsonBodiesIfEmptyObfuscateJsonBodyFieldNames() throws IOException, JSONException {
+        assertThat(jsonBodyFieldsFilter).isSameAs(BodyFilter.none());
+    }
+
+}