πŸ‡ΈπŸ‡°Slovakia @Matus Grinvalsky

Account created on 22 December 2018, over 5 years ago
#

Recent comments

πŸ‡ΈπŸ‡°Slovakia Matus Grinvalsky

Thank you! Now it make sense to me...

I am happy to test 4.x once stable release is available.

Matus

πŸ‡ΈπŸ‡°Slovakia Matus Grinvalsky

Actually, this behavior was introduced in version 3.9 and it is because of following chunk of code. Despite the name, the $admin_email holds the list of emails notification should be sent to, not admin email

The list if addresses for notification is populated to bcc and dummy address is used in to fields

      // Set a dummy no reply email if email list is not empty.
      $to = empty($admin_email) ? \Drupal::config('system.site')->get('mail') : 'noreply@noreply.com';
      $params = [
        'body' => $body,
        'subject' => $subject,
        'nid' => $node->id(),
      ];

      // Allow to alter $admin_email
      // by using hook_admin_content_notification_recipients_alter().
      // @see admin_content_notification.api.php
      \Drupal::moduleHandler()
        ->alter('admin_content_notification_recipients', $admin_email, $node);

      if (!empty($admin_email)) {
        $params['bcc'] = $admin_email;
      }

I assume that reason for this change is preventing full list of recipients to be disclosed to everyone on a list. In my opinion, it would be better to make the "to" address configurable instead hardcoding it to dummy address.

Sending/receiving bouncing emails is annoying and can potentially trigger some spam rules.

Production build 0.69.0 2024