add additional docker configuration environment variables - #4
Conversation
|
btw - should not squashing commits on merge do the same? One final consolidated commit in master on your repo? |
|
I haven't tried that with Github, but we were mixing a couple different things and I'd rather keep commits to a single purpose. I added this in the review of your comments in the README, but let's discuss it here: I run this behind an nginx proxy as a subfolder using linuxserver.io's swag container in the same docker-compose w/o having to do any of this. In your example, if you specify the port as 8080 then it will be available to whatever servers can reach your docker host. Are you running this on a private network with proxy between that and the Internet? Please provide more detail on where this configuration would be needed because as worded it seems wrong to me. |
|
Sorry, but i had not seen any comments in the changed files itself, only the conversation comment. Regarding the you are right - it is available than on all ip addresses and might not be the best example for a quick-start HowTo. It was a copy-paste error as i have to expose it on a different port than 8080 (8080 already used) and removed my local address there. And for all the The Most people will not need these PROXY settings as they might use the reverse proxy with a sub-path like |
|
@jchonig some progress here? |
this new PR replaces #2 and #3.