Page MenuHomePhabricator

Special:BlockedExternalDomains: TransactionProfiler warnings created when adding or removing entries
Closed, ResolvedPublic

Description

Summary

AbuseFilter Special:BlockedExternalDomains is not marked as performing writes, so causes TransactionProfiler warnings when a user adds or removes an entry from the list

Background

  • Example logstash log showing this issue
  • Special pages need to be marked as performing writes otherwise they are marked as nonwrite-POST for the TransactionProfiler
    • Marking the special page as performing writes should fix the TransactionProfiler warnings seen for this special page

Acceptance criteria

  • TransactionProfiler warnings are no longer created when using Special:BlockedExternalDomains

Event Timeline

Change #1246251 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/AbuseFilter@master] Special: Override doesWrites() for Special:BlockedExternalDomains

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

Change #1246251 merged by jenkins-bot:

[mediawiki/extensions/AbuseFilter@master] Special: Declare that Special:BlockedExternalDomains does writes

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