DNS Manager: ipv6 AAAA records cannot be added to match existing A records about sentora-core

[trs998]

Note that duplicate a and aaaa records are also valid, and a setting for the round-robin/cyclic setting would be nice. Bind should handle duplicate a and aaaa records happily anyway.

[Caffe1neAdd1ct]

Bug confirmed, we'll get back to you on which version we are planning to fix this in.

Feel free if you have time to fix this and raise a pull request, we are actively merging pull requests.

[MBlagui]

Duplicate AAA records should be allowed AS long they don't have same value.

[allebb]

Can anyone confirm if this is still an issue? - If so I'll look at getting this fixed in the next few days?

[MBlagui]

Still the case checked right now on last sentora.

I got error: ERROR DNS NOT SAVED

Hostnames must be unique.

Also I would notice all ip that are 192.168.x.x are rejected by DNS manager wich is totally crazy.

[trs998]

Possibly it should warn for IPs in non-global Ip ranges (fe80, 10.,
172.16., 192.168.) But it should accept them. There are many cases where
this is perfectly valid.

N.b. I'm the original reporter - to get my local copy to work I just binned
that existing-record check in the php validation, but ideally it needs to
check same-type same-target record matches before reporting invalid record.
Arguably this is actually also valid (round-robin with unequal server loads
for example) but this is not something users of a shared hosting panel are
likely to do. More commonly an exact matching duplicate record will be a
mistake.

Tristan Scott
On 19 Nov 2014 18:56, "MBlagui" [email protected] wrote:

Still the case checked right now on last sentora.

I got error: ERROR DNS NOT SAVED

Hostnames must be unique.

Also I would notice all ip that are 192.168.x.x are rejected by DNS
manager wich is totally crazy.

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

[michelsup]

This bug keep firing with 1.0.0 version.

[MBlagui]

yep still there

[michelsup]

I made a pull request solving this issue, but I do not believe it will be reviewed quickly, unfortunately.

[allebb]

Code reviewed - I've merged the changes - Sorry just been busy the last week!