Merge pull request #3 from jeffreyvdhondel/master

Updated the README.md

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3-alpine
+FROM frolvlad/alpine-python3
 
 RUN pip3 install docker
 RUN mkdir /hoster

README.md

@@ -19,12 +19,12 @@ Hoster provides by default the entries `<container name>, <hostname>, <container
 
 For example, the following container would be available via DNS as `myname`, `myhostname`, `et54rfgt567` and `myserver.com`:
 
-    docker run -d \
+    docker run --restart=unless-stopped -d \
         --name myname \
         --hostname myhostname \
         --network somenetwork --network-alias "myserver.com" \
         mycontainer
 
-If you need more features like **systemd interation** and **dns forwarding** please check [resolvable](https://hub.docker.com/r/mgood/resolvable/)
+If you need more features like **systemd integration** and **dns forwarding** please check [resolvable](https://hub.docker.com/r/mgood/resolvable/)
 
 Any contribution is, of course, welcome. :)

hoster.py

@@ -62,8 +62,6 @@ def get_container_data(dockerClient, container_id):
     container_hostname = info["Config"]["Hostname"]
     container_name = info["Name"].strip("/")
     container_ip = info["NetworkSettings"]["IPAddress"]
-    if info["Config"]["Domainname"]:
-        container_hostname = container_hostname + "." + info["Config"]["Domainname"]
     
     result = []
 
@@ -106,7 +104,6 @@ def update_hosts_file():
             break;
 
     #remove all the trailing newlines on the line list
-    if lines:
     while lines[-1].strip()=="": lines.pop()
 
     #append all the domain lines