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
net: LookupMX does not consider IP addresses as valid records #56025
Comments
By the way we ran into similar issue: the |
I've noticed this a bit, lately. Here's a handful of domains with MX records where at least one is an IP address (as of 2024-04-19):
|
CC @rolandshoemaker @golang/security |
We currently support domain names that contain digits in arbitrary places, but it also has to contain at least one non-digit character (except dots). Line 22 in 903e0ff
Also RFC 1123:
RFC 8499:
|
This is a followup to #46979. I found another case where https://golang.org/cl/322230 broke the original logic. Even though MX records MUST contain domain names, occasionally they contain IP addresses. And this change considers IP addresses as invalid records and returns an error making delivery to respective email service providers impossible. The number of such cases is minuscule in the great schema things, but still we at Mailgun handle a few thousand messages to such email providers per hour. So in the spirit of Robustness Principal I believe a change should be made to allow IP addresses in MX records.
The text was updated successfully, but these errors were encountered: