[Savannah-hackers-public] improper From_ headers [Re: Mail delivery failed: returning message to sender

[Jim Meyering]

Hi Sylvain,

Remember the problem I had with auto-generated mail sent
via the coreutils git-update (to-cvs) hook?
I thought I'd worked around it by using mail's -a "From ..." option,
but that doesn't work (and now that I check, I wonder if it ever did).
I've included below what I received today, in response to a just-pushed
change to coreutils.git.

You said there is a rewrite rule to use each user's email address as
the sender.  Is there a way to override that so I can send a message
with a From_ header that won't be rejected?

Jim

Mail Delivery System <address@hidden> wrote:
> This message was created automatically by mail delivery software.
>
> A message that you sent could not be delivered to one or more of its
> recipients. This is a permanent error. The following address(es) failed:
>
>   address@hidden
>     SMTP error from remote mailer after RCPT TO:<address@hidden>:
>     host mx.meyering.net [82.230.74.64]: 550 5.7.1 <address@hidden>:
>     Sender address rejected: Mail claims to be from OURDOMAIN.com yet isn't.
>
> ------ This is a copy of the message, including all the headers. ------
>
> Return-path: <address@hidden>
> Received: from cvs.savannah.gnu.org ([199.232.41.69] helo=savannah.gnu.org)
>       by savannah.gnu.org with esmtp (Exim 4.50)
>       id 1H7TPb-0006VE-Ad
>       for address@hidden; Thu, 18 Jan 2007 09:18:23 +0000
> Received: from meyering by savannah.gnu.org with local (Exim 4.50)
>       id 1H7TPb-0006V8-35
>       for address@hidden; Thu, 18 Jan 2007 09:18:23 +0000
> To: address@hidden
> Subject: GNU coreutils source repository: Changes to 'master'
> Message-Id: <address@hidden>
> From: Jim Meyering <address@hidden>
> Date: Thu, 18 Jan 2007 09:18:23 +0000
>
>>From address@hidden
>
>  ChangeLog            |   18 +++++++++++++++++
>  NEWS                 |    2 +
>  src/c99-to-c89.diff  |    6 ++--
>  src/mv.c             |    4 +-
>  src/remove.c         |   10 ++++++--
>  src/remove.h         |   13 ++++++++++-
>  src/rm.c             |   18 ++++++++--------
>  tests/rm/Makefile.am |    3 +-
>  tests/rm/i-never     |   53 
> ++++++++++++++++++++++++++++++++++++++++++++++++++
>  9 files changed, 107 insertions(+), 20 deletions(-)
>
> New commits:
> commit 03b1e07ae02d06d27a18836fa29ac2149bf08601
> Author: Jim Meyering <address@hidden>
> Date:   Thu Jan 18 10:18:02 2007 +0100
>
>     * src/c99-to-c89.diff: Adjust remove.c offsets.
>
> commit 0928c2414ab21682600e6572bc628a405fb1ea80
> Author: Jim Meyering <address@hidden>
> Date:   Wed Jan 17 17:02:40 2007 +0100
>
>     Make "rm --interactive=never ..." never prompt.
>     * NEWS: Mention this.
>     * src/remove.h (enum rm_interactive): New ternary type.
>     (struct rm_options) [interactive]: Use it, here -- rather than bool.
>     * src/remove.c (prompt): Reflect type change.
>     * src/mv.c (rm_option_init): Initialize to RMI_NEVER now.
>     * src/rm.c (main): Add a FIXME comment for '-d' option.
>     Adapt to type change of rm_options.interactive.
>
>     * tests/rm/i-never: New file.  Test for the above fix.
>     * tests/rm/Makefile.am (TESTS): Add i-never.
>
> mirroring 0928c2414ab21682600e6572bc628a405fb1ea80 to cvs...
> Applying to CVS commit 0928c2414ab21682600e6572bc628a405fb1ea80 from parent 
> d1ad73e0bf6a1b6c62e424c1f28cf344fca10600
> The commit affects:
>  tests/rm/i-never
>  ChangeLog
>  NEWS
>  src/mv.c
>  src/remove.c
>  src/remove.h
>  src/rm.c
>  tests/rm/Makefile.am
>
> Creating new directories
> 'Patching' binary files
> Patching non-binary files
> patching file ChangeLog
> patching file NEWS
> patching file src/mv.c
> patching file src/remove.c
> patching file src/remove.h
> patching file src/rm.c
> patching file tests/rm/Makefile.am
> patching file tests/rm/i-never
> Commit to CVS
> Patch: Make "rm --interactive=never ..." never prompt.
> Autocommit
>   cvs commit -F .msg tests/rm/i-never ChangeLog NEWS src/mv.c src/remove.c 
> src/remove.h src/rm.c tests/rm/Makefile.am
> /sources/coreutils/coreutils/tests/rm/i-never,v  <--  tests/rm/i-never
> initial revision: 1.1
> /sources/coreutils/coreutils/tests/rm/Makefile.am,v  <--  tests/rm/Makefile.am
> new revision: 1.44; previous revision: 1.43
> /sources/coreutils/coreutils/ChangeLog,v  <--  ChangeLog
> new revision: 1.2201; previous revision: 1.2200
> /sources/coreutils/coreutils/NEWS,v  <--  NEWS
> new revision: 1.467; previous revision: 1.466
> /sources/coreutils/coreutils/src/mv.c,v  <--  src/mv.c
> new revision: 1.179; previous revision: 1.178
> /sources/coreutils/coreutils/src/remove.c,v  <--  src/remove.c
> new revision: 1.179; previous revision: 1.178
> /sources/coreutils/coreutils/src/remove.h,v  <--  src/remove.h
> new revision: 1.18; previous revision: 1.17
> /sources/coreutils/coreutils/src/rm.c,v  <--  src/rm.c
> new revision: 1.143; previous revision: 1.142
> Committed successfully to CVS
> mirroring 03b1e07ae02d06d27a18836fa29ac2149bf08601 to cvs...
> Applying to CVS commit 03b1e07ae02d06d27a18836fa29ac2149bf08601 from parent 
> 0928c2414ab21682600e6572bc628a405fb1ea80
> The commit affects:
>  ChangeLog
>  src/c99-to-c89.diff
>
> Creating new directories
> 'Patching' binary files
> Patching non-binary files
> patching file ChangeLog
> patching file src/c99-to-c89.diff
> Commit to CVS
> Patch: * src/c99-to-c89.diff: Adjust remove.c offsets.
> Autocommit
>   cvs commit -F .msg ChangeLog src/c99-to-c89.diff
> /sources/coreutils/coreutils/ChangeLog,v  <--  ChangeLog
> new revision: 1.2202; previous revision: 1.2201
> /sources/coreutils/coreutils/src/c99-to-c89.diff,v  <--  src/c99-to-c89.diff
> new revision: 1.22; previous revision: 1.21
> Committed successfully to CVS