Changed extra conf location in template

[icamys]

In version 1.9.11 there were introduced dynamic modules. With this version has appeared a new directive load_module.

To include dynamic module, such as geoip, we should specify path relative to the source directory:

load_module "modules/ngx_http_geoip_module.so";

The point here is that this directive should be included in main context before any other contexts.

For example, this config triggers error:

user www-data;
pid        /run/nginx.pid;

worker_processes  4;

events {
    ...
}

load_module "modules/ngx_http_geoip_module.so";

http {
    ...
}
...

The error text:

root@server:/home/vagrant# nginx -t
nginx: [emerg] "load_module" directive is specified too late in /etc/nginx/nginx.conf:13
nginx: configuration file /etc/nginx/nginx.conf test failed

However this config works fine:

user www-data;
pid        /run/nginx.pid;

worker_processes  4;

load_module "modules/ngx_http_geoip_module.so";

events {
    ...
}

http {
    ...
}
...

That's why I propose to lift up extra conf location in this PR.

[thomas-frantz]

lol, I didn't even see someone did this already, I can close my PR if we want.

@geerlingguy Is there something wrong with this being merged in that one of us could fix?

[geerlingguy]

I'd be happy to merge this in—please fix the conflict/rebase then I'll merge it. Thanks!

[geerlingguy]

Merged! It's in 2.4.0.