jfryman/puppet-nginx — pull request #502 by cewood (2014-11-12 05:25:20+00:00)
Title: Reorganise whitespace in the vhost header and location header/footer.

The current vhost header and location header/footer templates insert new lines after a location, which can lead to overlapping/joined lines, and doesn't seem the best approach; as one can't assume there will always be more blocks that follow.

It would be more appropriate to include the new line before a new location block (or any other block inside the vhost for that matter) is inserted, which then naturally lends itself to the convention of only pre-pending new lines to said configuration blocks, never appending them.

I stumbled onto this when I came across a location statement that was appearing on the same line as other directives, the error log to be precise.