In T332437 we implemented instrumentation to track how temp users engage with editing interfaces.
This ticket involves the work of verifying those actions are being logged in EditAttemptStep in the ways we expect.
In T332437 we implemented instrumentation to track how temp users engage with editing interfaces.
This ticket involves the work of verifying those actions are being logged in EditAttemptStep in the ways we expect.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Restricted Task | |||||
| Resolved | kostajh | T294511 2021 Security Team wikireplicas audit | |||
| Declined | None | T284948 Raw IPs of logged-out users disclosed in wiki-replicas | |||
| Resolved | Niharika | T324492 Temporary accounts - MVP | |||
| Resolved | Madalina | T326816 [Epic] Update features for temporary accounts | |||
| Resolved | ppelberg | T326876 Update Editing Team-owned products that may be affected by temporary users | |||
| Open | SNowick_WMF | T341157 QA new field added to EditAttemptStep to track temp users |
This task was blocked on the deployment of temp accounts. While we verified that the new user_is_temp field now exists in EditAttemptStep, we wanted to wait for temp accounts to be deployed to confirm that data was logging for the new field as expected.
Now that it's deployed to testwiki, we can proceed with some initial checks. I'll triage this with @ppelberg and resolve once complete.
I reviewed events in EditAttemptStep and VisualEditorFeatureUse to confirm that the user_is_temp field is logging as expected now that temp accounts has been deployed to pilot wikis. See summary of checks and results below:
Further investigation needed:
@DLynch @SNowick_WMF - Any ideas to why some app edits by temporary users are being tagged as user_id = 0? I am wondering if this is expected or related to a difference in app instrumentation.
Passed QA Checks: ✅
Thanks for raising this issue @MNeisler, looking into this now. It does look like the user_id = 0 was happening before we were sending user_is_temp data so this may be unrelated to temp accounts and had been happening without our knowledge previously.
Looking at data from 2024-12 only I am seeing 7.6% of events have NULL values for user_is_temp which I would bet are coming from users who have not yet upgraded their app version to versions where sending user_is_temp code is implemented. We often have stragglers and I expect this percent to decline over time but will keep an eye on this as well.
I pulled data from 2024-09 and see that ~54% of iOS/Android events had user_id = 0 values (and no user_is_temp data). Note: since we use the app_install_id identifier that we send to EditAttemptStep (see https://phabricator.wikimedia.org/T353911) , I haven't worked with the user_id value data so didn't catch this.
I will make a new ticket for iOS/Android engineers to look into this, it looks to me like from your end the QA Task is ok to resolve.