Problem/Motivation

Drupal mail modules allow delivery to email addresses with a display name, eg "John Smith <john.smith@example.org>", however this module does not recognise such addresses and fails to match against the whitelisted addresses.

Steps to reproduce

Send to a recipient in the format "Display Name <email@example.org>". Add "email@example.org" to the whitelisted addresses. The email will be rerouted even though it is in the whitelist.

Proposed resolution

Keep the whitelist as is (no change to allowed delimiters and continue to enter an address), but in the code where the original addresses are split into an array rewrite this to be aware of recipients formatted with a display name and address.

Would there be interest in supporting this? I could have a crack at writing a patch if so.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

gaddman created an issue. See original summary.

bohart’s picture

Version: 8.x-1.3 » 2.x-dev
Status: Active » Needs work

Good catch! It should be a nice improvement for the module.
Looking forward to getting the merge request!

Please be aware to use 2.x-dev branch to start a new branch and for merge request branch destination.
Thanks!

gaddman’s picture

Assigned: Unassigned » gaddman
Status: Needs work » Needs review
_shy’s picture

Assigned: gaddman » _shy
Issue tags: +LutskCodeSprint2021

  • bohart committed a9d9d41 on 2.x
    Issue #3204208 by gaddman, _shY, bohart: Tests updates for display name...
bohart’s picture

Assigned: _shy » Unassigned
Status: Needs review » Fixed

@gaddman, @_shY, thanks for your contribution!
This new feature was tested and reviewed by our team at #LutskCodeSprint2021.

Committed to 2.x-dev branch.
There is no backward compatibility planned with 8.x-1.x branch.

So, marking this one as fixed.
Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.