Merge pull request #793 from readthedocs/davidfischer/20-tags-max

20 keywords maximum in API
readthedocs · Oct 9, 2023 · b3df592 · b3df592
2 parents 37e4c09 + 3e004cf
commit b3df592
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/adserver/api/serializers.py b/adserver/api/serializers.py
@@ -99,9 +99,13 @@ def validate_publisher(self, publisher_slug):
         return publisher
 
     def validate_keywords(self, keywords):
+        # API users may send more than this, but keywords beyond this are ignored
+        MAX_KEYWORDS = 20
+
         if keywords:
-            # Lowercase all the keywords and strip surrounding whitespace
-            keywords = [k.lower().strip() for k in keywords if k.strip()]
+            # Lowercase all the keywords and strip surrounding whitespace.
+            # Also, only take the first 20.
+            keywords = [k.lower().strip() for k in keywords if k.strip()][:MAX_KEYWORDS]
 
         return keywords