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

[BUG] import verification for CAA record #433

Closed
cm91 opened this Issue Mar 12, 2019 · 5 comments

Comments

Projects
None yet
2 participants
@cm91
Copy link

cm91 commented Mar 12, 2019

Please prefix your issue title with one of the following: [BUG]

Replace everything between stars with current version of your facileManager and module installations:
fM Version : 3.3
{fmDNS} Version : 3.2.1

In raising this issue, I confirm the following (please check boxes, eg [X]):

  • I have read and understood the contributors guide.
  • I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
  • I have checked that the issue I'm posting isn't already reported.
  • I have checked that the issue I'm posting isn't already solved and no duplicates exist in closed issues and opened issues
  • I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.

(BUG) Expected Behavior:

Added a CAA record for my domain. fmDNS should not display any import verification window.

(BUG ) Actual Behavior:
fmDNS displays a import verification window (see attached image)

(BUG) Steps to reproduce:
add a CAA record to a domain

{i added a CAA record to my domain. Every time i configure this domain fmDNS shows a import verification window. When i click import, fmDNS deletes my CAA record.
fm-caa
}

@WillyXJ

This comment has been minimized.

Copy link
Owner

WillyXJ commented Mar 12, 2019

Based on the screenshot, it looks like this zone has "Support Dynamic Updates" enabled which would present the import window anytime the zone records will get edited and the zone file on the server is different. What this is showing is that the CAA record does not exist on the DNS server so it would be deleted (as shown by the Action). You can skip it or cancel which will not delete the record from the database. Once you reload the zone (or build the server config), the CAA record would be on the DNS server and no longer be present in this Import window.

@cm91

This comment has been minimized.

Copy link
Author

cm91 commented Mar 13, 2019

You are right, The option "Support Dynamic Updates" is enabled and your answer makes sense to me.
I
cm91-caa
thought the same and checked the zone filre of my server before opening the zones for editing.
The CAA record is present in zone files. fmDNS brings up the import verification window anyway.

@WillyXJ

This comment has been minimized.

Copy link
Owner

WillyXJ commented Mar 13, 2019

That's interesting and unexpected behavior. Thanks for the follow-up to confirm the report.

@WillyXJ WillyXJ changed the title import verification for CAA record [BUG] import verification for CAA record Mar 13, 2019

@cm91

This comment has been minimized.

Copy link
Author

cm91 commented Mar 13, 2019

Thanks for your answer. I checked without the option "Support Dynamic Updates" enabled. The import verification window is not showing up. Unfortunately this is not a workaround for me because sometimes i need to add/delete records via nsupdate. For now it helps to skip import for the CAA record but it would be great if you could have a look at this problem and maybe fix it in a future release.

Thanks & Regards,
Christian

@WillyXJ WillyXJ added the In progress label Mar 15, 2019

WillyXJ pushed a commit that referenced this issue Mar 16, 2019

WillyXJ
fmDNS - #433 - Improved zone imports
Additional RR types are now supported which also fixes dynamic zone imports.
@WillyXJ

This comment has been minimized.

Copy link
Owner

WillyXJ commented Mar 17, 2019

This is now fixed in fmDNS 3.3 and later.

@WillyXJ WillyXJ closed this Mar 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.