Periodic cleanup of emails does not exist in 2.0

Currently I have this in a cron job:

$now = new DrupalDateTime('-90 days');
$delete = \Drupal::database()->delete('mail_safety_dashboard');
$delete->condition('sent', $now->getTimestamp(), '<');
$delete->execute();

We could set this up in a container with dependency injection, and add a config to set the retention time.

Thank you laboratory.mike, your solution is a perfect stop gap until we get an actual settings page and this functionality incorporated into the module itself.

use Drupal\Core\Datetime\DrupalDateTime;

/**
 * Implements hook_mail().
 */
function mymodule_helper_cron() {
  // Delete old mail logs.
  $now = new DrupalDateTime('-90 days');
  $delete = \Drupal::database()->delete('mail_safety_dashboard');
  $delete->condition('sent', $now->getTimestamp(), '<');
  $delete->execute();
}

+1 for this essential function being incorporated.

Here's a patch that adds the functionality. It adds a cron job that reads an offset value configured in the settings page of the module. If no value is configured the cron does nothing and everything remains as it is

@nsalves Thanks for providing a patch. Could you create a MR for this change?

There are some code style issues with this patch (in file `mail_safety.module`) which needs attention before we can merge this into the main branch: indent and function doc-block refers to the incorrect hook.

Hello sjerdo, thanks for the feedback and apologies for the oversight. Merge request opened.
Thanks

Thanks for providing a patch @nsalves

To stay in line with Drupal Core (eg temporary files retention), I've asked @drunxfish to update the MR accordingly, including updating the code style issues.

This wille change the setting from a string to an integer (`-90 days` => `7776000`)

I have tagged a new release 2.2.0 containing this cleanup feature