Notification checks should check full name and e-mail too

[Marcus Lindblom <macke@…>]

We've recently switched our committer textsfrom plain 'username' to Firstname Lastname <email@…>.

Now everyone gets notifications, even if the same full name and e-mail is set in Trac. It'd be nice if Trac could expand the check to also match agains these, to reduce e-mail.

[Marcus Lindblom <macke@…>]

If it wasn't obvious from context, we have never_notify_updater = True, which worked well before.

[rblank]

I'm afraid I don't understand the precise situation you describe. Could you please elaborate?

Also, there's no never_notify_updater setting. Did you mean always_notify_updater = False instead?

[marcusl]

Our commit-hook adds info to comments, i.e. "Changed foobar initalization. Fixed #42". Previously, the commiter name was the same as the Trac user-name 'marcusl' (for me), now it is 'Marcus Lindblom <marcus.lindblom@…>'.

When I push commits to our server, I get a notification by the commit-comments added to tickets. I did not previously, as Trac didn't send tickets to the updater (i.e. myself in this case), but now, since I push commits with my full name (and email) instead of only a short username (that matched my account in our local trac) I get e-mails each time I push changes referring to a ticket.

Trac doesn't seem to check more than username (or at least, it didn't in 0.11.5), whereas it'd be nice if it tried to parse the committers name for the common pattern: [A-Z ]+) <[a-z@.]>

I suppose it's configured 'always_notify_updater = false' as you say (not at work right now, so can't check). (I confused myself with the discussion on #2247.)

[rblank]

Ok, now I understand the use case. Previously, you had the same value for the committer name and the Trac user name. Now, your committer name includes your full name and e-mail address, and your Trac user name is still a short name.

I don't understand how this could affect ticket notification, though. Do you have an e-mail address configured in Trac?

Replying to marcusl:

Trac doesn't seem to check more than username (or at least, it didn't in 0.11.5), whereas it'd be nice if it tried to parse the committers name for the common pattern: [A-Z ]+) <[a-z@.]>

What do you mean by "check more than username"? What check do you refer to?

The ticket updater hook has been converted to a component in 0.12, so it's unlikely that we would fix the script in 0.11-stable. Also, the new component can be configured not to send notifications at all on automatic comments. Would you have the possibility to test /trunk and see if it works for you?

[macke@…]

Precisely. (sorry for not being more clear... :-|)

Yup. The full name and e-mail is configured for each user in Trac.

No fix for 0.11.x is understandable. I can try and check how trunk behaves, and apply a patch if necessary.

(Notifications from automatic comments are good, but when always_notify_updater is false, notifications should never go to the original writer.)