Change Details

- 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 ]]).

- 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.

- 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.