netbox-plugin-dns 0.22.5__tar.gz → 0.22.8__tar.gz

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: netbox-plugin-dns
-Version: 0.22.5
+Version: 0.22.8
 Summary: NetBox DNS is a NetBox plugin for managing DNS data.
 Home-page: https://github.com/peteeckel/netbox-plugin-dns
 License: MIT

netbox_plugin_dns-0.22.8/netbox_dns/__init__.py

@@ -0,0 +1,51 @@
+import sys
+
+from extras.plugins import PluginConfig
+from django.db.utils import OperationalError
+
+try:
+    # NetBox 3.5.0 - 3.5.7, 3.5.9+
+    from extras.plugins import get_plugin_config
+except ImportError:
+    # NetBox 3.5.8
+    from extras.plugins.utils import get_plugin_config
+
+__version__ = "0.22.8"
+
+
+class DNSConfig(PluginConfig):
+    name = "netbox_dns"
+    verbose_name = "NetBox DNS"
+    description = "NetBox plugin for DNS data"
+    min_version = "3.5.0"
+    max_version = "3.99.0"
+    version = __version__
+    author = "Peter Eckel"
+    author_email = "pete@netbox-dns.org"
+    required_settings = []
+    default_settings = {
+        "zone_default_ttl": 86400,
+        "zone_soa_serial_auto": True,
+        "zone_soa_serial": 1,
+        "zone_soa_refresh": 43200,
+        "zone_soa_retry": 7200,
+        "zone_soa_expire": 2419200,
+        "zone_soa_minimum": 3600,
+        "feature_ipam_coupling": False,
+        "tolerate_underscores_in_hostnames": False,
+        "tolerate_leading_underscore_types": [
+            "TXT",
+            "SRV",
+        ],
+        "tolerate_non_rfc1035_types": [],
+        "enable_root_zones": False,
+        "enforce_unique_records": True,
+        "enforce_unique_rrset_ttl": True,
+    }
+    base_url = "netbox-dns"
+
+
+#
+# Initialize plugin config
+#
+config = DNSConfig

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/api/serializers.py

@@ -241,6 +241,7 @@ class RecordSerializer(NetBoxModelSerializer):
             "display",
             "type",
             "name",
+            "fqdn",
             "value",
             "status",
             "ttl",

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/filters/record.py

@@ -3,17 +3,22 @@ from django.db.models import Q
 
 from netbox.filtersets import NetBoxModelFilterSet
 from tenancy.filtersets import TenancyFilterSet
+from utilities.filters import MultiValueCharFilter
 
-from netbox_dns.models import View, Zone, Record, RecordTypeChoices
+from netbox_dns.models import View, Zone, Record, RecordTypeChoices, RecordStatusChoices
 
 
 class RecordFilter(TenancyFilterSet, NetBoxModelFilterSet):
-    """Filter capabilities for Record instances."""
-
+    fqdn = MultiValueCharFilter(
+        method="filter_fqdn",
+    )
     type = django_filters.MultipleChoiceFilter(
         choices=RecordTypeChoices,
         null_value=None,
     )
+    status = django_filters.MultipleChoiceFilter(
+        choices=RecordStatusChoices,
+    )
     zone_id = django_filters.ModelMultipleChoiceFilter(
         queryset=Zone.objects.all(),
         label="Parent Zone ID",
@@ -39,10 +44,31 @@ class RecordFilter(TenancyFilterSet, NetBoxModelFilterSet):
 
     class Meta:
         model = Record
-        fields = ("id", "type", "name", "value", "status", "zone", "managed", "tenant")
+        fields = (
+            "id",
+            "type",
+            "name",
+            "fqdn",
+            "value",
+            "status",
+            "zone",
+            "managed",
+            "tenant",
+        )
+
+    def filter_fqdn(self, queryset, name, value):
+        if not value:
+            return queryset
+
+        fqdns = []
+        for fqdn in value:
+            if not fqdn.endswith("."):
+                fqdn = fqdn + "."
+            fqdns.append(fqdn)
+
+        return queryset.filter(fqdn__in=fqdns)
 
     def search(self, queryset, name, value):
-        """Perform the filtered search."""
         if not value.strip():
             return queryset
         qs_filter = (

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/filters/zone.py

@@ -8,7 +8,7 @@ from netbox_dns.models import View, Zone, ZoneStatusChoices, Registrar, Contact
 
 
 class ZoneFilter(TenancyFilterSet, NetBoxModelFilterSet):
-    status = django_filters.ChoiceFilter(
+    status = django_filters.MultipleChoiceFilter(
         choices=ZoneStatusChoices,
     )
     view_id = django_filters.ModelMultipleChoiceFilter(

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/forms/record.py

@@ -79,24 +79,31 @@ class RecordFilterForm(TenancyFilterForm, NetBoxModelFilterSetForm):
     model = Record
     fieldsets = (
         (None, ("q", "filter_id", "tag")),
-        ("Attributes", ("view_id", "zone_id", "name", "type", "value", "status")),
+        (
+            "Attributes",
+            ("view_id", "zone_id", "name", "fqdn", "type", "value", "status"),
+        ),
         ("Tenant", ("tenant_group_id", "tenant_id")),
     )
 
     type = forms.MultipleChoiceField(
-        choices=add_blank_choice(RecordTypeChoices),
+        choices=RecordTypeChoices,
         required=False,
     )
     name = forms.CharField(
         required=False,
         label="Name",
     )
+    fqdn = forms.CharField(
+        required=False,
+        label="FQDN",
+    )
     value = forms.CharField(
         required=False,
         label="Value",
     )
-    status = forms.ChoiceField(
-        choices=add_blank_choice(RecordStatusChoices),
+    status = forms.MultipleChoiceField(
+        choices=RecordStatusChoices,
         required=False,
     )
     zone_id = DynamicModelMultipleChoiceField(

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/forms/registrar.py

@@ -83,6 +83,13 @@ class RegistrarImportForm(NetBoxModelImportForm):
 class RegistrarBulkEditForm(NetBoxModelBulkEditForm):
     model = Registrar
 
+    iana_id = forms.IntegerField(
+        required=False,
+        label="IANA ID",
+    )
+    address = forms.CharField(
+        required=False,
+    )
     referral_url = forms.CharField(
         required=False,
         label="Referral URL",
@@ -104,6 +111,7 @@ class RegistrarBulkEditForm(NetBoxModelBulkEditForm):
         (
             None,
             (
+                "iana_id",
                 "address",
                 "referral_url",
                 "whois_server",
@@ -114,6 +122,7 @@ class RegistrarBulkEditForm(NetBoxModelBulkEditForm):
     )
 
     nullable_fields = (
+        "iana_id",
         "address",
         "referral_url",
         "whois_server",

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/forms/zone.py

@@ -258,8 +258,8 @@ class ZoneFilterForm(TenancyFilterForm, NetBoxModelFilterSetForm):
         required=False,
         label="View",
     )
-    status = forms.ChoiceField(
-        choices=add_blank_choice(ZoneStatusChoices),
+    status = forms.MultipleChoiceField(
+        choices=ZoneStatusChoices,
         required=False,
     )
     name = forms.CharField(
@@ -595,8 +595,9 @@ class ZoneBulkEditForm(NetBoxModelBulkEditForm):
         help_text="IPv4 network prefix with a mask length of at least 25 bits",
         validators=[validate_ipv4, validate_prefix, validate_rfc2317],
     )
-    rfc2317_parent_managed = forms.BooleanField(
+    rfc2317_parent_managed = forms.NullBooleanField(
         required=False,
+        widget=BulkEditNullBooleanSelect(),
         label="RFC2317 Parent Managed",
         help_text="IPv4 reverse zone for deletgating the RFC2317 PTR records is managed in NetBox DNS",
     )

netbox_plugin_dns-0.22.8/netbox_dns/migrations/0029_record_fqdn.py

@@ -0,0 +1,30 @@
+from dns import name as dns_name
+
+from django.db import migrations, models
+
+
+def update_record_fqdn(apps, schema_editor):
+    Record = apps.get_model("netbox_dns", "Record")
+
+    for record in Record.objects.filter(fqdn__isnull=True):
+        zone = dns_name.from_text(record.zone.name, origin=dns_name.root)
+        fqdn = dns_name.from_text(record.name, origin=zone)
+
+        record.fqdn = fqdn.to_text()
+
+        record.save()
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("netbox_dns", "0028_rfc2317_fields"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="record",
+            name="fqdn",
+            field=models.CharField(default=None, max_length=255, null=True, blank=True),
+        ),
+        migrations.RunPython(update_record_fqdn),
+    ]

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/models/record.py

@@ -114,17 +114,23 @@ class Record(NetBoxModel):
         Q(Q(type=RecordTypeChoices.A) | Q(type=RecordTypeChoices.AAAA)),
     )
 
+    name = models.CharField(
+        max_length=255,
+    )
     zone = models.ForeignKey(
         "Zone",
         on_delete=models.CASCADE,
     )
+    fqdn = models.CharField(
+        max_length=255,
+        null=True,
+        blank=True,
+        default=None,
+    )
     type = models.CharField(
         choices=RecordTypeChoices,
         max_length=10,
     )
-    name = models.CharField(
-        max_length=255,
-    )
     value = models.CharField(
         max_length=65535,
     )
@@ -209,14 +215,7 @@ class Record(NetBoxModel):
 
     def __str__(self):
         try:
-            name = (
-                dns_name.from_text(
-                    str(self.name),
-                    origin=dns_name.from_text(self.zone.name, origin=None),
-                )
-                .relativize(dns_name.root)
-                .to_unicode()
-            )
+            name = dns_name.from_text(self.fqdn).relativize(dns_name.root).to_unicode()
         except dns_name.IDNAException:
             name = self.name
         except dns_name.LabelTooLong as exc:
@@ -235,11 +234,14 @@ class Record(NetBoxModel):
         return reverse("plugins:netbox_dns:record", kwargs={"pk": self.id})
 
     @property
-    def fqdn(self):
+    def value_fqdn(self):
+        if self.type != RecordTypeChoices.CNAME:
+            return None
+
         zone = dns_name.from_text(self.zone.name)
-        name = dns_name.from_text(self.name, origin=zone)
+        value_fqdn = dns_name.from_text(self.value, origin=zone)
 
-        return name.to_text()
+        return value_fqdn.to_text()
 
     @property
     def address_from_name(self):
@@ -383,6 +385,22 @@ class Record(NetBoxModel):
 
         self.ptr_record = ptr_record
 
+    def remove_from_rfc2317_cname_record(self, save_zone_serial=True):
+        if self.rfc2317_cname_record.pk:
+            rfc2317_ptr_records = self.rfc2317_cname_record.rfc2317_ptr_records.exclude(
+                pk=self.pk
+            )
+
+            if rfc2317_ptr_records:
+                self.rfc2317_cname_record.ttl = rfc2317_ptr_records.aggregate(
+                    Min("ttl")
+                ).get("ttl__min")
+                self.rfc2317_cname_record.save(
+                    update_fields=["ttl"], save_zone_serial=save_zone_serial
+                )
+            else:
+                self.rfc2317_cname_record.delete()
+
     def update_rfc2317_cname_record(self, save_zone_serial=True):
         if self.zone.rfc2317_parent_managed:
             cname_name = dns_name.from_text(
@@ -390,48 +408,56 @@ class Record(NetBoxModel):
             ).relativize(dns_name.from_text(self.zone.rfc2317_parent_zone.name))
 
             if self.rfc2317_cname_record is not None:
-                self.rfc2317_cname_record.name = cname_name
-                self.rfc2317_cname_record.zone = self.zone.rfc2317_parent_zone
-                self.rfc2317_cname_record.value = self.fqdn
-                self.rfc2317_cname_record.ttl = min_ttl(
-                    self.rfc2317_cname_record.rfc2317_ptr_records.exclude(pk=self.pk)
+                if self.rfc2317_cname_record.name == cname_name.to_text():
+                    self.rfc2317_cname_record.zone = self.zone.rfc2317_parent_zone
+                    self.rfc2317_cname_record.value = self.fqdn
+                    self.rfc2317_cname_record.ttl = min_ttl(
+                        self.rfc2317_cname_record.rfc2317_ptr_records.exclude(
+                            pk=self.pk
+                        )
+                        .aggregate(Min("ttl"))
+                        .get("ttl__min"),
+                        self.ttl,
+                    )
+                    self.rfc2317_cname_record.save(save_zone_serial=save_zone_serial)
+
+                    return
+                else:
+                    self.remove_from_rfc2317_cname_record(
+                        save_zone_serial=save_zone_serial
+                    )
+
+            rfc2317_cname_record = Record.objects.filter(
+                name=cname_name,
+                type=RecordTypeChoices.CNAME,
+                zone=self.zone.rfc2317_parent_zone,
+                managed=True,
+                value=self.fqdn,
+            ).first()
+
+            if rfc2317_cname_record is not None:
+                rfc2317_cname_record.ttl = min_ttl(
+                    rfc2317_cname_record.rfc2317_ptr_records.exclude(pk=self.pk)
                     .aggregate(Min("ttl"))
                     .get("ttl__min"),
                     self.ttl,
                 )
-                self.rfc2317_cname_record.save(save_zone_serial=save_zone_serial)
+                rfc2317_cname_record.save(
+                    update_fields=["ttl"], save_zone_serial=save_zone_serial
+                )
+
             else:
-                rfc2317_cname_record = Record.objects.filter(
+                rfc2317_cname_record = Record(
                     name=cname_name,
                     type=RecordTypeChoices.CNAME,
                     zone=self.zone.rfc2317_parent_zone,
                     managed=True,
                     value=self.fqdn,
-                ).first()
-
-                if rfc2317_cname_record is not None:
-                    rfc2317_cname_record.ttl = min_ttl(
-                        rfc2317_cname_record.rfc2317_ptr_records.exclude(pk=self.pk)
-                        .aggregate(Min("ttl"))
-                        .get("ttl__min"),
-                        self.ttl,
-                    )
-                    rfc2317_cname_record.save(
-                        update_fields=["ttl"], save_zone_serial=save_zone_serial
-                    )
-
-                else:
-                    rfc2317_cname_record = Record(
-                        name=cname_name,
-                        type=RecordTypeChoices.CNAME,
-                        zone=self.zone.rfc2317_parent_zone,
-                        managed=True,
-                        value=self.fqdn,
-                        ttl=self.ttl,
-                    )
-                    rfc2317_cname_record.save(save_zone_serial=save_zone_serial)
+                    ttl=self.ttl,
+                )
+                rfc2317_cname_record.save(save_zone_serial=save_zone_serial)
 
-                self.rfc2317_cname_record = rfc2317_cname_record
+            self.rfc2317_cname_record = rfc2317_cname_record
 
         else:
             if self.rfc2317_cname_record is not None:
@@ -448,6 +474,7 @@ class Record(NetBoxModel):
             name.to_unicode()
 
             self.name = name.relativize(zone).to_text()
+            self.fqdn = fqdn.to_text()
 
         except dns.exception.DNSException as exc:
             raise ValidationError(
@@ -715,20 +742,7 @@ class Record(NetBoxModel):
 
     def delete(self, *args, save_zone_serial=True, **kwargs):
         if self.rfc2317_cname_record:
-            if self.rfc2317_cname_record.pk:
-                if self.rfc2317_cname_record.rfc2317_ptr_records.count() == 1:
-                    self.rfc2317_cname_record.delete()
-                else:
-                    self.rfc2317_cname_record.ttl = (
-                        self.rfc2317_cname_record.rfc2317_ptr_records.exclude(
-                            pk=self.pk
-                        )
-                        .aggregate(Min("ttl"))
-                        .get("ttl__min")
-                    )
-                    self.rfc2317_cname_record.save(
-                        update_fields=["ttl"], save_zone_serial=save_zone_serial
-                    )
+            self.remove_from_rfc2317_cname_record(save_zone_serial=save_zone_serial)
 
         if self.ptr_record:
             self.ptr_record.delete()

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/models/zone.py

@@ -738,6 +738,15 @@ class Zone(NetBoxModel):
                     ip.dns_name = f'{ip.custom_field_data["ipaddress_dns_record_name"]}.{self.name}'
                     ip.save(update_fields=["dns_name"])
 
+        if name_changed:
+            for _record in self.record_set.all():
+                _record.save(
+                    update_fields=["fqdn"],
+                    save_zone_serial=False,
+                    update_rrset_ttl=False,
+                    update_rfc2317_cname=False,
+                )
+
         self.save_soa_serial()
         self.update_soa_record()
 

{netbox_plugin_dns-0.22.5 → netbox_plugin_dns-0.22.8}/netbox_dns/templates/netbox_dns/record.html

@@ -112,12 +112,6 @@
                         <td><a href="{% url 'ipam:ipaddress' pk=object.ipam_ip_address.pk %}">{{ object.ipam_ip_address }}</td>
                     </tr>
                     {% endif %}
-                    {% if object.rfc2317_cname_record %}
-                    <tr>
-                        <th scope="row">RFC2317 CNAME Record</th>
-                        <td><a href="{% url 'plugins:netbox_dns:record' pk=object.rfc2317_cname_record.pk %}">{{ object.rfc2317_cname_record }}</td>
-                    </tr>
-                    {% endif %}
                     <tr>
                         <th scope="row">Status</th>
                         <td>{% badge object.get_status_display bg_color=object.get_status_color %}</td>
@@ -131,23 +125,13 @@
                 </table>
             </div>
         </div>
-        {% if object.rfc2317_ptr_records.all|length %}
+        {% if cname_target_table and cname_target_table.rows|length > 0 %}
         <div class="card">
-            <h5 class="card-header">RFC2317 Targets</h5>
-            <div class="card-body">
-                <table class="table table-hover attr-table">
-                    <tr>
-                        <th>Address Record</th>
-                        <th>PTR Record</th>
-                    </tr>
-                    {% for record in object.rfc2317_ptr_records.all %}
-                    <tr>
-                        <td><a href="{% url 'plugins:netbox_dns:record' pk=record.address_record.pk %}">{{ record.address_record }}</td>
-                        <td><a href="{% url 'plugins:netbox_dns:record' pk=record.pk %}">{{ record }}</td>
-                    </td>
-                    {% endfor %}
-                </table>
-            </div>
+            {% include 'inc/panel_table.html' with table=cname_target_table heading='CNAME Targets' %}
+        </div>
+        {% elif cname_table and cname_table.rows|length > 0 %}
+        <div class="card">
+            {% include 'inc/panel_table.html' with table=cname_table heading='CNAMEs' %}
         </div>
         {% endif %}
         {% if not object.managed %}

netbox_plugin_dns-0.22.8/netbox_dns/views/record.py

@@ -0,0 +1,159 @@
+from dns import name as dns_name
+
+from django.db.models import Q
+from django.db.models.functions import Length
+
+from netbox.views import generic
+
+from netbox_dns.filters import RecordFilter
+from netbox_dns.forms import (
+    RecordImportForm,
+    RecordFilterForm,
+    RecordForm,
+    RecordBulkEditForm,
+)
+from netbox_dns.models import Record, RecordTypeChoices, Zone
+from netbox_dns.tables import RecordTable, ManagedRecordTable, RelatedRecordTable
+from netbox_dns.utilities import value_to_unicode
+
+
+class RecordListView(generic.ObjectListView):
+    queryset = Record.objects.filter(managed=False).prefetch_related(
+        "zone", "ptr_record"
+    )
+    filterset = RecordFilter
+    filterset_form = RecordFilterForm
+    table = RecordTable
+
+
+class ManagedRecordListView(generic.ObjectListView):
+    queryset = Record.objects.filter(managed=True).prefetch_related(
+        "zone", "address_record"
+    )
+    filterset = RecordFilter
+    filterset_form = RecordFilterForm
+    table = ManagedRecordTable
+    actions = ("export",)
+    template_name = "netbox_dns/record/managed.html"
+
+
+class RecordView(generic.ObjectView):
+    queryset = Record.objects.all().prefetch_related("zone", "ptr_record")
+
+    def get_value_records(self, instance):
+        value_fqdn = dns_name.from_text(instance.value_fqdn)
+        value_zone_names = [
+            value_fqdn.split(length)[1].to_text().rstrip(".")
+            for length in range(2, len(value_fqdn) + 1)
+        ]
+
+        value_zone = (
+            Zone.objects.filter(instance.zone.view_filter, name__in=value_zone_names)
+            .order_by(Length("name").desc())
+            .first()
+        )
+        if not value_zone:
+            return None
+
+        value_name = value_fqdn.relativize(dns_name.from_text(value_zone.name))
+        cname_targets = Record.objects.filter(zone=value_zone, name=value_name)
+
+        if cname_targets:
+            return RelatedRecordTable(
+                data=cname_targets,
+            )
+
+        return None
+
+    def get_cname_records(self, instance):
+        view_filter = (
+            Q(zone__view__isnull=True)
+            if instance.zone.view is None
+            else Q(zone__view=instance.zone.view)
+        )
+        cname_records = set(
+            Record.objects.filter(
+                view_filter, value=instance.fqdn, type=RecordTypeChoices.CNAME
+            )
+        )
+
+        fqdn = dns_name.from_text(instance.fqdn)
+        parent_zone_names = [
+            fqdn.split(length)[1].to_text().rstrip(".")
+            for length in range(1, len(fqdn) + 1)
+        ]
+
+        parent_zones = Zone.objects.filter(
+            instance.zone.view_filter, name__in=parent_zone_names
+        )
+
+        for parent_zone in parent_zones:
+            parent_cname_records = Record.objects.filter(
+                view_filter, type=RecordTypeChoices.CNAME, zone=parent_zone
+            )
+            cname_records = cname_records.union(
+                set(
+                    record
+                    for record in parent_cname_records
+                    if record.value_fqdn == instance.fqdn
+                )
+            )
+
+        if cname_records:
+            return RelatedRecordTable(
+                data=cname_records,
+            )
+
+        return None
+
+    def get_extra_context(self, request, instance):
+        context = {}
+
+        name = dns_name.from_text(instance.name, origin=None)
+        if name.to_text() != name.to_unicode():
+            context["unicode_name"] = name.to_unicode()
+
+        unicode_value = value_to_unicode(instance.value)
+        if instance.value != unicode_value:
+            context["unicode_value"] = unicode_value
+
+        if instance.type == RecordTypeChoices.CNAME:
+            context["cname_target_table"] = self.get_value_records(instance)
+        else:
+            context["cname_table"] = self.get_cname_records(instance)
+
+        return context
+
+
+class RecordEditView(generic.ObjectEditView):
+    queryset = Record.objects.filter(managed=False).prefetch_related(
+        "zone", "ptr_record"
+    )
+    form = RecordForm
+    default_return_url = "plugins:netbox_dns:record_list"
+
+
+class RecordDeleteView(generic.ObjectDeleteView):
+    queryset = Record.objects.filter(managed=False)
+    default_return_url = "plugins:netbox_dns:record_list"
+
+
+class RecordBulkImportView(generic.BulkImportView):
+    queryset = Record.objects.filter(managed=False).prefetch_related(
+        "zone", "ptr_record"
+    )
+    model_form = RecordImportForm
+    table = RecordTable
+    default_return_url = "plugins:netbox_dns:record_list"
+
+
+class RecordBulkEditView(generic.BulkEditView):
+    queryset = Record.objects.filter(managed=False).prefetch_related("zone")
+    filterset = RecordFilter
+    table = RecordTable
+    form = RecordBulkEditForm
+
+
+class RecordBulkDeleteView(generic.BulkDeleteView):
+    queryset = Record.objects.filter(managed=False)
+    table = RecordTable