Update email_client.py

[Sekhar-Kumar-Dash]

Fixes #8965

Type of change

• 🐛 Bug fix (non-breaking change which fixes an issue)

Verification Process

To ensure the changes are working as expected:

• Test Location: Specify the URL or path for testing.
• Verification Steps: Outline the steps or queries needed to validate the change. Include any data, configurations, or actions required to reproduce or see the new functionality.

Additional Media:

• I have attached a brief loom video or screenshots showcasing the new functionality or change.
  

Checklist:

• My code follows the style guidelines(PEP 8) of MindsDB.
• I have appropriately commented on my code, especially in complex areas.
• Necessary documentation updates are either made or tracked in issues.
• Relevant unit and integration tests are updated or added.

[MinuraPunchihewa]

@Sekhar-Kumar-Dash This simply seems to set the imap_server and smtp_server attributes to None if an error occurs. I don't think this resolves the issue. If you do make any further changes, please also post a screenshot of a query making use of the connection (a SELECT is preferred).
Further, please make sure that there are no print statements used in the code-base; logging should be used.

[Sekhar-Kumar-Dash]

@Sekhar-Kumar-Dash This simply seems to set the imap_server and smtp_server attributes to None if an error occurs. I don't think this resolves the issue. If you do make any further changes, please also post a screenshot of a query making use of the connection (a SELECT is preferred). Further, please make sure that there are print statements used in the code-base; logging should be used.

@MinuraPunchihewa thank you for taking the time to review this implementation. Given the successful creation of the datasource, I had anticipated it would function as intended. However, I recognize that further adjustments are necessary. I will make the required modifications and update this pull request accordingly.