fix(api): make sure RRset.touched can't be NULL

api/desecapi/migrations/0015_rrset_touched_auto_now.py

@@ -0,0 +1,24 @@
+# Generated by Django 3.0.6 on 2020-05-22 12:10
+
+from django.db import migrations, models
+
+
+def migrate_data(apps, schema_editor):
+    RRset = apps.get_model('desecapi', 'RRset')
+    RRset.objects.filter(touched__isnull=True).update(touched=models.F('created'))
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('desecapi', '0014_rrset_touched'),
+    ]
+
+    operations = [
+        migrations.RunPython(migrate_data, migrations.RunPython.noop),
+        migrations.AlterField(
+            model_name='rrset',
+            name='touched',
+            field=models.DateTimeField(auto_now=True),
+        ),
+    ]

api/desecapi/models.py

@@ -6,7 +6,6 @@ import secrets
 import string
 import time
 import uuid
-from base64 import urlsafe_b64encode
 from datetime import timedelta
 from hashlib import sha256
 
@@ -380,7 +379,7 @@ class RRsetManager(Manager):
 class RRset(ExportModelOperationsMixin('RRset'), models.Model):
     id = models.UUIDField(primary_key=True, default=uuid.uuid4, editable=False)
     created = models.DateTimeField(auto_now_add=True)
-    touched = models.DateTimeField(null=True)
+    touched = models.DateTimeField(auto_now=True)
     domain = models.ForeignKey(Domain, on_delete=models.CASCADE)
     subname = models.CharField(
         max_length=178,
@@ -425,7 +424,6 @@ class RRset(ExportModelOperationsMixin('RRset'), models.Model):
         return self.construct_name(self.subname, self.domain.name)
 
     def save(self, *args, **kwargs):
-        self.touched = timezone.now()
         self.full_clean(validate_unique=False)
         super().save(*args, **kwargs)