Filter out the empty hosts from yves/zed on nginx conf.template

[Chemaclass]

Description

Allow defining null/empty external_hosts and filter them out from the conf.d yves|zed template.

[andytson-inviqa]

Thanks for removing the merge, I was about to comment on that :)

[Chemaclass]

@andytson-inviqa instead of using the raw-int 600, what about using @('php.fpm.ini.max_execution_time') for the other properties in zed.conf.template as well? Simillar as we currently do for the fastcgi_read_timeout:

proxy_read_timeout 600s;
proxy_send_timeout 600s;
fastcgi_read_timeout {{ @('php.fpm.ini.max_execution_time') + 1 }}s;
client_body_timeout 600s;
client_header_timeout 600s;
send_timeout 600s;

This could be done in this PR or in a follow-up PR.

UPDATED: I created an issue for this topic: #688

[andytson-inviqa]

might be worth a github issue or PR to discuss that, as I'm not sure but others might have an opinion. This PR will be merged when the jenkins build passes