Breaking change in AbuseFilter's "like" operator
Closed, ResolvedPublic

Description

The "like" operator in AbuseFilter started behaving differently, after Bug 66930 was resolved in https://gerrit.wikimedia.org/r/#/c/141483/ .

Currently:
"a-b" like "a-b"
=> false

Previously:
"a-b" like "a-b"
=> 1

I believe this was an unexpected side effect. At least the change was not documented anywhere to prevent breakage of existing filters.


Version: unspecified
Severity: normal

bzimport set Reference to bz67670.
whym created this task.Via LegacyJul 8 2014, 1:35 PM
whym added a comment.Via ConduitJul 8 2014, 1:40 PM

66935(In reply to Yusuke Matsubara from comment #0)

The "like" operator in AbuseFilter started behaving differently, after Bug
66930 was resolved in https://gerrit.wikimedia.org/r/#/c/141483/ .

Bug 66935, it seems.

Aklapper added a comment.Via ConduitJul 9 2014, 12:29 PM

Probably Ori and Aaron to comment here, as that patch seems to have created this problem?

gerritbot added a comment.Via ConduitJul 11 2014, 9:46 PM

Change 145697 had a related patch set uploaded by Ori.livneh:
Fix double escaping in AFPData::keywordLike()

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

whym added a comment.Via ConduitJul 11 2014, 9:51 PM

Sorry for not including this first, but "+" seems to need the same fix.

"a+b" like "a+b" => false

gerritbot added a comment.Via ConduitJul 12 2014, 2:43 AM

Change 145697 merged by jenkins-bot:
Fix double escaping in AFPData::keywordLike()

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

Add Comment