Fix rewrite option in modules template (#193)

Author: tcraxs

CHANGELOG.md

@@ -9,6 +9,7 @@ BUG FIXES:
 * NGINX listen `so_keepalive` parameter was not working as intended when setting specific values.
 * Make sure all template objects are properly transformed into strings before doing Jinja2 operations.
 * Remove unnecessary parentheses.
+* Fix rewrite option in modules template.
 
 ## 0.4.1 (October 25, 2021)
 

molecule/default/verify.yml

@@ -82,3 +82,12 @@
       check_mode: true
       register: conf
       failed_when: (conf is changed) or (conf is failed)
+
+    - name: Ensure default.conf contains 'rewrite (.*).html(.*) $1$2 last;'
+      lineinfile:
+        path: /etc/nginx/conf.d/default.conf
+        line: "    rewrite (.*).html(.*) $1$2 last;"
+        state: present
+      check_mode: true
+      register: conf
+      failed_when: (conf is changed) or (conf is failed)

templates/http/modules.j2

@@ -199,7 +199,7 @@ open_log_file_cache {{ 'off' if not log['open_log_file_cache'] else ('max=' + lo
 return {{ rewrite['return'] if (rewrite['return'] is string or rewrite['return'] is number) }}{{ rewrite['return']['code'] if rewrite['return']['code'] is defined }}{{ (' ' + rewrite['return']['text'] | string) if rewrite['return']['text'] is defined }}{{ (' ' + rewrite['return']['url'] | string) if rewrite['return']['url'] is defined }};
 {% endif %}
 {% if rewrite['rewrites'] is defined %}{# 'rewrite' directive is not available in the 'http' context #}
-{% for rewrite in rewrite['rewrites'] if (rewrite['rewrites'] is sequence and rewrite['rewrites'] is not string) %}
+{% for rewrite in rewrite['rewrites'] if (rewrite['rewrites'] is not mapping and rewrite['rewrites'] is not string) %}
 rewrite {{ rewrite['regex'] }} {{ rewrite['replacement'] }}{{ (' ' + rewrite['flag'] | string) if rewrite['flag'] is defined and rewrite['flag'] in ['last', 'break', 'redirect', 'permanent'] }};
 {% endfor %}
 {% endif %}