Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stream suddenly sent tons of alerts based on hidden post type, possibly due to REST API change in 3.9.0 #1314

Open
cbirdsong opened this issue Mar 10, 2022 · 2 comments
Labels
bug integration Integration with other plugins or WordPress core functionality

Comments

@cbirdsong
Copy link

cbirdsong commented Mar 10, 2022

Yesterday one of our sites suddenly started sending tons of Stream alert emails about deleted items. It had this alert configured:

CleanShot 2022-03-10 at 08 52 22

This site has Post SMTP plugin installed with logging enabled. It uses a hidden postman_sent_mail custom post type for logged items, and it limits the log size by is a fixed number of posts, instead of a time period like Stream:

CleanShot 2022-03-10 at 09 42 14

Stream's records show "item deleted" alerts being sent after Post SMTP sends and logs an email, which creates a new postman_sent_mail and presumably also deletes the oldest one: stream-filtered-redacted.csv

This seems like it could be a consequence of monitoring REST API changes (#1264), so it might be good if alert monitoring only applies to public post types by default?

@kasparsd kasparsd added bug integration Integration with other plugins or WordPress core functionality labels Mar 11, 2022
@kasparsd
Copy link
Contributor

Thanks for the detailed report @cbirdsong! Can you please confirm if this is happening when new postman_sent_mail posts are created and deleted as the Post SMTP plugin schedules and sends emails?

@cbirdsong
Copy link
Author

I disabled the alert as soon as I discovered it and didn't attempt to debug by triggering any further emails since it was on a live site, but the times in the Post SMTP log lined up exactly with the Stream log/emails.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug integration Integration with other plugins or WordPress core functionality
Projects
None yet
Development

No branches or pull requests

2 participants