Protected Folders: duplicate location problem (nginx)

Hi! I use nginx 1.11.13 with ISPConfig 3.1. I have a directive snippet with location block:

Code:

location / {
                try_files $uri $uri/ /index.php?$args;
        }

I want to create a protected "root" folder, i.e. "/". After I configure it in ISPConfig, I see that the new nginx vhost config becomes invalid and is not applied. The reason is that it contains 2 location blocks for "/" (nginx doesn't allow that):

Code:

location / {
                try_files $uri $uri/ /index.php?$args;
        }

###
# some other nginx directives
###

location / { ##merge##
                auth_basic "Members Only";
                auth_basic_user_file /var/www/clients/client1/web3/web/.htpasswd

                location ~ \.php$ {
                    try_files /aadedef43d867c0738663b0445264c50.htm @php;
                }
        }

It seems that the location "/" is not merged. Is it a bug or I do something wrong?

 

Thank you very much for a prompt response! I made another test and used "test" folder instead of "/" location:

Code:

location /test/ {
             try_files $uri $uri/ /index.php?$args;
        }

## some other nginx directives

location /test/ { ##merge##
                auth_basic "Members Only";
                auth_basic_user_file /var/www/clients/client1/web5/web/test/.htpasswd;

                location ~ \.php$ {
                    try_files /89f314d371fa173948fcad289dd51f95.htm @php;
                }
        }

The results are the same: still duplicate location error from nginx. FYI, the second location with ##merge## parameter was generated automatically by ISPConfig (not me!), if I enable the folder protection (Sites - Web Access - Protected Folders). I also tried to manually adjust and remove the space before ##merge## - no luck, still the same. Could it be a bug that the processing of ##merge## directive by ISPConfig is not performed for protected folders?

 

Thank you, created the issue https://git.ispconfig.org/ispconfig/ispconfig3/issues/4639