Add http_raw_prepend and http_raw_append parameters

[walkamongus]

No description provided.

[wyardley]

Overall, this looks good to me, thanks for squashing commits, adding tests, and reformatting in terms of spacing.

One quick question in the inline comments, but overall, I think this looks sensible.

[wyardley]

Just curious, in the templates, it seems like this is only putting it in if it's an array, so is the string type needed too? I see that this is how it's implemented in the server resource, just want to make sure this is done for a reason. Or am I misunderstanding how types work?

Just from a quick read, it seems like this would allow it to be defined as a string, but then not add anything in the resulting config?

[walkamongus]

Hi @wyardley -- thanks for the comments! If a String is passed in, the template will cast it into an Array with one member e.g.

[1] pry(main)> v = 'String'
=> "String"
[2] pry(main)> Array(v)
=> ["String"]

I just used the added flexibility that was already at

puppet-nginx/templates/server/location_header.erb

Line 67 in 360e700

<% if @raw_prepend && Array(@raw_prepend).size > 0 -%>

[wyardley]

Got it, that makes sense, didn't realize it would do that!