Page MenuHomePhabricator

Read user agent strings from the cu_useragent table in Special:CheckUser
Open, Needs TriagePublic

Description

Special:CheckUser needs to be updated to read the user agent string from the cu_useragent table by finding the row in that table with the cuua_id that matches the value in the agent_id column.

This could be done by joining the cu_useragent table to each query to the result tables (cu_changes, cu_log_event, and cu_private_event) or performing a separate query to lookup the user agent strings after the results have been selected.

Doing the latter may be more a better solution, as it should avoid making the query yet more complicated by avoiding the need to add a join. However, the former would likely involve less code changes (as the cuua_text column could be aliased as cuc_agent, cule_agent, or cupe_agent).

Acceptance criteria
  • Subclasses of AbstractCheckUserPager will read the user agent string from the cu_useragent table if the schema migration config is set to SCHEMA_COMPAT_READ_NEW.
  • If the schema migration is set to SCHEMA_COMPAT_READ_OLD, the code should continue to get the user agent string from the agent column (for example, cuc_agent)

Related Objects

StatusSubtypeAssignedTask
OpenFeatureNone
OpenFeatureNone
Resolved TBolliger
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenNone
OpenDreamy_Jazz
OpenNone
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedMarostegui
OpenNone
ResolvedMarostegui