Skip to content

Commit d66a69e

Browse files
authored
Re-backport 10713 - add regression test (#11807)
1 parent a76721a commit d66a69e

File tree

3 files changed

+2
-2
lines changed

3 files changed

+2
-2
lines changed

CHANGES/10713.misc.rst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Optimized web server performance when access logging is disabled by reducing time syscalls -- by :user:`bdraco`.

aiohttp/web_protocol.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -457,7 +457,7 @@ def force_close(self) -> None:
457457
def log_access(
458458
self, request: BaseRequest, response: StreamResponse, time: float | None
459459
) -> None:
460-
if self.access_logger is not None and self.access_logger.enabled:
460+
if self._logging_enabled and self.access_logger is not None:
461461
if TYPE_CHECKING:
462462
assert time is not None
463463
self.access_logger.log(request, response, self._loop.time() - time)

tests/test_web_log.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,6 @@ def log(
283283
assert "This should not be logged" not in caplog.text
284284

285285

286-
@pytest.mark.xfail(reason="#11778")
287286
async def test_logger_does_not_log_when_enabled_post_init(
288287
aiohttp_server: AiohttpServer,
289288
aiohttp_client: AiohttpClient,

0 commit comments

Comments
 (0)