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

fix: utilize custom SMTP domain if set #25248

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mccormickt
Copy link

@mccormickt mccormickt commented Jan 8, 2025

Fixes #25241

Confirmed settings are accepted locally with the below config, after setting my domain as the STMP domain in Advanced settings.
image

Checklist for submitter

  • Added/updated automated tests
  • A detailed QA plan exists on the associated ticket (if it isn't there, work with the product group's QA engineer to add it)
  • Manual QA for all new/changed functionality

@mccormickt mccormickt requested a review from a team as a code owner January 8, 2025 16:37
@mccormickt
Copy link
Author

Testing this build locally, invites also send successfully when the above configuration is set:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Custom SMTP HELO domain configuration not used
1 participant