search

lift / framework

Lift Framework
http://liftweb.net
Apache License 2.0
1.27k stars 278 forks source link

Date missing in header of ProtoUser emails #1284

Open tgpfeiffer opened 12 years ago

tgpfeiffer commented 12 years ago

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 commented 10 years ago

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 commented 7 years ago

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