clone 487185 -1 retitle -1 devscripts: [bts] Uses the wrong email address by default (broken config by default) severity -1 normal thanks
Justification: affects default configuration On Wed, 30 Dec 2015 19:35:13 -0500 James McCoy <[email protected]> wrote: > On Thu, Dec 31, 2015 at 12:15:56AM +0000, Daniel Shahaf wrote: > > James McCoy wrote on Wed, Dec 30, 2015 at 07:00:17 -0500: > > > On Wed, Dec 30, 2015 at 10:23:51AM +0000, Daniel Shahaf wrote: > > > > This seems to have been fixed: with devscripts-2.15.3 from stable, the > > > > SMTP MAIL FROM address is taken from the $DEBEMAIL envvar. > > > > > > We haven't made any changes related to setting the SMTP envelope, so I > > > doubt it. Maybe your local MTA handles setting the envelope as you > > > expect. > > > > When I run '[email protected] bts', [email protected] is the value passed > > to $smtp->mail(). My MTA doesn't even come into play. > > Right, that's not what this bug is about. It's about when sendmail is > used, not Net::SMTP. > I'd like to report that today I was surprised to find that this [now these] bug[s] is[are] still an issue. The man page indicates that $DEBEMAIL will be respected, so the bts program is not functioning as designed. IIRC --mutt works correctly, --smtp-host works correctly, so this bug only exists in the default configuration. That is to say: sendmail. I've cloned this bug because there are two issues: A) Using the wrong email address, by default (New cloned bug) B) Not supporting sendmail smtp envelope from (Bug #487185) For A) 1. Use the bugs.debian.org SMTP submitter by default. 2. This takes the --smtp-host path, which has been confirmed to do the right thing when DEBEMAIL or EMAIL are set. Prefer DEBEMAIL when set, of course. For B) 1. Add support for sendmail's smtp envelope from. Prefer DEBEMAIL when set, of course. There's arguably a third option: change the documentation to say that the bts command will only do what you expect it to do when you use it with the non-default --mutt or --smtp-host arguments, but I think we'll all agree that this isn't a good solution ;) Kind regards, Nicholas
signature.asc
Description: PGP signature
