Block directives of nginx site.conf are not supported

[pavel-shvagirev]

👻 Brief Description

nginx site.conf's "block directives" are not supported: template adds semicolon at the end of line, which leads to nginx test failure (block directives must not have semicolon at the end of line)

🥞 Cookbook version

12.2.13

👩‍🍳 Chef-Infra Version

18.6.2

🎩 Platform details

nginx version: nginx/1.27.4

Steps To Reproduce

    '~ ^(.*)$' => {
      'limit_except' => node['limit_except'],
    },

where limit_except is a role-defined attr:
"limit_except": "GET POST { deny all; }",

that leads to template rendering that string with semicolon at the end which fails nginx -t

🚓 Expected behavior

block directives of site.conf should be rendered without semicolon at the end

[pavel-shvagirev]

Frankly, this looks like a total usage blocker unlike #644 since there is no ways other than no use of such directives (like limit_except which is quite usefull), or, again, move everything to an includes...

[bmhughes]

Thanks for the the report @pavel-shvagirev, the next release should correct this.