drupalwxt / helm-drupal

Helm Chart for deploying an enterprise-grade Drupal environment.
https://drupalwxt.github.io/helm-drupal/index.yaml
MIT License
31 stars 22 forks source link

feat(nginx): allow custom nginx locations to be configured in the values #105

Closed spotzero closed 2 years ago

spotzero commented 2 years ago

I have a use case for needing a second location entry point from nginx into Drupal. I believe this could be accomplished transparently by making the nginx configuration configurable in a similar way to the Varnish config.

I think the following change is a reasonable and transparent way of adding that support, but I'd appreciate if someone could review the sanity of this approach :)

sylus commented 2 years ago

@spotzero this looks good to me can you do this for the drupal7 version and just bump the chart.yaml versions?

I will then merge!

spotzero commented 2 years ago

@sylus Updated :+1: