🔄 refactor: Remove unnecessary API logging due to the presence of logging middleware

Author: baoliay2008

api/entry.py

@@ -36,11 +36,11 @@ async def startup_event():
 
 @app.middleware("http")
 async def log_requests(request: Request, call_next):
-    logger.info(
-        f"Received request: {request.client.host} {request.method} {request.url.path}"
-    )
     t1 = time.time()
     response = await call_next(request)
     t2 = time.time()
-    logger.info(f"Cost {(t2 - t1) * 1e3:.2f} ms {response.status_code=}")
+    logger.info(
+        f"Received request: {request.client.host} {request.method} {request.url.path} "
+        f"Cost {(t2 - t1) * 1e3:.2f} ms {response.status_code=}"
+    )
     return response

api/routers/contest_records.py

@@ -2,7 +2,6 @@
 from typing import List, Optional
 
 from fastapi import APIRouter, Request
-from loguru import logger
 from pydantic import BaseModel, NonNegativeInt, conint, conlist
 
 from api.utils import check_contest_name
@@ -30,7 +29,6 @@ async def contest_records_count(
     :param archived:
     :return:
     """
-    logger.info(f"{request.client=}")
     await check_contest_name(contest_name)
     if not archived:
         total_num = await ContestRecordPredict.find(
@@ -64,7 +62,6 @@ async def contest_records(
     :param limit:
     :return:
     """
-    logger.info(f"{request.client=}")
     await check_contest_name(contest_name)
     if not archived:
         records = (
@@ -108,7 +105,6 @@ async def contest_records_user(
     :param archived:
     :return:
     """
-    logger.info(f"{request.client=}")
     await check_contest_name(contest_name)
     if not archived:
         records = await ContestRecordPredict.find(
@@ -147,7 +143,6 @@ async def predicted_rating(
     :param query:
     :return:
     """
-    logger.info(f"{request.client=} {request.url.path=}")
     await check_contest_name(query.contest_name)
     tasks = (
         ContestRecordPredict.find_one(
@@ -181,7 +176,6 @@ async def real_time_rank(
     :param query:
     :return:
     """
-    logger.info(f"{request.client=}")
     await check_contest_name(query.contest_name)
     return await ContestRecordArchive.find_one(
         ContestRecordArchive.contest_name == query.contest_name,

api/routers/contests.py

@@ -2,7 +2,6 @@
 from typing import List, Optional
 
 from fastapi import APIRouter, Request
-from loguru import logger
 from pydantic import BaseModel, NonNegativeInt, conint
 
 from app.db.models import Contest
@@ -56,7 +55,6 @@ async def contests_count(
     :param archived:
     :return:
     """
-    logger.info(f"{request.client=}")
     if archived:
         total_num = await Contest.count()
     else:
@@ -83,7 +81,6 @@ async def contests(
     :param limit:
     :return:
     """
-    logger.info(f"{request.client=}")
     if archived:
         records = (
             await Contest.find_all()

api/routers/questions.py

@@ -33,7 +33,6 @@ async def questions(
     :param query:
     :return:
     """
-    logger.info(f"{request.client=}")
     if not (bool(query.contest_name) ^ bool(query.question_id_list)):
         msg = "contest_name OR question_id_list must be given!"
         logger.error(msg)