We needed to allow a wildcard for the username (*@example.com) in the pass-through list.

CommentFileSizeAuthor
reroute_email.patch1.62 KBjody lynn

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

I'm not inclined to favor this patch, as the module is intended for development, not for fancier things. I haven't come to terms with the pass-through yet :-)

jody lynn’s picture

jody lynn
she/her
Primary language English
commented

Well, we're using it for development, but we have 15 developers working on the site and a whole slew of testers with email addresses of one organization, so it's inconvenient to add every email for everyone who ever might test the dev site to see how various emails are working.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Version: 7.x-1.0 » 7.x-1.x-dev

That's reasonable. I'd think it would be a lot easier to have a shared email account that you deliver these things to.

I'm listening :-)

jody lynn’s picture

jody lynn
she/her
Primary language English
commented

I'm using it now for another project, so it's proving useful. It seems a lot faster to add passthrough for ours and our clients' domain rather than asking them for a list of all staff members that might be involved in QA and setting up a shared email account for all of them (and then confusing them by having them all get all of the emails). The main reason these QA folks even need to be getting emails is that they keep needing password reset, so whatever I do needs to be simpler than remembering a password.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

I don't currently like the way the passthrough email is jammed into the same field with the "to" emails. IMO we should make these two separate fields. I'd probably be more OK with this at that point.

jweowu’s picture

jweowu commented

Cross-referencing with #1571500: Separate the recipient (list) from the address whitelist. Add domain whitelist.

jweowu’s picture

jweowu commented
Issue summary: View changes
Related issues: +#1571500: Separate the recipient (list) from the address whitelist. Add domain whitelist.
jedihe’s picture

jedihe commented
Title: Allow wildcards in passthrough emails (*@example.com) » Implement passthrough domain(s)
Status: Needs review » Closed (duplicate)

Even though the title of the issue mentions wildcards, the details of the request look much more as "Allow setting passthrough domains", which is being worked on in #1571500: Separate the recipient (list) from the address whitelist. Add domain whitelist..

Hence, setting as duplicate. Also updating the title.