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

dns_get_record(): A temporary server error occurred. #260

Closed
CarlAtIssl opened this issue Sep 4, 2020 · 1 comment
Closed

dns_get_record(): A temporary server error occurred. #260

CarlAtIssl opened this issue Sep 4, 2020 · 1 comment
Labels

Comments

@CarlAtIssl
Copy link

Hi

When we use this check on an invalild email address, we get the following exception emailed to us:

** Message **
dns_get_record(): A temporary server error occurred.

** File **
DNSCheckValidation.php

** Action **
update

** Line **
121

And then get some other file references:

** Trace **

Illuminate\Foundation\Bootstrap\HandleExceptions -> handleError
DNSCheckValidation.php : 121
dns_get_record
DNSCheckValidation.php : 107
Egulias\EmailValidator\Validation\DNSCheckValidation -> validateDnsRecords
DNSCheckValidation.php : 80
Egulias\EmailValidator\Validation\DNSCheckValidation -> checkDns
MultipleValidationWithAnd.php : 65
Egulias\EmailValidator\Validation\DNSCheckValidation -> isValid
EmailValidator.php : 37
Egulias\EmailValidator\Validation\MultipleValidationWithAnd -> isValid
ContactController.php : 876
Egulias\EmailValidator\EmailValidator -> isValid
ContactController -> update

Any idea why this just dies and gives an exception rather than invalidating the request?

Cheers in advance
Carl.

@egulias egulias added the bug label Sep 6, 2020
@egulias
Copy link
Owner

egulias commented Sep 6, 2020

hi @CarlAtIssl , thanks for reporting.
Two bugs in one :).
First, there's a php bug: https://bugs.php.net/bug.php?id=73149 that prevents the exception to be handled.
Second, there's no exception control within the validator itself.

There's a workaround, will implement.

Eduardo

egulias added a commit that referenced this issue Sep 6, 2020
@egulias egulias closed this as completed in f46887b Sep 6, 2020
egulias added a commit that referenced this issue Sep 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants