Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add note to manhole.md about address binding when using docker #8508

Closed
ShadowJonathan opened this issue Oct 8, 2020 · 1 comment · Fixed by #8526
Closed

Add note to manhole.md about address binding when using docker #8508

ShadowJonathan opened this issue Oct 8, 2020 · 1 comment · Fixed by #8526
Labels
A-Docker Docker images, or making it easier to run Synapse in a container. A-Docs things relating to the documentation Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution

Comments

@ShadowJonathan
Copy link
Contributor

I recently came across this problem, where the manhole was binding to 127.0.0.1 within the container, it wasn't clear to me at first why docker immediately closed all connections then.

I think a note should be added to manhole.md, when using docker, that it has to be set to 0.0.0.0, or else docker cant forward the port to it.
@clokep clokep added A-Docker Docker images, or making it easier to run Synapse in a container. A-Docs things relating to the documentation Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution labels Oct 9, 2020
maquis196 added a commit to maquis196/synapse that referenced this issue Oct 12, 2020
@maquis196
matrix-org#8508 - Add note to manhole.md about bind_addresses when us…
a4da2a7 
…ing docker
@maquis196
Copy link
Contributor

PR added with note about the above.

@richvdh richvdh mentioned this issue Oct 13, 2020
Merged
4 tasks
@richvdh richvdh linked a pull request Oct 13, 2020 that will close this issue
Add note to manhole.md about bind_address when using with docker #8526
Merged
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Docker Docker images, or making it easier to run Synapse in a container. A-Docs things relating to the documentation Z-Help-Wanted We know exactly how to fix this issue, and would be grateful for any contribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add note to manhole.md about bind_address when using with docker maquis196/synapse
3 participants
@clokep @maquis196 @ShadowJonathan