search

znc / znc-docker

https://hub.docker.com/_/znc/
51 stars 29 forks source link

is readme wrong? #26

Closed felixsanz closed 4 years ago

felixsanz commented 4 years ago

Readme says:

To run ZNC:

$ docker run -p 6697:6697 -v znc-cfg:/znc-data znc

The port should match the port you used during --makeconf. Note that 6667 is often blocked by web browsers, and therefore is not recommended.

Why the docker run uses port 6697 but then port 6667 is mentioned? which one exposes the container?

DarthGandalf commented 4 years ago

No port is exposed by default. During --makeconf you need to select a port. And don't use 6667 or 6697 for it, because browsers refuse to connect to these ports. Then, use the same port in the -p flag as what you chose.

What wording do you suggest?

felixsanz commented 4 years ago

to me it was confusing because i did not executed --makeconf yet

Maybe give information about that or use the same port in the docker readme, so at least is not that confusing

thanks!