refactor: remove custom access logger

It is fine when application is able to log request, however, normally it
produces a lot noise, which has been handled by adding special logger.
However, nowadays it requires a lot endpoints to be filtered and doesn't
provide any choice.

Instead of it lets disable access logger by default and let users decide
do they need or not to see access log messages

src/ahriman/core/log/filtered_access_logger.py

@@ -1,108 +0,0 @@
-#
-# Copyright (c) 2021-2024 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-import re
-
-from aiohttp.web import AccessLogger, BaseRequest, StreamResponse
-
-
-class FilteredAccessLogger(AccessLogger):
-    """
-    access logger implementation with log filter enabled
-
-    Attributes:
-        DISTRIBUTED_PATH_REGEX(str): (class attribute) regex used for distributed system uri
-        HEALTH_PATH_REGEX(re.Pattern): (class attribute) regex for health check endpoint
-        LOG_PATH_REGEX(re.Pattern): (class attribute) regex for logs uri
-        PROCESS_PATH_REGEX(re.Pattern): (class attribute) regex for process uri
-    """
-
-    DISTRIBUTED_PATH_REGEX = "/api/v1/distributed"
-    HEALTH_PATH_REGEX = "/api/v1/info"
-    LOG_PATH_REGEX = re.compile(r"^/api/v1/packages/[^/]+/logs$")
-    # technically process id is uuid, but we might change it later
-    PROCESS_PATH_REGEX = re.compile(r"^/api/v1/service/process/[^/]+$")
-
-    @staticmethod
-    def is_distributed_post(request: BaseRequest) -> bool:
-        """
-        check if the request is for distributed services ping
-
-        Args:
-            request(BaseRequest): http reqeust descriptor
-
-        Returns:
-            bool: True in case if request is distributed service ping endpoint and False otherwise
-        """
-        return request.method == "POST" and FilteredAccessLogger.DISTRIBUTED_PATH_REGEX == request.path
-
-    @staticmethod
-    def is_info_get(request: BaseRequest) -> bool:
-        """
-        check if the request is for health check
-
-        Args:
-            request(BaseRequest): http reqeust descriptor
-
-        Returns:
-            bool: True in case if request is health check and false otherwise
-        """
-        return request.method == "GET" and FilteredAccessLogger.HEALTH_PATH_REGEX == request.path
-
-    @staticmethod
-    def is_logs_post(request: BaseRequest) -> bool:
-        """
-        check if request looks like logs posting
-
-        Args:
-            request(BaseRequest): http reqeust descriptor
-
-        Returns:
-            bool: True in case if request looks like logs positing and False otherwise
-        """
-        return request.method == "POST" and FilteredAccessLogger.LOG_PATH_REGEX.match(request.path) is not None
-
-    @staticmethod
-    def is_process_get(request: BaseRequest) -> bool:
-        """
-        check if request looks like process status request
-
-        Args:
-            request(BaseRequest): http reqeust descriptor
-
-        Returns:
-            bool: True in case if request looks like process status request and False otherwise
-        """
-        return request.method == "GET" and FilteredAccessLogger.PROCESS_PATH_REGEX.match(request.path) is not None
-
-    def log(self, request: BaseRequest, response: StreamResponse, time: float) -> None:
-        """
-        access log with enabled filter by request path
-
-        Args:
-            request(BaseRequest): http reqeust descriptor
-            response(StreamResponse): streaming response object
-            time(float): log record timestamp
-        """
-        if self.is_distributed_post(request) \
-                or self.is_info_get(request) \
-                or self.is_logs_post(request) \
-                or self.is_process_get(request):
-            return
-        AccessLogger.log(self, request, response, time)

src/ahriman/web/web.py

@@ -29,7 +29,6 @@ from ahriman.core.configuration import Configuration
 from ahriman.core.database import SQLite
 from ahriman.core.distributed import WorkersCache
 from ahriman.core.exceptions import InitializeError
-from ahriman.core.log.filtered_access_logger import FilteredAccessLogger
 from ahriman.core.spawn import Spawn
 from ahriman.core.status.watcher import Watcher
 from ahriman.models.repository_id import RepositoryId
@@ -122,7 +121,7 @@ def run_server(application: Application) -> None:
     unix_socket = _create_socket(configuration, application)
 
     run_app(application, host=host, port=port, sock=unix_socket, handle_signals=True,
-            access_log=logging.getLogger("http"), access_log_class=FilteredAccessLogger)
+            access_log=logging.getLogger("http"))
 
 
 def setup_server(configuration: Configuration, spawner: Spawn, repositories: list[RepositoryId]) -> Application: