Fix db field overflow when copying a check with a long name

CHANGELOG.md

@@ -18,6 +18,9 @@ All notable changes to this project will be documented in this file.
 - Implement badge mode (up/down vs up/late/down) selector (#282)
 - Add Ping.exitstatus field, store client's reported exit status values (#455)
 
+## Bug Fixes
+- Fix db field overflow when copying a check with a long name
+
 ## v1.17.0 - 2020-10-14
 
 ## Improvements

hc/front/tests/test_copy.py

@@ -41,3 +41,13 @@ class CopyCheckTestCase(BaseTestCase):
         self.client.login(username="[email protected]", password="password")
         r = self.client.post(self.copy_url)
         self.assertEqual(r.status_code, 403)
+
+    def test_it_handles_long_check_name(self):
+        self.check.name = "A" * 100
+        self.check.save()
+
+        self.client.login(username="[email protected]", password="password")
+        self.client.post(self.copy_url)
+
+        q = Check.objects.filter(name="A" * 90 + "... (copy)")
+        self.assertTrue(q.exists())

hc/front/views.py

@@ -623,8 +623,13 @@ def copy(request, code):
     if check.project.num_checks_available() <= 0:
         return HttpResponseBadRequest()
 
+    new_name = check.name + " (copy)"
+    # Make sure we don't exceed the 100 character db field limit:
+    if len(new_name) > 100:
+        new_name = check.name[:90] + "... (copy)"
+
     copied = Check(project=check.project)
-    copied.name = check.name + " (copy)"
+    copied.name = new_name
     copied.desc, copied.tags = check.desc, check.tags
     copied.subject, copied.subject_fail = check.subject, check.subject_fail
     copied.methods = check.methods