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

Update README.md #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #65

wants to merge 1 commit into from

Conversation

grantland
Copy link
Contributor

Add instructions to connect with torrent containers

Add instructions to connect with torrent containers
@cpoppema
Copy link
Owner

Thanks @grantland, Any particular reason why you wouldn't want to use the host ip ?

@grantland
Copy link
Contributor Author

I actually didn't know about host ip, so TIL!

I'm pretty new with docker and I had trouble figuring out what host to use. After looking into docker host IPs, it seems like mac/windows has host.docker.internal and linux has to manually get the host IP via ip -4 route list match 0/0 | cut -d' ' -f3.

Do you think it would be worthwhile to add that information instead or would most docker users already know this and I somehow missed it?

@jmuchovej
Copy link

There's also the ability to just not expose the ports and have your containers on the same network. @grantland that makes it nicer, if you're trying to decrease the number of ports you open on your host machine. 🙂

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.

3 participants