search

x86dev / docker-ttrss

Tiny Tiny RSS feed reader as a Docker image.
54 stars 41 forks source link

Fix port setting #10

Closed mestery closed 7 years ago

mestery commented 7 years ago

This corrects the port setting introduced in commit ecd2c19 (and likely earlier). If a user does not specify a port, just use the default ports and do not force setting one. By forcing it, it breaks logging in via the web console.

Signed-off-by: Kyle Mestery mestery@mestery.com

x86dev commented 7 years ago

Could you please elaborate which web console you mean which breaks?

Defaulting to port 8080 if nothing else is specified has the reason that I don't want to run nginx as root inside the container.

x86dev commented 7 years ago

Just re-tested and committed your patch in a slightly different way. Thanks for noticing!