Page MenuHomePhabricator

🟩 Create a configuration setting to enable wikibase REST API
Closed, ResolvedPublic3 Estimated Story Points

Description

  • enable the REST endpoints if setting is true OR disable the REST endpoints if setting is false

* find or create a test system to enable this setting on test system decision was deferred

Event Timeline

Silvan_WMDE set the point value for this task to 3.
Silvan_WMDE renamed this task from Create a configuration setting to enable wikibase REST API to 🟩 Create a configuration setting to enable wikibase REST API.Feb 23 2022, 8:42 AM
Silvan_WMDE added a project: Story.

I cannot find any hook to dynamically (un)register REST API routes. As an alternative I could imagine a decorator that wraps all our route handlers in the factory method and returns a 4xx (403? 404?) on execute (or checkPreconditions?) if the REST API setting isn't enabled.

Change 766621 had a related patch set uploaded (by Jakob; author: Jakob):

[mediawiki/extensions/Wikibase@master] REST: add feature toggle setting for REST API

https://gerrit.wikimedia.org/r/766621

Change 766778 had a related patch set uploaded (by Jakob; author: Jakob):

[mediawiki/extensions/Wikibase@master] REST: add route handler to use with feature toggle

https://gerrit.wikimedia.org/r/766778

Change 766811 had a related patch set uploaded (by Jakob; author: Jakob):

[mediawiki/extensions/Wikibase@master] REST: add example route handler for feature toggle

https://gerrit.wikimedia.org/r/766811

Change 766621 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: Add feature toggle setting for REST API

https://gerrit.wikimedia.org/r/766621

Change 766778 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: add feature toggle to use with route handler

https://gerrit.wikimedia.org/r/766778

Change 766811 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: add example feature toggled route handler

https://gerrit.wikimedia.org/r/766811

Tested and confirmed it works as intended. Thanks!