Page MenuHomePhabricator

🟦️ Check authorization for PUT statement requests
Closed, ResolvedPublic

Description

Authentication comes for free via the framework, but authorization for PUT requests needs to be implemented on the use case level:

  • check in use case via PermissionChecker
  • create tests for both route handlers

Event Timeline

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

[mediawiki/extensions/Wikibase@master] REST: Extract protectItem test helper

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

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

[mediawiki/extensions/Wikibase@master] REST: Add authorization check to ReplaceItemStatement

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

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

[mediawiki/extensions/Wikibase@master] REST: Extract frameworkish permission error creation

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

Change 819496 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: Extract protectItem test helper

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

Change 819579 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: Add authorization check to ReplaceItemStatement

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

Change 819587 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] REST: Extract frameworkish permission error creation

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

@Jakob_WMDE: Assuming this task is resolved. Thus setting resolved task status. Please reopen and add an active project tag if that is not the case. Thanks.