Maniphest T431908

Set last_login on first OAuth user create
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	Arian_Ar
	Jul 10 2026, 10:09 PM

Project Tags

Referenced Files

None

Subscribers

Aklapper

Arian_Ar

EbenezerRao

Description

Summary

First-time user creation never sets last_login; only subsequent OAuth callbacks do. Minor admin/analytics inconsistency.

Code

backend/app/api/auth.py — new user branch

Acceptance criteria

last_login set on create and on later logins

Event Timeline

Arian_Ar created this task.Jul 10 2026, 10:09 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 10 2026, 10:09 PM

Arian_Ar claimed this task.Jul 10 2026, 10:18 PM

Arian_Ar triaged this task as Low priority.

Hi! I'd like to work on this task, but I couldn't find the corresponding repository. Could you please link the repository (or provide the clone URL) that contains backend/app/api/auth.py? Thanks!

@EbenezerRao The tool is linked in the tool profile page in Phab, but here you go: https://gitlab.wikimedia.org/toolforge-repos/sentinel

Thanks a lot @Arian_Ar

Submitted a fix for review.

Merge Request: https://gitlab.wikimedia.org/toolforge-repos/sentinel/-/merge_requests/12

This change sets last_login during the initial OAuth user creation while preserving the existing behavior for subsequent OAuth logins.

@EbenezerRao Merged, Thanks for your contribution!

EbenezerRao added a project: RoadToWiki.Tue, Jul 21, 4:46 PM

Adarsh2406 moved this task from Incoming to Completed on the RoadToWiki board.Wed, Jul 22, 8:28 AM

Set last_login on first OAuth user createClosed, ResolvedPublicBUG REPORTActions