Skip to content

Commit

Permalink
Merge pull request #100 from mbentley/update-readme-ports
Browse files Browse the repository at this point in the history
Updated README with a note about the port mapping issue; fixes #99
  • Loading branch information
mbentley authored Apr 17, 2021
2 parents 0c94e75 + c1cddb3 commit fb02b11
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,8 @@ To run this Docker image and keep persistent data in named volumes:

### Using port mapping

*Note*: If you want to change the controller ports from the default mappings, you *absolutely must* update the port binding inside the container via the environment variables. The ports exposed must match what is inside the container. The Omada Controller software expects that the ports are the same inside the container and outside and will load a blank page if that is not done. See [#99](https://github.com/mbentley/docker-omada-controller/issues/99#issuecomment-821243857) for details and and example of the behavior.

```
docker run -d \
--name omada-controller \
Expand Down

0 comments on commit fb02b11

Please sign in to comment.