Skip to content

Commit

Permalink
Remarks from review
Browse files Browse the repository at this point in the history
  • Loading branch information
ernstdevreedeKNMI committed Mar 18, 2024
1 parent da337c4 commit 27f38f1
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
6 changes: 1 addition & 5 deletions Docker/docker-compose-generate-env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ ADAGUC_AUTOWMS_DIR=${HOME}/adaguc-docker/adaguc-autowms
ADAGUC_DATASET_DIR=${HOME}/adaguc-docker/adaguc-datasets
ADAGUC_NUMPARALLELPROCESSES=4

usage() { echo "Usage: $0 -p <port number> -e <external adress> -a <autowmsdir> -d <dataset dir> -f <datadir> -t <num parallel processes> -r <redis_portnumber>" 1>&2; exit 1; }
usage() { echo "Usage: $0 -p <port number> -e <external adress> -a <autowmsdir> -d <dataset dir> -f <datadir> -t <num parallel processes>" 1>&2; exit 1; }


while getopts ":e:p:h:a:d:f:t:" o; do
Expand All @@ -27,9 +27,6 @@ while getopts ":e:p:h:a:d:f:t:" o; do
f)
ADAGUC_DATA_DIR=${OPTARG}
;;
r)
REDIS_PORT=${OPTARG}
;;
t)
ADAGUC_NUMPARALLELPROCESSES=${OPTARG}
;;
Expand Down Expand Up @@ -62,7 +59,6 @@ echo "ADAGUC_DATASET_DIR=${ADAGUC_DATASET_DIR}" >> .env
echo "ADAGUC_NUMPARALLELPROCESSES=${ADAGUC_NUMPARALLELPROCESSES}" >> .env
echo "ADAGUC_PORT=${ADAGUC_PORT}" >> .env
echo "EXTERNALADDRESS=${EXTERNALADDRESS}" >> .env
echo "REDIS_PORT=${REDIS_PORT} >>.env
echo "############### env file ###############"
cat .env
echo "############### env file ###############"
Expand Down
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ Version 2.6.1 2021-11-09
- Moving to a python wrapper to run adaguc-server. For development with flask, for production with gunicorn.

**Version 2.5.15 2021-09-22**
- The /data folder can now completely2.20.2 be mounted from another filesystem, all adaguc internal folders have moved to /adaguc.
- The /data folder can now completely be mounted from another filesystem, all adaguc internal folders have moved to /adaguc.

**Version 2.5.14 2021-09-17**
- WCS GetCoverage with Native parameters now returns data
Expand Down
2 changes: 1 addition & 1 deletion python/python_fastapi_server/routers/caching_middleware.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ async def dispatch(self, request, call_next):

if data:
# Fix Age header
headers["Age"] = f"{age:1d}"
headers["Age"] = f"{age}"
headers["adaguc-cache"] = "hit"
return Response(
content=data,
Expand Down

0 comments on commit 27f38f1

Please sign in to comment.