Skip to content

Commit fbfa774

Browse files
hrodmnvincentsarago
andcommitted
apply suggestions from review
Co-authored-by: Vincent Sarago <[email protected]>
1 parent 4d07c6e commit fbfa774

File tree

2 files changed

+20
-15
lines changed

2 files changed

+20
-15
lines changed

docker-compose.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ services:
2121
- DB_MIN_CONN_SIZE=1
2222
- DB_MAX_CONN_SIZE=1
2323
- USE_API_HYDRATE=${USE_API_HYDRATE:-false}
24+
- ENABLED_EXTENSIONS=transaction,bulk_transactions,query,sort,fields,filter,pagination,collection_search
2425
ports:
2526
- "8082:8082"
2627
volumes:

stac_fastapi/pgstac/app.py

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,8 @@
4949

5050
settings = Settings()
5151

52-
# application extensions
53-
application_extensions_map = {
52+
# transaction extensions
53+
transaction_extensions_map = {
5454
"transaction": TransactionExtension(
5555
client=TransactionsClient(),
5656
settings=settings,
@@ -93,27 +93,31 @@
9393
"pagination": TokenPaginationExtension(),
9494
}
9595

96-
known_extensions = {
97-
*application_extensions_map.keys(),
96+
enabled_extensions = {
9897
*search_extensions_map.keys(),
9998
*cs_extensions_map.keys(),
10099
*itm_col_extensions_map.keys(),
101100
"collection_search",
102101
}
103102

104-
disabled_extensions = {"transaction", "bulk_transactions"}
103+
if ext := os.environ.get("ENABLED_EXTENSIONS"):
104+
enabled_extensions = set(ext.split(","))
105105

106-
enabled_extensions = (
107-
os.environ["ENABLED_EXTENSIONS"].split(",")
108-
if "ENABLED_EXTENSIONS" in os.environ
109-
else known_extensions - disabled_extensions
110-
)
106+
application_extensions = []
111107

112-
application_extensions = [
113-
extension
114-
for key, extension in application_extensions_map.items()
115-
if key in enabled_extensions
116-
]
108+
if "transaction" in enabled_extensions:
109+
application_extensions.append(
110+
TransactionExtension(
111+
client=TransactionsClient(),
112+
settings=settings,
113+
response_class=ORJSONResponse,
114+
),
115+
)
116+
117+
if "bulk_transactions" in enabled_extensions:
118+
application_extensions.append(
119+
BulkTransactionExtension(client=BulkTransactionsClient()),
120+
)
117121

118122
# /search models
119123
search_extensions = [

0 commit comments

Comments
 (0)