Add a status report check for transport connection exceptions

Created on 4 November 2019, over 4 years ago
Updated 20 June 2023, about 1 year ago

We had our SMTP server credentials changed, and we didn't notice anything was wrong until we started reviewing log messages. It would be nice if there was a check on the admin/reports/status page if anything was wrong with our SMTP connection.

✨ Feature request
Status

Needs review

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States Dave Reid Nebraska πŸ‡ΊπŸ‡Έ

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡©πŸ‡ͺGermany gngn

    I know "Minimally maintained" ... but there are still a lot of installs and I would like to have the status message...

    #3 modifies swiftmailer_requirements($phase) to

        if ($phase == 'install') {
          ...
        }
        elseif ($phase === 'runtime') {
          ...
    +  else {
    +    try {
    +      ...
    +    }
    +  }
    

    (at least with current 8.x-2.x).

    According to API: hook_requirements $phase can have the following values:

    • Checking installation requirements ($phase == 'install').
    • Checking update requirements ($phase == 'update').
    • Status reporting ($phase == 'runtime').

    The above codes will only run in phase 'update'.I think we need the 'runtime' phase, so we should get rid of the 'else' - or what I am not getting?

    P.S. Changing to current 8.x-2.x

  • πŸ‡ΊπŸ‡ΈUnited States Dave Reid Nebraska πŸ‡ΊπŸ‡Έ

    You can see the patch was originally provided for the 8.x-1.x version which does not currently have an if check for $phase == 'runtime' so It needed to add it. You are correct that it needed to be adjusted for the 8.x-2.x version which does have that already.

  • πŸ‡©πŸ‡ͺGermany gngn

    Thank you for the quick response.

Production build 0.69.0 2024