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

domainname/hostname validation with _ #70

Closed
MikeAT opened this issue Apr 29, 2020 · 5 comments
Closed

domainname/hostname validation with _ #70

MikeAT opened this issue Apr 29, 2020 · 5 comments

Comments

@MikeAT
Copy link

MikeAT commented Apr 29, 2020

Hi,

When parsing records like this

$ORIGIN _domainkey.mydomain.at.
XXXXXXXXXXl 3600 CNAME XXXXXXXX.dkim.amazonses.com.

I got this exceptions

Zone "_domainkey.mydomain.at." is not a fully qualified domain name.

I think this is related to #65 (domainanme vs. hostname).

Best,
Michael

@samuelwilliams
Copy link
Contributor

My understanding is that the zone had to be a FQDN - this is the first-time I have encountered an underscore in a zone name.

@MikeAT
Copy link
Author

MikeAT commented May 5, 2020

The zonefile is like this:

$ORIGIN mydomain.at.
www
....
$ORIGIN _domainkey.mydomain.at.
XXXXXXXXXXl 3600 CNAME XXXXXXXX.dkim.amazonses.com.

Bind is processing this without any errors/warnings.

@samuelwilliams
Copy link
Contributor

Sorry for the long delay - this is the first time I have seen multiple $ORIGIN control entries defined in a zone. I had to look up the docs to ensure it was legit.

I will need to make a BC breaking change to the Parser in order to address the issue see https://github.com/Badcow/DNS/blob/master/lib/Parser/Parser.php#L223

Thanks for the issue!

@samuelwilliams
Copy link
Contributor

@MikeAT - once again, thank you for bringing this issue to light. I have pushed a change to the master branch that should resolve your issue here. Could you please confirm that it has?

@MikeAT
Copy link
Author

MikeAT commented Nov 12, 2020

Looks good now. Thank you very much for the fix.

@MikeAT MikeAT closed this as completed Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants