Rietveld mails include reply@codereview-hr.appspotmail.com in various headers

[matze]

INFO     2015-06-15 08:46:55,517 mail_stub.py:143] MailService.Send
  From: sebastian@adblockplus.org
  To: d...@adblockplus.org
  To: m...@adblockplus.org
  Cc: t...@adblockplus.org
  Cc: reply@codereview-hr.appspotmail.com
  Reply-to: s...@adblockplus.org, d...@adblockplus.org, m...@adblockplus.org, t...@adblockplus.org, reply@codereview-hr.appspotmail.com
  Subject: Issue 2681 - Fixed: Thunderbird 31 is listed twice on requirements page (issue 29317012 by s...@adblockplus.org)
  Body:
    Content-type: text/plain
    Data length: 813
INFO     2015-06-15 08:46:56,371 xmpp_service_stub.py:110] Sending an XMPP Message:
    From:
       codereview-hr@appspot.com/bot
    Body:
       Comments published by Sebastian Noack: Issue 2681 - Fixed: Thunderbird 31 is listed twice on requirements page
https://codereview.adblockplus.org/29317012/
    Type:
       chat
    Raw Xml:
       0
    To JIDs:
       d...@adblockplus.org

Whatever this is meant for, it is surely not wanted in our current setup.

[trev]

As noted in the review, we shouldn't fork Rietveld for this - we can override properties of the settings module from the app engine's sandbox. Something like this will work:

  from google.appengine.tools.devappserver2.python.sandbox import
_MODULE_OVERRIDE_POLICIES, ModuleOverridePolicy
  _MODULE_OVERRIDE_POLICIES['settings'] = ModuleOverridePolicy(
      overrides={'RIETVELD_INCOMING_MAIL_ADDRESS': None},
      default_pass_through=True
  )