Opened 11 years ago
Closed 11 years ago
#53 closed task (fixed)
Extend record handling from current "pure/strict"
Reported by: | Kris Deugau | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 1.2.4 |
Version: | Keywords: | ||
Cc: |
Description
Code to date, primarily for reverse zones, sharply limits what records it will allow to be edited (or added).
Extend the validation layers to allow any syntactically valid record in any zone, and
Consider adding a configuration knob to keep the current behaviour ("strict"), the primary extension allowing import, export, and editing ("relaxed"), and a wide-open "anything syntactically valid" mode ("open").
Change History (19)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
comment:3 by , 11 years ago
comment:4 by , 11 years ago
(In [611]) /trunk
Move call to _check_hostname_form() into the individual validator subs;
it's easier to call it from those on the appropriate part(s) of the
record as needed rather than always calling it on just one part, and
then having to reproduce the per-record behaviours in _check_hostname_form().
See #53.
comment:5 by , 11 years ago
comment:6 by , 11 years ago
comment:7 by , 11 years ago
comment:8 by , 11 years ago
comment:9 by , 11 years ago
comment:10 by , 11 years ago
comment:11 by , 11 years ago
comment:12 by , 11 years ago
(In [630]) /trunk
Add a bit more validation in _validate_65280() (A+PTR) Just In Case.
See #53.
A+PTR and AAAA+PTR can have stricter validation because they're logical
representations of pairs of matched records; if there's a mismatch then
they should automatically be coerced down to single A, AAAA, or PTR
records as appropriate.
comment:13 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Primary goal tracked by this ticket is complete; all real record types should now be usable in either forward or reverse zones (no matter how silly it might be to have them there), and a hint of additional related validation is now done for the single-record pseudotypes.
Resolving and opened #54 for adding a configuration knob to control the validation strictness.
comment:14 by , 11 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:15 by , 11 years ago
Reopening, on review I haven't committed (m?)any changes outside the add/edit code path
comment:16 by , 11 years ago
comment:17 by , 11 years ago
(In [634]) /trunk
Add commented fragment in _revswap() sub-sub in importAXFR() to fail on
multicharacter "nibbles" in IPv6 records, since they wouldn't generally
be reachable. However, any-record-in-any-zone means something harmlessly
bizarre may come in that is not reachable via "normal" reverse zone
lookups.
See #53.
comment:18 by , 11 years ago
comment:19 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Calling this complete; can't recall any code paths that are not restricted-use and therefore exempt that are missing updates for this ticket. Resolving since I've called 1.2.4 complete and released.
(In [602]) /trunk