On 06/13/2015 08:41 AM, Thomas De Schampheleire wrote:

On 06/13/2015 08:41 AM, Thomas De Schampheleire wrote:
On June 12, 2015 11:41:30 PM CEST, Mads Kiilerich <m...@kiilerich.com> wrote:
On 06/12/2015 09:09 PM, Thomas De Schampheleire wrote:
# HG changeset patch
# User Cedric De Herdt <cedric.de_he...@alcatel-lucent.com>
# Date 1434135934 -7200
#      Fri Jun 12 21:05:34 2015 +0200
# Node ID c9c5310da1771baed04e70fb45293b8d212e8bb9
# Parent  42feaacb78feecb2a07a3ca19db6cf7ace3fd4c1
notification: use Sender and From header to clarify comment and pull
request mails
Current e-mails are sent from the Kallithea-configured e-mail
address. The
subject line then needs to refer to the user to be useful.
Instead, use the author of comments and pull requests as 'From', and
make
the Kallithea-configured address the 'Sender' in accordance with
RFC5322.

So this will not cause problems for domains using domain keys or other
anti spam "authenticated sender" schemes?
I don't know for sure, but I do know that many other sites use the same 
principle of different from+sender.

Many sites also use a "dead" address as header "from" but use the name of the user it is sending en behalf of in the name part. (That might however be to avoid leaking users' email addresses ...)

And: wouldn't it be more useful to have the first line of the actual
comment in the mail subject instead of the first line of the PR
description?
Maybe, but given your findings on threading with gmail, it would mean that 
comment mails for pull requests are no longer threaded together with the pull 
request creation mail, which made sense to me.

It would have to be in the [] part.

I'm however not sure I would like it. It was merely a question to make sure the options have been explored.


(Somewhat related: I have had a request to put the branch name
"earlier"
in the subject line. This will go in the opposite direction. One thing
that could help could be to hide the branch name if it is default (hg)
or master (git).)
Our original patch had [repo#branch] in front of the subject. I'm open to 
changing that, what would your prefer as subjects?

I don't know. I haven't really though about it yet. It can probably follow later.

Main items from our side are the removal of the user from the subject

Should the "automatic mail" footer also be changed when the user appear as "from" so the recipient actually _can_ respond to the mail? Or was the footer so ambiguous that it still is correct?

/Mads
_______________________________________________
kallithea-general mailing list
kallithea-general@sfconservancy.org
http://lists.sfconservancy.org/mailman/listinfo/kallithea-general

Reply via email to