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

Docker-compose up runs successfully but is inaccessible by browser #435

Open
rstmsn opened this issue Sep 15, 2023 · 3 comments · May be fixed by #445
Open

Docker-compose up runs successfully but is inaccessible by browser #435

rstmsn opened this issue Sep 15, 2023 · 3 comments · May be fixed by #445

Comments

@rstmsn
Copy link

rstmsn commented Sep 15, 2023

I've cloned the repo and launched docker-compose up.

As you can see in the following screenshot, the iris-messenger service has started successfully.
When i point my browser to localhost:8080, however, nothing is being served.

Screen Shot 2023-09-15 at 23 17 18

I did spot the following dependency warnings, when the Dockerfile image was being built:

Screen Shot 2023-09-15 at 23 14 05

Couild this be implicated?

I'm using MacOS.
All suggestions welcome.

@mmalmi
Copy link
Member

mmalmi commented Sep 18, 2023

The correct address is on port 5173, http://localhost:5173

@rstmsn
Copy link
Author

rstmsn commented Sep 18, 2023

Strange that I can't access the service from the host machine.

I did docker-compose exec iris-messenger apt-get install wget
and
docker-compose exec iris-messenger wget localhost:5173 , and it succesfully fetched the iris index page.

Any suggestions as to why it's inaccessible from the host machine ?

@rstmsn
Copy link
Author

rstmsn commented Sep 18, 2023

Update: I added the --host flag to the final CMD command in the Dockerfile.dev:

CMD [ "yarn", "dev", "--host"]

This has exposed the service and it is now accessible by the host machine.

gasull added a commit to gasull/iris-messenger that referenced this issue Oct 22, 2023
Fixes irislib#435 .

There might be better ways of fixing this.  If so, please suggest.
@gasull gasull linked a pull request Oct 22, 2023 that will close this issue
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 a pull request may close this issue.

2 participants