kobotoolbox / kpi / 24983934715

82%

master: 76%

Build Type

push

github

Committed by web-flow

Commit Message

fix(middleware): track v1 endpoint usage for Token-authenticated requests DEV-2025 (#6966)

### 📣 Summary
Move `V1AccessLoggingMiddleware` logic to the response phase to
correctly capture Token-authenticated API usage.

### 📖 Description
The `V1UserTracker` was failing to record activity from programmatic
scripts using Token Authentication. This occurred because Django's
`process_request` runs before the DRF view layer, leaving `request.user`
as `AnonymousUser` during the initial check.

Fix:
- Relocated the tracking logic from `process_request` to
`process_response` within `V1AccessLoggingMiddleware`.
- This ensures that `request.user` is fully populated by DRF's
authentication classes before the logging logic executes.
- Updated the logic to only track successful, authenticated requests to
the defined legacy patterns.

Coverage Stats

7784 of 11978 branches covered (64.99%)

6 of 6 new or added lines in 1 file covered. (100.0%)

68 existing lines in 5 files now uncovered.

29590 of 36033 relevant lines covered (82.12%)

5.78 hits per line