Page MenuHomePhabricator

Bot object name handler should probably only match whole words
Closed, ResolvedPublic

Description

Someone posted a mac address and got a lot of spam

<christopher> foo bar:D2
<phabot> D2: Code Review is a pretty okay idea - http://phabricator.local/D2 
<christopher> asdfda sadfdaD3
<christopher> foo bar-D3
<phabot> D3: Cool Changes - http://phabricator.local/D3

On the other hand I see things like (D3) fairly often. Noting a minor issue, not super important.

Revisions and Commits

Event Timeline

cburroughs raised the priority of this task from to Needs Triage.
cburroughs updated the task description. (Show Details)
cburroughs added a project: Phabot.
cburroughs added a subscriber: cburroughs.

We no longer match X-Dnnn at HEAD, but the (Dnnn) is a valid use case. We can blacklist :, I guess. You can also use remarkup.ignored-object-names to ignore objects in the form Dn.

(Most of this doesn't apply to the IRC bot, which uses different rules.)

epriestley triaged this task as Wishlist priority.Sep 10 2014, 9:42 PM
epriestley renamed this task from object name handler should probably only match whole words to Bot object name handler should probably only match whole words.Sep 10 2014, 9:50 PM