This is getting more clear and easier to review - thanks!


On 09/01/2015 06:28 PM, Thomas De Schampheleire wrote:
# HG changeset patch
# User Thomas De Schampheleire <[email protected]>
# Date 1441051494 -7200
#      Mon Aug 31 22:04:54 2015 +0200
# Node ID 3ca9738eb4f87f38b99404a1eda22fb889e2d0d9
# Parent  6f6acca6da4ad913f45eb18441575ea04d27eedd
email: allow specifying a different noreply email than app_email_from

Perhaps make it more clear what config setting you are introducing.

The configuration setting app_email_from is currently used both for the SMTP
envelope sender as for the From header inside the email (what mail clients
show). It is recommended to use an anonymous but monitored address here.
However, when no such mailbox can be created (e.g. due to corporate
IT rules), app_email_from would need to be set to that of a real (admin)
user. As this address would then be used in the From header of various
emails, to which users could be tempted to reply to, the admin user would
unwillingly be spammed with such reply mails (including out-of-office
mails).

Instead, add an extra configuration setting app_email_noreply that is used
for the From header. Setting app_email_from remains being used for the SMTP
envelope.

diff --git a/development.ini b/development.ini
--- a/development.ini
+++ b/development.ini
@@ -24,13 +24,23 @@ pdebug = false
  ## validation and spam filtering in mail servers.                             
##
  
################################################################################
-## 'From' header for application emails. You can optionally add a name.
+## SMTP envelope sender for application emails.
+## This setting also provides the default for app_email_noreply.
  ## Default:
  #app_email_from = Kallithea
  ## Examples:
  #app_email_from = Kallithea <[email protected]>
  #app_email_from = [email protected]
+## 'From' header for application emails. You can optionally add a name.
+## Defaults to the value of app_email_from.
+## This setting is only useful if you cannot use the value of app_email_from
+## for app_email_noreply, for example because app_email_from is not an 
anonymous
+## address and you do not wish it to be visible to recipients of email.
+## Examples:
+#app_email_noreply = Kallithea <[email protected]>
+#app_email_noreply = [email protected]

"noreply" feels wrong. "noreply" is just a policy thing and one way of using it. It could also go to a place where replies actually are handled.

Also, comparing the names "app_email_noreply" and "app_email_from", it is far from obvious which is which.

One of them could perhaps be "app_email_envelope_from", "app_envelope_from", or perhaps even better something with "bounce" (which might be a more user friendly explanation of "envelope from"). (Alternatively, considering this is for smtp and very close to the smtp settings like smtp_username, it could perhaps be smtp_from?)

These also suggests that the existing header should be preserved for the email address users can see and know: the from header. Also because that is where people are used to specify a name. It could be clarified that the new envelope from config only will use the email address.

What are your thoughts about that?

  ## Subject prefix for application emails.
  ## A space between this prefix and the real subject is automatically added.
  ## Default:
diff --git a/docs/usage/email.rst b/docs/usage/email.rst
--- a/docs/usage/email.rst
+++ b/docs/usage/email.rst
@@ -35,14 +35,23 @@ When Kallithea wants to send an email bu
  determine the intended recipients, the administrators and the addresses
  specified in ``email_to`` in the configuration file are used as fallback.
-Recipients will see these emails originating from the sender specified in the
-``app_email_from`` setting in the configuration file. This setting can either
+The email address used as the SMTP envelope sender -- a technical address which
+is not visible to the recipients of the email

but is used for bounce processing.

-- is specified in the
+``app_email_from`` setting in the configuration file.  This setting can either
+contain only an email address, like `[email protected]`, or both a
+name and an address in the following format: `Kallithea
+<[email protected]>`.
+
+Recipients will see application emails originating from the sender specified in
+the ``app_email_noreply`` setting in the configuration file, which defaults to
+the value of ``app_email_from`` and thus normally need not be set explicitly if
+``app_email_from`` is already set. This setting can either
  contain only an email address, like `[email protected]`, or both
  a name and an address in the following format: `Kallithea
  <[email protected]>`. However, if the email is sent due to an
  action of a particular user, for example when a comment is given or a pull
  request created, the name of that user will be combined with the email address
-specified in ``app_email_from`` to form the sender (and any name part in that
+specified in ``app_email_noreply`` to form the sender (and any name part in 
that
  configuration setting disregarded).
The subject of these emails can optionally be prefixed with the value of


_______________________________________________
kallithea-general mailing list
[email protected]
http://lists.sfconservancy.org/mailman/listinfo/kallithea-general

Reply via email to