Add default e-mail domain mapping

[mrw7mrw@…]

I am using Trac for an internal project, so every user has the same e-mail domain. I am also using LDAP, so I prefer to have people login with a username and not an e-mail address. At the same time, I would like people to be able to receive e-mail notifications when tickets are updated.

To accomplish this, I put together a small patch that adds a new configuration variable:

[notification] smtp_defaultdomain=mydomain.com

The patch is small and simply adds another elif statement within Notify.py:

around line 98:
        self.replyto_email = self.config.get('notification', 'smtp_replyto')
        self.from_email = self.from_email or self.replyto_email
        self.default_domain = self.config.get('notification', 'smtp_defaultdomai
n')

around line 309:

for recipient in recipients:
            if recipient.find('@') >= 0:
                emails.append(recipient)
            elif self.email_map.has_key(recipient):
                emails.append(self.email_map[recipient])
            elif self.default_domain != '':
                emails.append(recipient+'@'+self.default_domain)

The idea is that if the system currently can't resolve a username to an e-mail address, it simply appends the default domain to create the e-mail address. Although this isn't suitable for projects with users from different internet domains, it is very useful for projects like mine where every user has the same e-mail domain (i.e. they are all my employees).

[mrw7mrw@…]

Modified Notify.py

[Emmanuel Blot]

I also use LDAP, but as I have access to the SMTP server, I've set up the SMTP server to resolve the username into the user e-mail address, using the LDAP directory.

However, I agree that a default domain may also be useful.

[Haisoo Shin]

This feature should definitely be in the next release. Very useful for small company.

[nielsen@…]

The same functionality as a patch

[anonymous]

This is a duplicate of #464. And decided that it wasnt a neccessary enhancement.

[jornh]

No this is not a duplicate of #464 which was about mapping usernames to user accounts on the host where trac is running. This is about adding an optional fallback to a default domain name that you can configure in trac.ini which is more generic.

IMHO one good argument for having this feature is in a company setting with several trac environments you would otherwise require all users to either use full email adresses in the tickets or that all users set their full email adresses under 'settings' for each separate trac environment.

See also #1286 which describes the need for adding a default domain for ticket notification mails. I think #1286 can be closed because it's essentially about the same requirement but this ticket (#2365) is better since it contains a simpler and newer patch that can be applied to trac 0.9.

[jornh]

improved the patch a bit - applies to the 0.9 stable branch

[Emmanuel Blot]

Should be fixed in [2936].

[notification] option to use in trac.ini is smtp_default_domain.