Update filter code

    * We do not want to process download urls as a query

Signed-off-by: Jono Yang <jyang@nexb.com>

Author: JonoYang

packagedb/filters.py

@@ -81,10 +81,10 @@ def filter(self, qs, value):
         if value.startswith("pkg:"):
             return qs.for_package_url(value)
 
-        if ":" in value:
+        if "://" not in value and ":" in value:
             return super().filter(qs, value)
 
-        search_fields = ["type", "namespace", "name", "version"]
+        search_fields = ["type", "namespace", "name", "version", "download_url"]
         lookups = Q()
         for field_names in search_fields:
             lookups |= Q(**{f"{field_names}__{self.lookup_expr}": value})

packagedb/tests/test_api.py

@@ -392,6 +392,8 @@ def test_package_api_list_endpoint_search(self):
         assert response.data.get('count') == 1
         response = self.client.get('/api/packages/?search={}'.format('12.35'))
         assert response.data.get('count') == 1
+        response = self.client.get('/api/packages/?search={}'.format('https://dummy.com/dummy'))
+        assert response.data.get('count') == 1
 
     def test_package_api_retrieve_endpoint(self):
         response = self.client.get('/api/packages/{}/'.format(self.package.uuid))