fix(collections): make pagination optional (#35)

if offset-pagination is disabled:
- do not limit the list of collections returned
- do not generate the pagination links

Author: alambare

stac_fastapi/eodag/app.py

@@ -190,7 +190,6 @@ async def lifespan(app: FastAPI) -> AsyncGenerator[None, None]:
 search_post_model = create_post_request_model(search_extensions)
 search_get_model = create_get_request_model(search_extensions)
 
-
 collections_model = create_request_model(
     "CollectionsRequest",
     base_model=EmptyRequest,

stac_fastapi/eodag/core.py

@@ -244,7 +244,7 @@ async def all_collections(
 
         next_link: Optional[dict[str, Any]] = None
         prev_link: Optional[dict[str, Any]] = None
-        first_link: dict[str, Any] = {"body": {"limit": limit, "offset": 0}}
+        first_link: Optional[dict[str, Any]] = None
 
         # get provider filter
         provider = None
@@ -285,19 +285,8 @@ async def all_collections(
                 ).intersection(bbox_geom)
             ]
 
-        limit = limit if limit is not None else 10
-        offset = offset if offset is not None else 0
-
-        paged_collections = collections[offset : offset + limit]
-
         total = len(collections)
 
-        if offset + limit < total:
-            next_link = {"body": {"limit": limit, "offset": offset + limit}}
-
-        if offset > 0:
-            prev_link = {"body": {"limit": limit, "offset": max(0, offset - limit)}}
-
         links = [
             {
                 "rel": Relations.root,
@@ -306,6 +295,21 @@ async def all_collections(
                 "title": get_settings().stac_fastapi_title,
             },
         ]
+
+        if self.extension_is_enabled("OffsetPaginationExtension"):
+            limit = limit if limit is not None else 10
+            offset = offset if offset is not None else 0
+
+            collections = collections[offset : offset + limit]
+
+            if offset + limit < total:
+                next_link = {"body": {"limit": limit, "offset": offset + limit}}
+
+            if offset > 0:
+                prev_link = {"body": {"limit": limit, "offset": max(0, offset - limit)}}
+
+            first_link = {"body": {"limit": limit, "offset": 0}}
+
         extension_names = [type(ext).__name__ for ext in self.extensions]
 
         paging_links = CollectionSearchPagingLinks(
@@ -315,10 +319,10 @@ async def all_collections(
         links.extend(paging_links)
 
         return Collections(
-            collections=paged_collections or [],
+            collections=collections,
             links=links,
             numberMatched=total,
-            numberReturned=len(paged_collections),
+            numberReturned=len(collections),
         )
 
     async def get_collection(self, collection_id: str, request: Request, **kwargs: Any) -> Collection: