Update to 0.17.0 breaks image about docker-nginx-unprivileged

[alessfg]

Hi @smueller18!

Can you give me some more details about how and where are you trying to run the image? I tested it locally and on OpenShift and I didn't run into any issues. /var/cache/nginx is no longer in use either (switched the default location for NGINX files to /tmp/ too).

Cheers,
Alessandro.

[smueller18]

I just saw https://github.com/nginxinc/docker-nginx-unprivileged/blob/master/mainline/alpine/Dockerfile#L115 . There are 2 reasons why this new image fails for my setup:

1. The user changed from 1001 to 101
2. /var/cache/nginx/ is now moved to /tmp

I did not use the earlier provided nginx.conf so I have to manually adjust the changes in all my configs. I guess there are other people that are facing the same issue.
But after changing my custom nginx.conf to the new paths and switching the user to 101 it works again now.

[smueller18]

Last weekend, I opened a PR (nextcloud/docker#772) for the nextcloud docker project.
There is a config that also fails now until it is adjusted: https://github.com/nextcloud/docker/blob/2e00c8cb18ac6d94b756612f5f9223470c4a44d9/.examples/docker-compose/insecure/mariadb-cron-redis/fpm/web/nginx.conf .

[alessfg]

Ah yeah I'm sorry about that. I didn't make any notes anywhere mentioning the two breaking changes and forgot to update the README too while at it (that's been already fixed).

The main NGINX Docker image introduced some changes to make it easier to run the image as a non root user and in the process of porting those changes the uid and temp folder locations changed. However, on a more positive note, I expect few breaking changes moving forward.

[smueller18]

I am fine with that, it was just a bit unexpected 😄