address comment by @minwoox

Author: jrhee17

core/src/main/java/com/linecorp/armeria/common/logging/BeanFieldMaskerSelector.java

@@ -16,8 +16,6 @@
 
 package com.linecorp.armeria.common.logging;
 
-import static java.util.Objects.requireNonNull;
-
 import com.linecorp.armeria.common.annotation.UnstableApi;
 
 /**
@@ -44,19 +42,4 @@
 @UnstableApi
 @FunctionalInterface
 public interface BeanFieldMaskerSelector extends FieldMaskerSelector<BeanFieldInfo> {
-
-    /**
-     * Delegates {@link FieldMasker} selection to a different {@link BeanFieldMaskerSelector}
-     * if the current {@link BeanFieldMaskerSelector} returns {@link FieldMasker#fallthrough()}.
-     */
-    default BeanFieldMaskerSelector orElse(BeanFieldMaskerSelector other) {
-        requireNonNull(other, "other");
-        return beanFieldInfo -> {
-            final FieldMasker fieldMasker = fieldMasker(beanFieldInfo);
-            if (fieldMasker != FieldMasker.fallthrough()) {
-                return fieldMasker;
-            }
-            return other.fieldMasker(beanFieldInfo);
-        };
-    }
 }

core/src/main/java/com/linecorp/armeria/common/logging/FieldMaskerSelector.java

@@ -16,6 +16,8 @@
 
 package com.linecorp.armeria.common.logging;
 
+import static java.util.Objects.requireNonNull;
+
 import java.util.function.Function;
 
 import com.linecorp.armeria.common.annotation.UnstableApi;
@@ -67,5 +69,20 @@ static BeanFieldMaskerSelector ofBean(Function<BeanFieldInfo, FieldMasker> field
      * @see FieldMasker
      */
     FieldMasker fieldMasker(T info);
+
+    /**
+     * Delegates {@link FieldMasker} selection to a different {@link FieldMaskerSelector}
+     * if the current {@link FieldMaskerSelector} returns {@link FieldMasker#fallthrough()}.
+     */
+    default FieldMaskerSelector<T> orElse(FieldMaskerSelector<T> other) {
+        requireNonNull(other, "other");
+        return fieldInfo -> {
+            final FieldMasker fieldMasker = fieldMasker(fieldInfo);
+            if (fieldMasker != FieldMasker.fallthrough()) {
+                return fieldMasker;
+            }
+            return other.fieldMasker(fieldInfo);
+        };
+    }
 }