Merge pull request #154 from M2Mobi/server_config_lists

Fix multiple values for the same key (include/listen) in server config file

nginx/ng/files/server.conf

@@ -2,17 +2,17 @@
 {%- macro server_config(values, key='', ind=0, lb='\n', delim=';', operator=' ') -%}
     {%- for value in values -%}
         {%- if value is number or value is string -%}
-{{ lb }}{{ key|indent(ind, True) }}{{ operator }}{{ value }}{{ delim }}
+{{ key|indent(ind, True) }}{{ operator }}{{ value }}{{ delim }}{{ lb }}
         {%- elif value is mapping -%}
             {%- for k, v in value.items() -%}
                 {%- if v is number or v is string -%}
 {{ server_config([v], k, ind) }}
                 {%- elif v|length() > 0 and (v[0] is number or v[0] is string) -%}
-{{ lb }}{{ k|indent(ind,True) }}{{ server_config(v,'', 0, '', '')}}{{ delim }}
+{{ server_config(v, k, ind) }}
-                {%- else %}
+                {%- else -%}
 {{ lb }}{{ k|indent(ind, True) }} {{ '{' }}
-{{- server_config(v, '', ind + ind_increment) }}
+{{ server_config(v, '', ind + ind_increment) -}}
-{{ '}'|indent(ind, True) }}
+{{ '}'|indent(ind, True) }}{{ lb }}
                 {%- endif -%}
             {%- endfor -%}
         {%- elif value is iterable -%}

pillar.example

@@ -201,28 +201,40 @@ nginx:
             - server:
               - server_name: localhost
               - listen:
-                - 80
+                - '80 default_server'
-                - default_server
+              - listen:
-              - index:
+                - '443 ssl'
-                - index.html
+              - index: 'index.html index.htm'
-                - index.htm
               - location ~ .htm:
-                - try_files:
+                - try_files: '$uri $uri/ =404'
-                  - $uri
-                  - $uri/ =404
                 - test: something else
               - include 'snippets/letsencrypt.conf'
 
-          # The above outputs:
+          # Or a slightly more compact alternative syntax:
+
+            - server:
+              - server_name: localhost
+              - listen:
+                - '80 default_server'
+                - '443 ssl'
+              - index: 'index.html index.htm'
+              - location ~ .htm:
+                - try_files: '$uri $uri/ =404'
+                - test: something else
+              - include 'snippets/letsencrypt.conf'
+
+          # both of those output:
           # server {
           #    server_name localhost;
           #    listen 80 default_server;
+          #    listen 443 ssl;
           #    index index.html index.htm;
           #    location ~ .htm {
           #        try_files $uri $uri/ =404;
           #        test something else;
           #    }
           # }
+
         mysite2: # Using source_path options to upload the file instead of templating all the file
           enabled: True
           available_dir: /etc/nginx/sites-available