How to get the sender's email address from Zend_Mail_Message? - php

I'm using the Zend_Mail_Message class to display emails inside of my PHP app, and am able to output the subject, content, date and even the NAME of the sender (using $message->from) but I can't figure out how to get the email address of the person who sent the message. The documentation is no help and googling finds a million results for how to send messages with Zend, but nothing about getting the address that sent the message.

EDIT:
This is how I ended up doing it. After some more digging, I found the sender's email in a field called 'return-path'. Unfortunately, this field has a dash in the name (WTF??) so to access it, I had to do this:
$return_path = 'return-path';
$message->reply_to = $zendMessage->$return_path;
Using the return-path caused some problems with some emails, specifically messages from no-reply accounts (mail-noreply#gmail.com, member#linkedin.com etc). These addresses would end up looking something like this:
m-9xpfkzulthmad8z9lls0s6ehupvordjdcor30geppm12kbvyropj1zs5#bounce.linkedin.com
...which obviously doesn't work for display in a 'from' field on the front-end. Email clients like Apple Mail and Gmail show mail-noreply#gmail.com or member#linkedin.com, so that's what I was going for too.
Anyways, after some more research, I discovered that the 'from' field in a Zend Mail Message object always looks something like this:
"user account name" <user#email.com>
The part in < > is what I was after, but simply doing $from = $zend_message->from only gave me the user's account name (hence my original question). After some more playing around, this is how I finally got it working:
$from = $zendMessage->from;
$start = strpos($from, '<');
$email = substr($from, $start, -1);
$result = str_replace('<', '', $email);
Hopefully this will save someone some frustration. If anyone knows of a simpler way of doing this, please let me know.

This works well..
$senderMailAddress = null;
foreach ( $message->getHeader('from')->getAddressList() as $address ) {
if ( $senderMailAddress === null) {
$senderMailAddress = $address->getEmail();
}
}

The main problem here is that many email programs, relay agents and virus scanner along the way do funny stuff to an actually simple and well defined email standard.
Zend_Mail_Message extends to Zend_Mail_Part which has a method called getHeaders(). This will have all the data from an email stored in the head versus the body which is accessed with getContent() and the actual email message.
With this method you'll get an array of all the key/value pairs in the header and while developing you should be able to determine which header field you will actually want. Once you know that you can then get the actual field with getHeader('field_name') or with its actual name directly.
However, if you have to many different email senders you may want to stick with the complete header array though and evaluate multiple fields for the best result like if there's an "reply-to" address. Again there are many uncertainties because the standard isn't always obeyed.

Related

How can I send an email to lots of users using a proper way in CodeIgniter 3

I need some advice because I am building a subscription module. And I have a list of so many emails. Let say 1052 emails. And I have a code like this:
$email_list = $this->getClientEmails(); //all email for now returns 1052 valid emails
$valid_email = array();
$invalid_email = array();
if(count($email_list) > 0) {
for($x = 0; $x < count($email_list); $x++) {
if(valid_email($email_list[$x]['email'])) {
$valid_email[] = $email_list[$x]['email'];
}
//get all invalid emails
/*else {
$invalid_email[] = array(
'id' => $email_list[$x]['id'],
'email' => $email_list[$x]['email']
);
}*/
}
}
$email_string = implode(',', $valid_email);
$this->email->set_mailtype("html");
$this->email->from($from, 'Sample Admin');
$this->email->to($email_string); //send an email to 1052 users
$this->email->cc('test#sampleemail.com.ph');
$this->email->subject($subj);
$this->email->message($content);
$send_mail = $this->email->send();
if($send_mail) {
fp('success');
} else {
fp('failed');
}
Is it fine if I send an email like this? Or should I make a loop to send my email to different users? Means I will not use my imploded string. I will do the sending once in every week. And also what if the sending of email suddenly stops in the middle what should I do? Do I need to resend it again? Or should I make a column in my table that will update if the email is sent or not?
Can you give me some advice about this? That's all thanks.
Okay because you have a mailing list the first thing that i would recommend is that you push the script to background. Use selinium or cron for the same that way page render won't get stuck.
Once done You can send emails either way, send to multiple people or one at a time, both of them are valid and won't cause any problem. The point you need to consider here is the SMTP connection that you maintain.
If you are sending them all individually, you don't want to close connection to SMTP server and reconnect every time to send the mail which would only cause the overhead.
I should say that from your case the most valid way to send email is make a queue on some database preferably redis and have a task handle them in background (cron job if you are on cpanel or selinium if you own the server)
Finally this is a part that you might wanna test out. Because you have a mailing list i am guessing you don't want people to see through your whole list so check the headers when you are sending mails to all at once and if you don't see email from other users , you are good to go else send to each of them separately.
Also one final thing, emails not being delivered is usually bounced which may reflect bad on your server so have a script that flags emails that are constantly rejected and stop sending mails to the same or your ip address might end with bad repo and mails might end up in spam.
Have you thought of using PHPMailer as a library on your CodeIgniter installation?
You could just do it like this:
if(count($email_list) > 0) {
for($x = 0; $x < count($email_list); $x++) {
if(valid_email($email_list[$x]['email'])) {
$mail->addAddress($email_list[$x]['email'], $x);
}
}
}
Please refer to this example on how to use PHPMailer.
I hope this helps, or at least that it gives you a different perspective on how can this be done.
Referring to:
Or should I make a column in my table that will update if the email is sent or not?
Yes, I think that if you want to control if an email has been sent you should use a 1 character field on your table as a "flag" to corroborate that the email has been sent to your users.

Multiple CC emails says email does not comply with RFC 2822, 3.6.2

I create a simple CC script that will attach from database to Mail::send. However, previous code was successfully submitting and working, until later this weekend, it went back to this message along with the Laravel error.
Swift_RfcComplianceException
Address in mailbox given [email1#domain.com,email2#domain.com]
does not comply with RFC 2822, 3.6.2.
This is the code where I pull out the email(s) from database. This will depend if all the properties have emails.
if( !empty($contact) ) {
$property_contact = implode(',', array_filter(array_unique($contact)));
}
// Output when using dd($property_contact);
// “email1#domain.com,email2#domain.com”
Here’s the script for Mail::send
//send email to owner
Mail::send('_dash.emails.profile', $contacts, function ($message) use ($property_contact) {
$ownerEmail = "myemail#info.com";
$ownerName = "Information";
$recipient_name = Request::get('firstname') .' '. Request::get('lastname');
$recipient_email = explode(';', Input::get('email')); // Need to explode the array if value are more than one
print_r($property_contact);
$message->from($recipient_email[0],$recipient_name);
$message->to($ownerEmail,$ownerName)->cc($property_contact);
$message->subject("Profile Update");
});
Explanation:
This is for the user when they updated their profile it will update/notify the admin about their new info along with the other property managers ($property_contact). These property managers will be added as CC and numbers of property managers will depend how many property this user connected.
If the user have 9 properties connection, whenever he update his profile, these 9 property managers will receive the update information.
Case Issue:
Base on Laravel error info, it does not comply the email format that passing from my script. I believe I do not understand the standard format that this Laravel is looking for from my script to pass the value. If I pass 1 value, it is working fine. But, whenever I add more emails, it ended up to this error.
Thank you for your help. I appreciate it.
cc() method accepts either one email or several email addresses in array.
So just edit Your $property_contact variable and You are good to go:
$property_contact = array_filter(array_unique($contact));
Code: https://github.com/laravel/framework/blob/5.2/src/Illuminate/Mail/Message.php#L132. Every method like to(), cc() bcc(), replyTo() using same method.

Parsing values in php

so I have a table in which I have email templates, and these email templates can be later fetched for sending emails.
The structure is :-
for_status subject message
int text text
An example entry is :-
for_status = 1,
subject = Transaction Status Changed,
message = Hi $user->firstname, this is a test message.
Ok, so the problem is, when I send the email with the current message and subject, it displays in the email Hi $user->firstname, this is a test message
Instead of showing, Hi thefirstname here, this is a test message.
I'm fetching user details successfully on the same page in the $user variable.
What's going wrong here?
When you say "table of email templates", is the actual text of the email template in some sort of database?
If this is the case and you have $user->firstname in the database, I'm pretty sure its going to spit that out directly.
You need something whats called placeholders. You need to decide what to have as a place holder, I personally use the following:
{%first_name%}, this is a test message.
Then in your PHP code you just have an array of placeholders & values like this:
$arr = array(
'{%first_name%}' => $user->firstname,
);
//and now replace the body
$body = str_replace(array_keys($arr), $arr, $body);
There is a better way of doing this by using already written libraries or using Regular Expression to correctly parse, but I will leave that up to you to figure it out.
PHP will only perform the substitution for you on strings you have coded into your own PHP files, not those that have come from a database or any other source. If it did, that would be a massive security risk!
You will need to perform substitution on your template yourself.
// $message is: Hi %{userFirstName}, this is a test message.
$message = str_replace('%{userFirstName}', $user->firstname, $message);

Proper formatting of "from" in php mail() function

I want to have the "From" part of a php generated email be just from the company name. Apparently that makes spam filters sad. So, my code is...
$mail->FromName = 'Company Name <some_email#domain.com>';
My issue is that gmail and aol keep returning these emails and the from part looks like this...
From: "Company Name <some_email#domain.com>" <>
Any thoughts about the "<>" at the end?
The <>at the end of "Company Name <some_email#domain.com>" <> indicates that the address is interpreted as containing only the associated name part,with no real email address.
Try generating the From address as 'Company Name' <some_email#domain.com> or as some_email#domain.com (Company Name)
Edit:
Another possible reason for this problem is that your mailer is using separate fields for the name part and the address part of the From header. If so:
$mail->From = "some_email#domain.com";
$mail->FromName = "Company Name";
should solve the problem.
In any decent mail program (MUA) you should be able to see the raw content and headers of emails you've sent and which have been sent to you. If you have a look at some of the latter you'll see that the correct way to do it is:
$from='"Human readable version of address" <mailbox#domain.com>';
BTW: The title of your post says you are using the mail() function but your example code does not call an functions. Your code implies that you are some sort of class to implement your email, but you've provided no details of what that class is - and AFAIK there is no standard class bundled in PHP. Therefore we've no idea what your code is actually doing with the address you feed to it - if it's an off the shelf package it should have come with documentation.

Send multiple mail at a time but other reciepient should not display i.e. other recipient in bcc

I want to send mail to multiple user at a time but one user should not see other users addresses i.e. make them in bcc but mail should be sent only once.I have used PHPMailer for that.
$i = 1;
$emailCount = count($newEmail);
foreach($newEmail as $emailAddress)
{
if($emailCount != $i)
{
$phpmail->AddAddress($emailAddress);
}
$i++;
}
The code you are looking for is:
$phpmail->AddBCC($emailAddress);
Place this in your loop to add all of the addresses you would like to send to as BCCs. I believe that you do not need to specify an address using AddAddress, rather you can just add many BCCs and send the email that way. See this other thread for more info.
See this forum thread for information on including the correct files.
As MrGingerbear pointed out, AddBCC is all you need to do.
Each person will only receive one message and they will not be able to see other recipients.
Here is what I have:
$recipients = array(
'recipient1#domain.com' => 'Alex Baker',
'recipient2#domain.com' => 'Charles Dickens',
);
foreach($recipients as $email => $name)
{
$mail->AddBCC($email, $name);
}
EDIT: Clarifying that AddAddress() is not required, simply used most of the time because you usually include an address in the To: field. However, in your scenario, it would be disadvantageous to do so because then all users would see the address of each user.
EDIT 2: It is not possible, because of the way email is designed, to send only one message, but have the To: field look different to each recipient and also not see the email addresses of the other recipients.

Categories