Skip to content

Fix docker port #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ScriptAutomate
Copy link

Port 5000 should be 5005 due to the configurations set in this repo, which can easily be seen with grep -R 5005 from the repo root:

# grep -R 5005
gunicorn-cfg.py:bind = '0.0.0.0:5005'
docker-compose.yml:      - "5005:5005"
nginx/appseed-app.conf:        proxy_pass http://localhost:5005/;
app/base/static/assets/scss/material-dashboard/variables/_colors.scss:$pink-a400: #f50057 !default;
Dockerfile:EXPOSE 5005

The following all reference port 5005:

  • docker-compose
  • Dockerfile
  • gunicorn
  • nginx/appseed-app.conf

Listing port 5000 in the readme is incorrect, it should be 5005.

Attempting a similar grep -R 5000 shows:

# grep -R 5000
README.md:$ # --port=5000    - specify the app port (default 5000)  
README.md:$ flask run --host=0.0.0.0 --port=5000
README.md:$ # Access the dashboard in browser: http://127.0.0.1:5000/
README.md:Visit `http://localhost:5000` in your browser. The app should be up & running.
app/base/static/assets/scss/material-dashboard/variables/_colors.scss:$red-a700: #d50000 !default;
app/base/static/assets/js/plugins/bootstrap-notify.js:    delay: 5000,

Those are all unrelated to a port listening or exposed at port 5000, other than the incorrect README directions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant