Missing expose statements in Dockerfile

ttimasdf / docker-cve-search

Dockerized cve-search as a web app

MIT License

16 stars 15 forks source link

Closed electricworry closed 6 years ago

electricworry commented 6 years ago

I'm relatively new to Docker, but shouldn't the Dockerfile have the following lines?

EXPOSE 27017
EXPOSE 5000

ttimasdf commented 6 years ago

27017 is already exposed by mongo image. 5000 is fixed by commit above. Pull the latest and see if that works? 😄

electricworry commented 6 years ago

Sorry about that. Learning docker as I go, but that makes sense that it would inherit from the mongo image. Thanks for the reply!