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

Jump to bottom

Email reports not working when email SMTP server host contains a space #7157

Closed
GJjoe opened this issue Feb 9, 2015 · 8 comments
Closed

Email reports not working when email SMTP server host contains a space #7157

GJjoe opened this issue Feb 9, 2015 · 8 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone
Piwik 2.11.0

Comments

@GJjoe
Copy link

GJjoe commented Feb 9, 2015

Gets this error using .info TLD

An error occured while sending 'HTML Email Report - 3.2015-02-07.11.en.html' to user@gmail.com. Error was ''mail.xxx.info ' appears to be a DNS hostname but cannot match TLD against known list, 'mail.xxx.info ' does not appear to be a valid URI hostname, 'mail.digianalytics.info ' does not appear to be a valid local network name'
@ThaDafinser
Copy link
Contributor

The error comes from here https://github.com/piwik/piwik/blob/5d799ee1a434ce97921b412bc34568b7241870ea/libs/Zend/Validate/Hostname.php#L75

@mattab
Copy link
Member

mattab commented Feb 9, 2015

maybe upgrading our Zend Mail library would help this issue?

@mattab mattab added the Bug For errors / faults / flaws / inconsistencies etc. label Feb 9, 2015
@mattab mattab added this to the Short term milestone Feb 9, 2015
@mnapoli
Copy link
Contributor

mnapoli commented Feb 10, 2015

@ThaDafinser info seems to be a valid TLD in the file though

@mnapoli
Copy link
Contributor

mnapoli commented Feb 10, 2015

FYI the history of https://github.com/zendframework/zf1/blob/master/library/Zend/Validate/Hostname.php doesn't show anything obvious that has been fixed related to the issue here.

@GJjoe
Copy link
Author

GJjoe commented Feb 10, 2015

changed to .INFO and it worked.
On Feb 9, 2015 11:55 PM, "Matthieu Napoli" notifications@github.com wrote:

FYI the history of
https://github.com/zendframework/zf1/blob/master/library/Zend/Validate/Hostname.php
doesn't show anything obvious that has been fixed related to the issue here.

—
Reply to this email directly or view it on GitHub
#7157 (comment).

@ThaDafinser
Copy link
Contributor

@F0NIX @mnapoli yes it's in there, but the error messages is exactly from there.

It should also work regardless of the casesentivitiy...
https://github.com/zendframework/zf1/blob/master/library/Zend/Validate/Hostname.php#L1433-L1434

@mnapoli
Copy link
Contributor

mnapoli commented Feb 10, 2015

Ha ok having a closer look at the error message:

'mail.xxx.info ' appears to be a DNS hostname but cannot match TLD against known list

'mail.xxx.info ' there's an extra space at the end. I'll rename the issue so that we remember to trim extra spaces.

@mnapoli mnapoli changed the title Email reports not working with .info TLD Email reports not working when email address contains a space Feb 10, 2015
@mattab mattab changed the title Email reports not working when email address contains a space Email reports not working when email SMTP server host contains a space Feb 11, 2015
@mattab mattab closed this as completed in af5b516 Feb 11, 2015
@mnapoli mnapoli modified the milestones: Piwik 2.11.0, Short term Feb 11, 2015
@rsolvang
Copy link

I have the same problem when trying to send emails from a .link SMTP address. Is there any way to bypass this check, as it seems out of date?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Milestone
Piwik 2.11.0
Development

No branches or pull requests
5 participants
@mattab @ThaDafinser @mnapoli @rsolvang @GJjoe