Fixes #28: Use port 8080 by default for status page (#29)

Co-authored-by: Alessandro Fael Garcia <[email protected]>

Author: tux-o-matic

.travis.yml

@@ -12,6 +12,7 @@ jobs:
       env:
         scenario: default
     - name: (CentOS/Debian/Ubuntu) Test config generation with NGINX Plus directives
+      if: not (type = "pull_request" and fork = "true")
       env:
         scenario: plus
     - name: (Alpine Linux/CentOS/Debian/Ubuntu) Install stable branch and push a config

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.3.0 (Unreleased)
+
+BREAKING CHANGES:
+
+*   The default port of the status module is now 8080 and matches the CI molecule test which already used it. Set ```nginx_config_status_port```to another desired value.
+
+BUG FIXES:
+
+*   Prevent TravisCI from trying to build (and failing) NGINX Plus images on external PRs.
+
 ## 0.2.0 (September 24, 2020)
 
 BREAKING CHANGES:

defaults/main/template.yml

@@ -361,7 +361,7 @@ nginx_config_status_enable: false
 nginx_config_status_template_file: http/status.conf.j2
 nginx_config_status_file_location: /etc/nginx/conf.d/status.conf
 nginx_config_status_log: false
-nginx_config_status_port: 80
+nginx_config_status_port: 8080
 nginx_config_status_allow: 127.0.0.1
 nginx_config_status_deny: all
 

templates/http/status.conf.j2

@@ -1,15 +1,15 @@
 {{ ansible_managed | comment }}
 
 server {
-    listen {{ nginx_config_status_port | default('80') }};
+    listen {{ nginx_config_status_port | default('8080') }};
     access_log {{ nginx_config_status_log | ternary('on', 'off') }};
     location /nginx_status {
         stub_status on;
+    }
 {% if nginx_config_status_allow is defined %}
-        allow {{ nginx_config_status_allow }};
+    allow {{ nginx_config_status_allow }};
 {% endif %}
 {% if nginx_config_status_deny is defined %}
-        deny {{ nginx_config_status_deny }};
+    deny {{ nginx_config_status_deny }};
 {% endif %}
-    }
 }