feat: ❇️ enable post_filter for POST query

Previously only enabled in GET query, see issue #208
biothings · Sep 12, 2023 · 83748c2 · 83748c2
1 parent e6916ff
commit 83748c2
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/biothings/web/settings/default.py b/biothings/web/settings/default.py
@@ -119,24 +119,26 @@
     "POST": {"id": {"type": list, "max": 1000, "required": True, "alias": "ids"}},
 }
 QUERY_KWARGS = {
-    "*": COMMON_KWARGS.copy(),
+    "*": {
+        **COMMON_KWARGS.copy(),
+        **{
+            "from": {"type": int, "max": 10000, "alias": "skip"},
+            "sort": {"type": list, "max": 10},
+            "post_filter": {"type": str, "default": None},
+        },
+    },  # for py3.9+, we can just use `|` operator like `COMMON_KWARGS.copy() | {...}`
     "GET": {
         "q": {"type": str, "default": None},
         "aggs": {"type": list, "max": 1000, "alias": "facets"},
-        "post_filter": {"type": str, "default": None},
         "facet_size": {"type": int, "default": 10, "max": 1000},
-        "from": {"type": int, "max": 10000, "alias": "skip"},
         "userquery": {"type": str, "alias": ["userfilter"]},
-        "sort": {"type": list, "max": 10},
         "explain": {"type": bool},
         "fetch_all": {"type": bool},
         "scroll_id": {"type": str},
     },
     "POST": {
         "q": {"type": list, "required": True},
         "scopes": {"type": list, "default": ["_id"], "max": 1000},
-        "from": {"type": int, "max": 10000, "alias": "skip"},
-        "sort": {"type": list, "max": 10},
         "with_total": {"type": bool},
         "analyzer": {"type": str},  # any of built-in analyzer (overrides default index-time analyzer)
         # Ref: https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-analyzers.html