Remove redundant styles

.vscode/settings.json

@@ -35,6 +35,7 @@
     "Keanu",
     "Kombucha",
     "LANCZOS",
+    "maga",
     "Memegen",
     "Minecraft",
     "myapikey",

app/tests/test_views_memes.py

@@ -36,6 +36,18 @@ def it_returns_an_image_url(expect, client, as_json):
                 "url": "http://localhost:5000/images/iw/foo/bar.png"
             }
 
+        def it_removes_redundant_styles(expect, client):
+            data = {
+                "template_id": "iw",
+                "text_lines[]": ["foo", "bar"],
+                "style[]": [" ", "test", "default"],
+            }
+            request, response = client.post("/images", data=data)
+            expect(response.status) == 201
+            expect(response.json) == {
+                "url": "http://localhost:5000/images/iw/foo/bar.png?style=default,test"
+            }
+
         def it_returns_gif_when_animated(expect, client):
             data = {
                 "template_id": "iw",

app/views/templates.py

@@ -116,23 +116,31 @@ async def generate_url(
     if request.form:
         payload = dict(request.form)
         for key in list(payload.keys()):
-            if "lines" not in key:
+            if "lines" not in key and "style" not in key:
                 payload[key] = payload.pop(key)[0]
     else:
         payload = request.json or {}
     with suppress(KeyError):
         payload["text_lines"] = payload.pop("text_lines[]")
+    with suppress(KeyError):
+        payload["style"] = payload.pop("style[]")
 
     if template_id_required:
         try:
             template_id = payload["template_id"]
         except KeyError:
             return response.json({"error": '"template_id" is required'}, status=400)
 
+    from sanic.log import logger
+
+    logger.critical(payload)
+
     text_lines = utils.urls.arg(payload, [], "text_lines")
     style: str = utils.urls.arg(payload, "", "style", "overlay", "alt")
     if isinstance(style, list):
-        style = ",".join(style)
+        style = ",".join([(s.strip() or "default") for s in style])
+    while style.endswith(",default"):
+        style = style.removesuffix(",default")
     background = utils.urls.arg(payload, "", "background", "image_url")
     extension = utils.urls.arg(payload, "", "extension")