Page MenuHomePhabricator

Install Adiutor extension on trwiki
Open, Stalled, Needs TriagePublic

Event Timeline

Aklapper renamed this task from Activate Adiutor at trwiki to Install Adiutor extension on trwiki.Jan 7 2024, 9:23 PM
Aklapper changed the task status from Open to Stalled.

This extension is not yet deployed on any Wikimedia wiki. Thus it needs several review, for example security and performance.
Please see https://www.mediawiki.org/wiki/Writing_an_extension_for_deployment for the required steps before it can get deployed

Few points that probably need to be resolved:

  • Adding a rest API that allows pinging arbitrary editors is not a great idea (Fixed by @Vikipolimer in this patch)
  • Loading configuration via this method isn't going to do wonders for page startup times, I'd suggest using packageFiles to load the config instead.

Few points that probably need to be resolved:

  • Adding a rest API that allows pinging arbitrary editors is not a great idea
  • Loading configuration via this method isn't going to do wonders for page startup times, I'd suggest using packageFiles to load the config instead.

Hi, I'll see what can I do with this. Thanks for your feedback.

Few points that probably need to be resolved:

  • Adding a rest API that allows pinging arbitrary editors is not a great idea (Fixed by @Vikipolimer in this patch)
  • Loading configuration via this method isn't going to do wonders for page startup times, I'd suggest using packageFiles to load the config instead.

If we use this configuration JSON as a packageFiles, we cannot allow wikis to edit this configuration on their MediaWiki. We also need to keep configurations like 'MediaWiki:Config.json' accessible on their wikis.