Fix error :from option value for inquiry mailer #126

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@Martin91

Because the SMTP limits, it is not allowed to prepend any host name before the mailbox of the sender.

More details about MAIL for SMTP can be found here: http://tools.ietf.org/html/rfc5321#section-4.1.1.2
And you should know why unable to prepend any hosts through: http://tools.ietf.org/html/rfc5321#appendix-C

Fix error :from option value for inquiry mailer
Because the SMTP limits, it is not allowed to prepend any host name
before the mailbox of the sender.
@Martin91

This comment has been minimized.

Show comment
Hide comment
@Martin91

Martin91 Sep 23, 2013

Owner

Some issues report can be found here: refinery#123

Owner

Martin91 commented on c514d9a Sep 23, 2013

Some issues report can be found here: refinery#123

@ugisozols

This comment has been minimized.

Show comment
Hide comment
@ugisozols

ugisozols Oct 1, 2013

Member

After reading RFCs you linked to and googling around I believe that it's applicable to address that is enclosed in <> and not the name that comes before that.

Please correct me if I'm wrong and I'll reopen.

Member

ugisozols commented Oct 1, 2013

After reading RFCs you linked to and googling around I believe that it's applicable to address that is enclosed in <> and not the name that comes before that.

Please correct me if I'm wrong and I'll reopen.

@ugisozols ugisozols closed this Oct 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment