jenkins-bot has submitted this change and it was merged.
Change subject: Run contentlength filter first, as it's the fastest
......................................................................
Run contentlength filter first, as it's the fastest
We should try to fail early and skip complex rules if possible. The
contentlength filter is extremely simple, so run it first.
Change-Id: Ia41f0d185def7d94b85c793f3b362b587c47518f
---
M container.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Mattflaschen: Looks good to me, approved
jenkins-bot: Verified
diff --git a/container.php b/container.php
index d5c471a..3e632c4 100644
--- a/container.php
+++ b/container.php
@@ -838,12 +838,12 @@
$c['controller.spamfilter'] = function( $c ) {
return new Flow\SpamFilter\Controller(
+ $c['controller.contentlength'],
$c['controller.spamregex'],
$c['controller.ratelimits'],
$c['controller.spamblacklist'],
$c['controller.abusefilter'],
- $c['controller.confirmedit'],
- $c['controller.contentlength']
+ $c['controller.confirmedit']
);
};
--
To view, visit https://gerrit.wikimedia.org/r/248118
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia41f0d185def7d94b85c793f3b362b587c47518f
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
Gerrit-Reviewer: Mattflaschen <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
