Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Cms] Allow setting name in PHPMailer functions like `replyTo`, `to`, `cc`,.. #42

Closed
move bot opened this issue Oct 25, 2018 · 2 comments
Closed

[Cms] Allow setting name in PHPMailer functions like `replyTo`, `to`, `cc`,.. #42

move bot opened this issue Oct 25, 2018 · 2 comments

Comments

@move
Copy link

@move move bot commented Oct 25, 2018

@wottpal commented on Aug 7, 2018, 10:41 PM UTC:

In a lot of PHPMailers functions you can not only set an address (like done in Kirby\Email\PHPMailer) but also a name as the second parameter.

http://edd.wp-a2z.org/oik_api/phpmaileraddreplyto/

This issue was moved by bastianallgeier from k-next/kirby#848.

@Draluak

This comment has been minimized.

Copy link

@Draluak Draluak commented Jun 23, 2019

This would definitely be a an important update, because today you must always use your own email address as 'from', when using phpmailer. Otherwise, because of (dmarc-policy) emails from yahoo or hotmail user will never get send through a contact form. For other services it may cause security warnings or spam detection.

So the workaround is to set 'reply-to' address to the senders email and 'from' to your own. Doing this without adding the name option will show all messages sent through the form in your inbox as sent by yourself, which is not only confusing but making it hard to retrieve messages by sender.

afbora added a commit to getkirby/kirby that referenced this issue Oct 8, 2019
@afbora afbora mentioned this issue Oct 8, 2019
3 of 4 tasks complete
lukasbestle added a commit to getkirby/kirby that referenced this issue Dec 9, 2019
lukasbestle added a commit to getkirby/kirby that referenced this issue Dec 9, 2019
lukasbestle added a commit to getkirby/kirby that referenced this issue Dec 9, 2019
bastianallgeier added a commit to getkirby/kirby that referenced this issue Dec 10, 2019
bastianallgeier added a commit to getkirby/kirby that referenced this issue Dec 10, 2019
bastianallgeier added a commit to getkirby/kirby that referenced this issue Dec 10, 2019
@bastianallgeier

This comment has been minimized.

Copy link
Contributor

@bastianallgeier bastianallgeier commented Dec 10, 2019

Roadmap automation moved this from Prio: mid to Done Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
  
Done
3 participants
You can’t perform that action at this time.