akhoury / nodebb-plugin-spam-be-gone

yup
MIT License
23 stars 25 forks source link

Send more data to Akismet #54

Closed cfinke closed 6 years ago

cfinke commented 7 years ago

Currently, calls to Akismet only include user_ip, user_agent, permalink, comment_content, and comment_author. Ideally, they should also include referrer, comment_type, and comment_author_email.

comment_author_email is probably the most important one. A blank comment_type is assumed to be "comment", but it should probably be "forum-post" for new threads and "comment" for replies to threads.

Chris Finke Akismet

akhoury commented 7 years ago

sounds fair

akhoury commented 7 years ago

according to this: https://blog.akismet.com/2012/06/19/pro-tip-tell-us-your-comment_type/

forum-post Forum posts and replies.

which mean both topics and posts should be send as forum-post

akhoury commented 7 years ago

~Also, since when blog is required? https://akismet.com/development/api/#comment-check~ strike that. I'm an idiot, the module takes in the blog as a constructor option.

cfinke commented 7 years ago

both topics and posts should be send as forum-post

That post predates some work I've done on differentiating new forum posts from replies to posts, and the advice in it is still ok, but sending comment for post replies is now marginally more effective. I'll update the post.

akhoury commented 7 years ago

got it, thanks!

barisusakli commented 6 years ago

Seems like referrer, comment_type and comment_author_email are added