Page MenuHomePhabricator

logevents letype=stable returns poorly formatted expiry
Open, LowPublic

Description

A config or modify action (not reset) includes the parameters of the applied protection, but the expiry isn't in a meaningful format as far as JavaScript is concerned. See, e.g., https://en.wikipedia.org/w/api.php?action=query&format=json&list=logevents&titles=Wikipedia%3AAdministrators'%20guide%2FProtecting%2FProtect&formatversion=2&leprop=type%7Ctimestamp%7Cdetails&letype=stable&letitle=Wikipedia%3AAdministrators'%20guide%2FProtecting%2FProtect&lelimit=10

"logevents": [
            {
                "params": {
                    "override": 1,
                    "autoreview": "autoconfirmed",
                    "expiry": "20201101171850"
                },
                "type": "stable",
                "action": "config",
                "timestamp": "2020-11-01T17:08:50Z"
            },

The timestamp is nice but params.expiry is awful. It should be ISO-8601(ish) like the timestamp and other core queries (e.g., other logevents details do this properly (e.g. protect, block)).

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 16 2020, 1:16 PM
Restricted Application added a project: Platform Engineering. · View Herald TranscriptNov 16 2020, 1:19 PM
AMooney triaged this task as Low priority.Nov 17 2020, 2:08 PM
AMooney moved this task from Inbox to Volunteer Needed Tasks on the Platform Engineering board.
Amorymeltzer updated the task description. (Show Details)Nov 21 2020, 1:07 AM