operasoftware / dns-ui

Opera's LDAP-authenticated PowerDNS user interface
Apache License 2.0
282 stars 57 forks source link

Zone creation with faulty SOA-API-DNSUPDATE setting #185

Open emollusion opened 2 years ago

emollusion commented 2 years ago

Hi!

I found that creating a new zone with dns-ui generated an error. dns-ui presents the value powerdns config has for default-soa-update, which in my case is INCEPTION-INCREMENT, but this value should not be presented to this API. https://doc.powerdns.com/md/authoritative/upgrading/ Instead, the value DEFAULT should be presented, or either of these values: https://doc.powerdns.com/authoritative/dnsupdate.html#soa-edit-dnsupdate-settings

The error appears both when the zone is created as well as updated with a record.

This is present for the following versions: PowerDNS Server 4.3.2 dnsui 0.2.7

This setting should perhaps be set as DEFAULT, unless something else is set in the dns-ui config, a default setting could perhaps also be presented in the dns-ui config either way.

Work around is to either change the specified row to DEFAULT or whichever setting is applicable for the current user case, or to edit the pdns database to same.

Thank you!

emollusion commented 2 years ago

Hmm, looked through the code a bit better and see that it is referring to a conf setting, so my initial request above might be in error, So it seems that this has not been included in the latest version? A wuestion though, why is INCEPTION-INCREMENT set as fallback?

nhulsch commented 2 years ago

backwards compatibilty as this setting was introduced later and might not have been set. INCEPTION-INCREMENT was the way to go due to powerdns version at time of that commit.