Return True or False after sending email

[grim-reapper]

Hi, I'm requesting new features which is very important that is missing from laravel. The feature is related to laravel Mail::send() function. Laravel Mail::send() function does not return anything but null on sending email. So why i am requesting this features because in most we have to store email logs where we can set either email is sent or not base in Mail::send() function whereas Mail function doesent return anything so it's not possible to determine either email is sent or not. I have used PHPMailer library which is good mail library. PHPMailer library send() method return true or false on sending email which is very helpful. So could you please add this little feature? It would be very helpful for those who want to perform some action base on mail true false.
I'm hoping my request will be considered.

[mfn]

For one, there's \Illuminate\Contracts\Mail\Mailer::failures

Get the array of failed recipients

But to the best of my knowledge, it depends on the actual transport to properly support this.

AFAIK with SMTP it won't work as emails are usually enqueued and bounce back so you don't get back a synchronous feedback.

But I know that other transport like postmarkapp using their API, they report many errors synchronously back before spooling them internally.

YMMV

[grim-reapper]

Yes, i know about this function. But i have gone through the core mailer file of framework where send function is written The code of that function is

public function send($view, array $data = [], $callback = null)
    {
        if ($view instanceof MailableContract) {
            return $this->sendMailable($view);
        }
        [$view, $plain, $raw] = $this->parseView($view);

        $data['message'] = $message = $this->createMessage();

        $callback($message);

        $this->addContent($message, $view, $plain, $raw, $data);

        if (isset($this->to['address'])) {
            $this->setGlobalToAndRemoveCcAndBcc($message);
        }

        $swiftMessage = $message->getSwiftMessage();

        if ($this->shouldSendMessage($swiftMessage, $data)) {
            $this->sendSwiftMessage($swiftMessage);

            $this->dispatchSentEvent($message, $data);
        }
    }

if you look at the function it returns nothing, So, to send a message it calls following method inside $this->sendSwiftMessage($swiftMessage);
this method is actually used for sending email, if you explore this method it's return something, whereas in the above method where it's called nothing return back. So if the above function return the output of this method $this->sendSwiftMessage($swiftMessage); we can determine either email sent or not. so my request is just return the output of this $this->sendSwiftMessage($swiftMessage); function in above send method, it would be very helpful.

[mfn]

, if you explore this method it's return something, whereas in the above method where it's called nothing return back

Right, but it doesn't need to necessary, if you look at

framework/src/Illuminate/Mail/Mailer.php

Lines 516 to 525 in 1331ba8

	protected function sendSwiftMessage($message)
	{
	$this->failedRecipients = [];
	try {
	return $this->swift->send($message, $this->failedRecipients);
	} finally {
	$this->forceReconnection();
	}
	}

$this->failedRecipients is used by reference from \Swift_Mailer::send.

What send returns is the number of recipient it sent it to, but failedRecipients contains the ones it couldn't (given my exception above too).

So, whilst I'm not arguing against changing / improving the return value, I'm not sure if it changes anything for your goal, as you already have access to what you need. IHMO 😄