Skip to content
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

Scenario "Salta": too restrictive solution script #11

Open
fduran opened this issue Nov 16, 2023 · 1 comment
Open

Scenario "Salta": too restrictive solution script #11

fduran opened this issue Nov 16, 2023 · 1 comment
Labels
scenario Scenario Issue or Comment

Comments

@fduran
Copy link
Owner

fduran commented Nov 16, 2023

tTe regex for checking the docker published port can be relaxed, as we don/t really care if the node app is not started on :8888, as long as the port is mapped correctly when executing "docker run".

@fduran fduran added the scenario Scenario Issue or Comment label Nov 16, 2023
@fduran
Copy link
Owner Author

fduran commented May 16, 2024

from @erjan it seems to me there is only 1 way it is solved, to do

sudo docker run -d -p 8880:8888 myapp

and actually myapp should be accepted as name, as long as only 1 container is running, but it seems it only accept the container named 'app'.

but the docker file has CMD['node', 'server.js'] - it still does not run as expected.

the only solution that works is if i run docker run -d app node server.js

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

No branches or pull requests

1 participant