AYR-1348 - Page view logs #674

ltrasca · 2024-11-28T09:42:15Z

Changes in this PR

Added middleware for logging page views on authenticated pages plus tests to cover the changes

N/A

app/tests/test_middlewares.py

app/main/middlewares/log_page_view.py

app/tests/test_middlewares.py

ltrasca added 3 commits November 27, 2024 12:39

added middleware logic

bb7a1bb

added unit tests for middleware

59bfc37

fixed tests

6b719d9

ltrasca requested review from colinbowen and anthonyhashemi November 28, 2024 09:42

ltrasca self-assigned this Nov 28, 2024

Base automatically changed from AYR-1297/audit-app-logs to main November 28, 2024 18:01

anthonyhashemi reviewed Nov 29, 2024

View reviewed changes

app/tests/test_middlewares.py Outdated Show resolved Hide resolved

anthonyhashemi reviewed Nov 29, 2024

View reviewed changes

app/main/middlewares/log_page_view.py Outdated Show resolved Hide resolved

removed if statement from test_log_page_view

336da12

anthonyhashemi reviewed Nov 29, 2024

View reviewed changes

app/tests/test_middlewares.py Show resolved Hide resolved

anthonyhashemi reviewed Nov 29, 2024

View reviewed changes

app/tests/test_middlewares.py Outdated Show resolved Hide resolved

ltrasca added 4 commits November 29, 2024 14:16

fixed unit tests and removed logger mock in favor of caplog

2e079f2

removed access log from download route and adjusted unit tests

e8242a3

merged main to branch

56add2b

removed reference to caplog fixture

e449f2b

anthonyhashemi approved these changes Nov 29, 2024

View reviewed changes

ltrasca merged commit 083293b into main Nov 29, 2024
7 checks passed

ltrasca deleted the AYR-1348/page-view-logs branch November 29, 2024 15:06