feat:introduce shouldWriteAccessLog

core/src/main/java/com/linecorp/armeria/server/AbstractHttpResponseHandler.java

@@ -222,12 +222,12 @@ final void endLogRequestAndResponse(@Nullable Throwable cause) {
     }
 
     /**
-     * Writes an access log if the {@link TransientServiceOption#WITH_ACCESS_LOGGING} option is enabled for
-     * the {@link #service()}.
+     * Writes an access log if the {@link ServiceConfig#shouldWriteAccessLog} is true
+     * for the {@link #service()}.
      */
     final void maybeWriteAccessLog() {
         final ServiceConfig config = reqCtx.config();
-        if (config.transientServiceOptions().contains(TransientServiceOption.WITH_ACCESS_LOGGING)) {
+        if (config.shouldWriteAccessLog()) {
             reqCtx.log().whenComplete().thenAccept(log -> {
                 try (SafeCloseable ignored = reqCtx.push()) {
                     config.accessLogWriter().log(log);

core/src/main/java/com/linecorp/armeria/server/ServiceConfig.java

@@ -474,6 +474,16 @@ List<ShutdownSupport> shutdownSupports() {
         return shutdownSupports;
     }
 
+    /**
+     * Returns whether the access log should be written for this {@link #service()}.
+     * Access log is written only when the {@link TransientServiceOption#WITH_ACCESS_LOGGING} is set and the
+     * access logger is not the the {@link AccessLogWriter#DISABLED}.
+     */
+    public boolean shouldWriteAccessLog() {
+        return !accessLogWriter.isDisabled() &&
+                transientServiceOptions.contains(TransientServiceOption.WITH_ACCESS_LOGGING);
+    }
+
     /**
      * Returns the {@link EventLoopGroup} dedicated to the execution of services' methods.
      */

core/src/main/java/com/linecorp/armeria/server/logging/AccessLogWriter.java

@@ -32,6 +32,11 @@
 @FunctionalInterface
 public interface AccessLogWriter {
 
+    /**
+     * A disabled access log writer that performs no operation.
+     */
+    AccessLogWriter DISABLED = requestLog -> { /* No operation. */ };
+
     /**
      * Returns an access log writer with a common format.
      */
@@ -50,7 +55,15 @@ static AccessLogWriter combined() {
      * Returns disabled access log writer.
      */
     static AccessLogWriter disabled() {
-        return requestLog -> { /* No operation. */ };
+        return DISABLED;
+    }
+
+    /**
+     * Determines if this writer is the disabled writer.
+     *
+     */
+    default boolean isDisabled() {
+        return this == DISABLED;
     }
 
     /**