Merge branch 'fix/build-n-run-issues' of jonas/DynDns into master

- using ./ instead of ${PWD} is more universal and doesn't depend on the corresponding env variable to be set in the environment
- when mounting /app/lastIp, docker wants this to be a directory and not a file for it to work

Signed-off-by: Jonas Sulzer <jonas@violoncello.ch>

docker-compose.yml

@@ -4,9 +4,9 @@ services:
         build: '.'
         container_name: dyndns
         volumes:
-            - ${PWD}/src:/app/src
-            - ${PWD}/log:/app/log
-            - ${PWD}/lastIp:/app/lastIp
+            - ./src:/app/src
+            - ./log:/app/log
+            - ./lastIp:/app/lastIp
         env_file:
             - app.env
 

src/main.py

@@ -18,7 +18,7 @@ logging.config.fileConfig(fname=loggerConfigPath, defaults={'logfilename': f'{ho
 
 logger = logging.getLogger(__name__)
 
-ipFile = f'{home}/lastIp'
+ipFile = f'{home}/lastIp/ip'
 timeout = 60
 
 if __name__ == "__main__":