Merge pull request #2873 from jgmize/max-query-length

Add max_length param to search forms: bug 1286313
mozilla · Jul 13, 2016 · 27b8136 · 27b8136
2 parents 2ad2f0f + ad0b1c4
commit 27b8136
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/kitsune/search/forms.py b/kitsune/search/forms.py
@@ -12,13 +12,14 @@
 from kitsune.wiki.config import CATEGORIES
 
 
+MAX_QUERY_LENGTH = 200
 SEARCH_LANGUAGES = [(k, LOCALES[k].native) for
                     k in settings.SUMO_LANGUAGES]
 
 
 class SimpleSearchForm(forms.Form):
     """Django form to handle the simple search case."""
-    q = forms.CharField(required=True)
+    q = forms.CharField(required=True, max_length=MAX_QUERY_LENGTH)
 
     w = forms.TypedChoiceField(required=False, coerce=int,
                                widget=forms.HiddenInput,
@@ -71,7 +72,7 @@ def clean_products(self):
 class AdvancedSearchForm(forms.Form):
     """Django form for handling display and validation"""
     # Common fields
-    q = forms.CharField(required=False)
+    q = forms.CharField(required=False, max_length=MAX_QUERY_LENGTH)
 
     w = forms.TypedChoiceField(required=False, coerce=int,
                                widget=forms.HiddenInput,