Now this is a weird one. I just reproduced this on HEAD, maybe it also affects 0.3. My welcome email was sent to "testssl" (which got expanded to "testssl@anarcat.ath.cx" by postfix) instead of "anarcat@localhost". The task log does say the right email:

Sent welcome mail to anarcat@localhost

But my mail logs say otherwise:

Sep  7 14:51:49 mumia postfix/smtp[29728]: E34448F440: to=<testssl@anarcat.ath.cx>, orig_to=<testssl>, relay=209.44.112.66[209.44.112.66]:587, delay=1.9, delays=0.25/0.42/0.92/0.26, dsn=2.0.0, status=sent (250 2.0.0 Ok:  queued as 4FDBB14B89)

This is the relevant code in install_6.inc:

  $mail_success = drupal_mail('install', 'welcome-admin', $client_email, user_preferred_language($account), $mail_params, $from, TRUE);

  if ($mail_success) {
    drush_log(t('Sent welcome mail to @client', array('@client' => $client_email)), 'message');
  }

Obviously, something is wrong with drupal_mail, probably the argument ordering or something...

Comments

anarcat’s picture

anarcat commented
Status: Active » Postponed (maintainer needs more info)

Hum. I have no idea how this happened in the first place, but I can't reproduce anymore... Postponing.

anarcat’s picture

anarcat commented

mig5 says he had problems with $account->mail and used $client_email instead, which was committed to D6. i have backported to d5, maybe that's all we needed to fix this issue.

Anonymous’s picture

Anonymous (not verified) commented

Yeah my task logs had said they sent to my normal email address too but wasn't the case.

That's weird because this issue only started to occur when that other fix was backported from install_5.inc (the 'create admin account anyway' issue), so it's surprising something needed to be backported in the other direction! I guess more investigation is required on this one.

anarcat’s picture

anarcat commented
Status: Postponed (maintainer needs more info) » Fixed

So nobody can reproduce this anymore, my guess is that we fixed it. Otherwise, i'm sure we'll stumble upon this later in the release cycle.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit e91a05e on debian, dev-dns, dev-envobject, dev-koumbit, dev-log_directory, dev-migrate_aliases, dev-multiserver-install, dev-newhooks, dev-nginx, dev-platform_management, dev-ports, dev-purgebackup, dev-restore, dev-services, dev-simplerinstaller, dev-site_rename, dev-ssl, dev_716166_apache_conf, dev_dns, dev_server_verify, prod-koumbit, ssl, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x authored by anarcat: 
    reduce the diff between the drupal 5 and drupal 6 install handlers,...