Nginx full config optional (via installer)

[fastlorenzo]

The installer will comment/uncomment the include reference to the part that is used only in the "full" install mode.

Fixes #150

[xychix]

Bit thrown off here. is LetsEncrypt the default by now? Guess that wouldn't work in our environment does this mean wel always run dev? or should it be the other way around and make LE an additional flag.

Also kinda more @MarcOverIP territory

[fastlorenzo]

@xychix nope, it's just a fix that applies only for Let's Encrypt, which is using symlinks to parent directory (so if you mount the base directory that contains the certificates, Nginx will complain as the referenced files are outside the mounted folder, and thus not accessible to docker)

[MarcOverIP]

@xychix no worries, Le's Encrypt is optional and not the default.

@fastlorenzo have you fully tested this change?

[fastlorenzo]

with the full install, yes (spinned up a clean server in Azure and did the test with this branch and LE).
There is apparently still an issue to be fixed for the limited install though, I'll try to test it this week so we can merge

[MarcOverIP]

Cool, thnx. You also testing without using LE?

[fastlorenzo]

will do ;)

[fastlorenzo]

Fully tested and ready to be merged