Edge case: "awww..." is matched
eric-wood opened this issue · 5 comments
eric-wood commented
I'm not sure how important this is, but I figured it was worth documenting!
It turns out the regex used will match the string "awww...", which apparently occurs on a regular basis in our application :\
Example using the autolink regex: http://rubular.com/r/YsunYu1etE
nozpheratu commented
Disallow the posting of cute cat pictures in your application. Problem solved.
eric-wood commented
Good point! I'll see if I can adapt https://github.com/harthur/kittydar to keep the kittens at bay.
eric-wood commented
Suggested patch to the regex: http://rubular.com/r/YsunYu1etE
I can submit a patch if anyone wants this fixed :)
coreyhaines commented
👎 for disallowing the posting of cute cat pictures in ANY application. All applications should have AT LEAST one feature which provides pictures of cats.