Add host Docker environment variables to Dockerfile and README.md

kamilrybacki · Apr 23, 2024 · b053196 · b053196
1 parent 19bb215
commit b053196
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -30,4 +30,6 @@ WORKDIR /app
 
 USER ppeagent
 
+ENV PPE_AGENT_IS_DOCKERIZED=1
+
 CMD ["python", "serve.py"]
diff --git a/README.md b/README.md
@@ -69,6 +69,7 @@ Below is a list of **environment variables** that control the main settings of t
 
 #### Optional
 
+* `PPE_AGENT_HOST` - Hostname on which the application will listen for incoming requests (default: `'127.0.0.1'` on local deployments, `'0.0.0.0'` in Docker containers)
 * `PPE_AGENT_PORT` - Port on which the application will listen for incoming requests (default: 8000)
 * `PPE_AGENT_CONFIG` - path to the configuration file (default: `''` i.e. reads hardcoded defaults)
 

diff --git a/src/serve.py b/src/serve.py
@@ -63,6 +63,7 @@ def main():
     server = ThreadedPPEServer({
         'app': ppe_agent._app,  # pylint: disable=protected-access
         'port': int(os.getenv('PPE_AGENT_PORT', '8000')),
+        'host': '0.0.0.0' if os.getenv('PPE_AGENT_IS_DOCKERIZED') else os.getenv('PPE_AGENT_HOST', '127.0.0.1'),
         'log_config': ppe_agent._log_config  # pylint: disable=protected-access
     })
     try: