From 21a69c75153fd0e5773311cf7c9b6b3abf2f2e1a Mon Sep 17 00:00:00 2001
From: Eden Kirin <eden.kirin@intis.hr>
Date: Fri, 21 Apr 2023 12:03:42 +0200
Subject: [PATCH] API port through env variable

---
 .docker/run.sh | 6 ++++--
 Dockerfile     | 4 +++-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/.docker/run.sh b/.docker/run.sh
index b1b2c0c..fe4bcc8 100755
--- a/.docker/run.sh
+++ b/.docker/run.sh
@@ -1,9 +1,11 @@
 #!/bin/sh
 
-echo "Starting FairHopper game server"
+PORT=${FAIRHOPPER_API_PORT=8010}
+
+echo "Starting FairHopper game server on port ${PORT}"
 
 uvicorn \
     main:app \
     --host 0.0.0.0 \
-    --port ${FAIRHOPPER_API_PORT} \
+    --port ${PORT} \
     --workers=1
diff --git a/Dockerfile b/Dockerfile
index 9f9cd59..dbd7f58 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,5 +1,6 @@
 FROM python:3.10.11-alpine3.17
 
+# take arguments
 ARG INTERNAL_API_PORT
 ARG INTERNAL_WS_PORT
 
@@ -14,7 +15,7 @@ COPY poetry.lock .
 
 # create virtual environment
 RUN python -m venv /venv
-# set python thingies and activate virtual environment
+# set python thingies, set environment variables and activate virtual environment
 ENV \
     PYTHONDONTWRITEBYTECODE=1 \
     PYTHONUNBUFFERED=1 \
@@ -28,6 +29,7 @@ RUN \
     # install python libs
     pip install -r requirements.txt --no-cache-dir --prefer-binary
 
+# copy all relevant files
 COPY ./.docker/* ./
 COPY ./hopper ./hopper
 COPY ./main.py .