From dfb676a29b9a8861970304e1cb580a1deee508eb Mon Sep 17 00:00:00 2001 From: Bindu Wavell Date: Fri, 15 Dec 2017 12:40:55 -0700 Subject: [PATCH] Don't try to port map to 80 as it is < 1024 (#161) In the docker example we were proposing that folks map port 80 from the docker container to port 80 on localhost. Unless the user is running as root, they likely won't be able to bind to any port < 1024. As such, I am proposing we bind to the local port 8888. Could change it to 3000 to be in-line with the `docker compose` example; but to me the use cases are different so I like having different ports... --- docs/docker.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/docker.md b/docs/docker.md index ac19fb4fe..fcddc4aca 100644 --- a/docs/docker.md +++ b/docs/docker.md @@ -35,10 +35,10 @@ docker image build -t content-app . To run the image locally, you can use the following command: ```sh -docker container run -p 80:80 --rm content-app +docker container run -p 8888:80 --rm content-app ``` -Navigate to "http://localhost" to access the running application. +Navigate to "http://localhost:8888" to access the running application. ## Docker Compose file