Skip to:
Content

Opened 6 years ago

Closed 5 years ago

#1233 closed enhancement (fixed)

Topic subscription and Akismet/spam?

Reported by: chrishajer Owned by:
Milestone: 1.1 Priority: normal
Severity: normal Version: 1.0.2
Component: Back-end Keywords: has-patch akismet
Cc: nightgunner5@…

Description

I am subscribed to this topic:
http://bbpress.org/forums/topic/wp-integration-coockie-does-not-work

I just received this email notification (see text attachment).

I am wondering if the subscriptions are sent out before or after Akismet catches it, or if this one happened to be manually moderated at bbPress.org, so when the notification was sent out it was NOT spam.

I would like to ensure that notifications of spam are not sent out to subscribers.

Attachments (2)

notification-email.txt (2.8 KB) - added by chrishajer 6 years ago.
copy of email notification I received
1233.patch (1.8 KB) - added by Nightgunner5 6 years ago.
Suggested fix - Untested, but should work.

Download all attachments as: .zip

Change History (7)

@chrishajer6 years ago

copy of email notification I received

comment:1 @chrishajer6 years ago

Linking relevant reports from the bbpress.org/forums

http://bbpress.org/forums/topic/try-out-email-subscriptions

@Nightgunner56 years ago

Suggested fix - Untested, but should work.

comment:2 follow-up: @matt6 years ago

Your patch includes other changes that seem to have no relevance, and not sure what you mean by it's "untested. It either works or it doesn't.

comment:3 in reply to: ↑ 2 @Nightgunner56 years ago

  • Cc nightgunner5@… added
  • Keywords has-patch added

Replying to matt:

Your patch includes other changes that seem to have no relevance, and not sure what you mean by it's "untested. It either works or it doesn't.

I haven't been able to test it because my test forum is on my local computer and can't send mail.

The "unrelated" changes are removal of whitespace from the ends of lines and a logic fix for a related function.

comment:4 @GautamGupta5 years ago

  • Keywords subscription notification removed
  • Priority changed from low to normal

comment:5 @chrishajer5 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [2419]) Check bb_ksd_pre_post_status before sending out "subscribe to topic" notifications". Fixes #1233, props Nightgunner5

Note: See TracTickets for help on using tickets.