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

Bot Accounts get created with email notification of Immediately #15921

Open
mattermod opened this issue Oct 12, 2020 · 5 comments
Open

Bot Accounts get created with email notification of Immediately #15921

mattermod opened this issue Oct 12, 2020 · 5 comments
Assignees
Labels
Area/Toolkit Plugins and integrations framework Difficulty/1:Easy Easy ticket Good First Issue Suitable for first-time contributors Hacktoberfest Help Wanted Community help wanted Tech/Go Server

Comments

@mattermod
Copy link
Contributor

When a bot account is created the email notifications are set to immediately. Depending on server config, this makes sense for other accounts but, bot accounts should have this set to never. Because bots have token email addresses, every time someone DMs a bot, it will cause a bounce-back on the corresponding mail server.

This was reported by a community member here #15769

Steps:

  • Set Enable Email Notifications: to true
  • Set Enable Email Batching: to false
  • Enable bot creation
  • Create a bot
  • Check NotifyProps in the Users table for the bot you just created

Observed: email":"true and there is no way to change it via the UI as the bot cannot login. The mail server will now attempt to deliver notifications for DMs
Expected: When a Bot is created or a user is converted to a bot, set email notifications to never or some other solution that prevents Mattermost from trying to send out email notifications when the user is a bot.


If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-29374

@hanzei hanzei added Area/Toolkit Plugins and integrations framework Difficulty/1:Easy Easy ticket Good First Issue Suitable for first-time contributors Tech/Go Server labels Oct 12, 2020
@lurcio
Copy link
Contributor

lurcio commented Oct 12, 2020

May I have a go at this issue please?

@complexanomaly
Copy link

@lurcio Any update with this? I'm happy to take it on if not 👍🏻

@lurcio
Copy link
Contributor

lurcio commented Apr 12, 2021

@lurcio Any update with this? I'm happy to take it on if not 👍🏻

@complexanomaly Sorry, no. Life happened and I totally forgot about this. Please go for it!

@lurcio lurcio removed their assignment Apr 12, 2021
@complexanomaly
Copy link

@lurcio No problem, it happens! 😀

@amyblais Could you update please?

@ashikjm
Copy link

ashikjm commented Aug 27, 2022

@complexanomaly Could I pick this up if you are not actively working on this? Thanks.
cc @amyblais

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/Toolkit Plugins and integrations framework Difficulty/1:Easy Easy ticket Good First Issue Suitable for first-time contributors Hacktoberfest Help Wanted Community help wanted Tech/Go Server
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants