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

HTML Decode errors? #54

Closed
iBrabus opened this issue Sep 17, 2018 · 0 comments · May be fixed by Mistrfilda/ofce-homebridge#4
Closed

HTML Decode errors? #54

iBrabus opened this issue Sep 17, 2018 · 0 comments · May be fixed by Mistrfilda/ofce-homebridge#4

Comments

@iBrabus
Copy link

@iBrabus iBrabus commented Sep 17, 2018

When i got an error: Fatal error: Uncaught Nette\Mail\SendException: Unable to send email: Failed to connect to mailserver at &\quot;ssl://smtp.gmail.com&\quot; port 465, verify your &\quot;SMTP&\quot; and &\quot;smtp_port&\quot; I think it's not beautiful.. Than i change this: (48-51 line @ SendmailMailer.php)

$info = ": $message";
});
if ($res === FALSE) {
    throw new SendException("Unable to send email$info.");

to this

$info = $message;
});
if ($res === FALSE) {
    throw new SendException("Unable to send email: " . html_entity_decode($info));

and the result little bit beautiy for client eyes:
Fatal error: Uncaught Nette\Mail\SendException: Unable to send email: Failed to connect to mailserver at "ssl://smtp.gmail.com" port 465, verify your "SMTP" and "smtp_port" setting in php.ini or use ini_set()

I put backslashes, so you see the problem, because on preview that's decode, but in script no))
Thank you.

@iBrabus iBrabus changed the title I dont know, if it's correct and good HTML Decode errors? Sep 17, 2018
dg added a commit to nette/utils that referenced this issue Sep 18, 2018
dg added a commit to nette/utils that referenced this issue Sep 18, 2018
@dg dg closed this in 28d4e3e Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

1 participant