cloudfoundry / nginx-buildpack

Cloud Foundry buildpack that provides NGINX
Apache License 2.0
30 stars 67 forks source link

supply: when {{port}} validation fails, log it clearly #258

Closed arjun024 closed 11 months ago

arjun024 commented 11 months ago

Previously, only the final error return statement in validateNginxConfHasPort() would log the message that the {{port}} template is missing. On any errors before that statement (like varify returning exit 1), no messages were logged that lets a user know that {{port}} template is missing for the listen directive.

Fixes https://github.com/cloudfoundry/nginx-buildpack/issues/253