Date missing in header of ProtoUser emails

[tgpfeiffer]

trafficstars

Emails sent by ProtoUser.sendValidationEmail (but also those sent from sendPasswordReset) don't have a Date header in them, which makes them (usually) appear at the very bottom of the email inbox. Also, emails without a Date header are often filtered out by spam filters installed on mail servers.

I suggest to make Mailer.sendMail include the current local time as a default Date header, if not passed in. Otherwise basically everyone needs to override sendValidationEmail etc.

[Shadowfiend]

Hmm… I've never needed to explicitly set the Date header in emails. I think this is because many/most SMTP servers add that header if it's missing.

It's fair to request that sendMail add it if the header is absent; however, as a stopgap, consider looking into your SMTP server/MTA and see if there's a configuration option for it to fill in a missing Date header.

[Shadowfiend]

Unclear how important this is, so I'm going to drop the milestone.