Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Nginx http service runs hook_provision_nginx_vhost_config() and hook_provision_nginx_server_config() but extra configuration lines don't get inside the configs because of missing template variables.
I had to add this:
<?php print $extra_config; ?>
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | provision_nginx_extra_config_1170498.patch | 1.18 KB | crea |
Comments
Comment #1
anarcat CreditAttribution: anarcat commentedI would let omega8cc comment on that, but if you could provide a patch, that would be awesome.
Comment #2
crea CreditAttribution: crea commentedSure
Comment #3
omega8cc CreditAttribution: omega8cc commentedWell done.
Thanks for the patch!
Comment #4
anarcat CreditAttribution: anarcat commentedawesome, thanks! pushed to 1.x and 2.x.
Comment #6
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedI believe this may not actually be fixed in Provision 2.x. I have an nginx server that seems unaffected by the
<?php print $extra_config; ?>in the two aforementioned files.The locations of these differ from the patch from what I can tell.
/var/aegir/.drush/provision/http/Provision/Config/Nginx/for me.Comment #7
omega8cc CreditAttribution: omega8cc commentedAre you aware that since 2.x and introduction of Drush 5 and now Drush 6, you have to clear Drush own cache before any extra file will be really used/visible to Drush and thus Provision via this method?
Comment #8
omega8cc CreditAttribution: omega8cc commentedThis particular issue has been fixed. Please open new issue if you experience problems.
Comment #9
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedActually, false alarm, a drush cc drush fixed it. Sorry for the noise.