🐛 Bug Report: test email address cause SMTP disconnected

[gepd]

👟 Reproduction steps

1. Add your SMTP credentials in your .env file
2. Run docker exec appwrite doctor
3. You can get SMTP.............disconnected 👎

👍 Expected behavior

You should get SMTP................connected 👍

👎 Actual Behavior

Currently the doctor try to send an email to [email protected]

appwrite/app/tasks/doctor.php

Line 139 in 15107f0

$mail->addAddress('[email protected]', 'Example.com');

debuggin the error returned by the code, I have found this message:

PHPMailer\PHPMailer\Exception: SMTP Error: The following recipients failed: [email protected]: The mail server could not deliver mail to [email protected].  The account or
domain may not exist, they may be blacklisted, or missing the proper dns
entries.

So my SMTP server haven't any problem but as the server can't find the address [email protected] it fails

In my instance I have updated the code and I've added the env var _APP_SMTP_TEST_EMAIL to change this address

Would you accept a PR to fix this issue with this change?

🎲 Appwrite version

Version 1.2.x

💻 Operating system

Linux

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

• I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

• I have read the Code of Conduct

[stnguyen90]

@gepd thanks for raising this! let me check with the team on this.

[LyVanBong]

I am also encountering a problem with the SMTP configuration of the Outlook Office 365 email. Has anyone encountered this issue before? Can anyone help me with this, I would greatly appreciate it. Thank you.

[stnguyen90]

Actually, instead of an environment variable, I think it would be good to expose the email as a parameter to the CLI command.

[mustansirgodhrawala]

Hey there, can this issue be assigned to me? I would like to go a step forward and include a small feature with your permission to create a quick smtp test button with CSRF in the UI for quick smtp testing for easier onboarding as well.

I'm quite intrigued by Appwrite and would like an opportunity to delve into the codebase and contribute beyond bugs.

[mustansirgodhrawala]

Hey there @stnguyen90, can I handle this issue?

[Haimantika]

Hey there @stnguyen90, can I handle this issue?

Assigning this to you.

[mustansirgodhrawala]

@Haimantika Rest assured, I'm handling this issue. I should have a PR out by eod tomorrow, some of my changes could break our e2e tests so I'm trying to avert major changes.

[stnguyen90]

@mustansirgodhrawala will you be able to address the comments in the PR and fix the merge conflict?

[DevilsAutumn]

@mustansirgodhrawala will you be able to address the comments in the PR and fix the merge conflict?

Can I continue this ? If yes, please assign me.

[stnguyen90]

Unassigning @mustansirgodhrawala due to inactivity. Assigning @DevilsAutumn now.

[nick2432]

can i work on this?

[stnguyen90]

@DevilsAutumn, are you still working on this? @nick2432, are you still interested in working on this?