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

[Sylvain Beucler]

Using sendmail options work for me:

$ mail -s Test address@hidden -- -f "address@hidden" < /dev/null

Gives:
>From address@hidden  Thu Jan 18 21:19:37 2007
Return-Path: address@hidden

It's more a workaround than a solution though.

-- 
Sylvain

On Thu, Jan 18, 2007 at 08:30:28PM +0100, Jim Meyering wrote:
> 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
> 
>