- In {T399713}, the `/revision` endpoint in CheckUser is being deprecated as a first step to remove it completely.
- However, the endpoint is being called directly by PageTriage [[ https://gerrit.wikimedia.org/g/mediawiki/extensions/PageTriage/+/af21cb76369bcf180b783676974dfd0184806d7a/modules/ext.pageTriage.showIp/showIp.js#6 | here ]] to retrieve the IP of temp accounts associated with a given revision ID.
- That usage must be updated to use CheckUser's [[ https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/CheckUser/+/refs/heads/master/extension.json#344 | batch endpoint ]] instead.
=== Techncal details
- An example of how to call that endpoint can be found [[ https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CheckUser/+/1171567/39/modules/ext.checkUser.tempAccounts/rest.js#202 | here ]] (the format for the payload can be found [[ https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CheckUser/+/1171567/39/modules/ext.checkUser.tempAccounts/rest.js#96 | here ]]).
- Rationale for this change:
- Using a GET-based endpoint was problematic when a long list of revision IDs were provided, since that resulted in a very long URL that exceeded the URL length limit allowed by the HTTP server or browser (which is typically a few kilobytes).
- By contrast, browsers and servers typically support POST paloads of various megabytes.
- Additoinally, this prevents flooding server logs with URLs containing 500+ IDs.