From 2360d722b64abe8d8e04f41c880fe8659bcc0ed3 Mon Sep 17 00:00:00 2001
From: Louis Haddrell <louishaddrell@gmail.com>
Date: Wed, 2 Aug 2017 00:00:36 +0100
Subject: [PATCH] Replaced use_require with Apache version check (fixes #188)

---
 apache/map.jinja            | 17 -----------------
 apache/vhosts/proxy.tmpl    |  4 ++--
 apache/vhosts/standard.tmpl |  4 ++--
 3 files changed, 4 insertions(+), 21 deletions(-)

diff --git a/apache/map.jinja b/apache/map.jinja
index e10dfb7..f86d07f 100644
--- a/apache/map.jinja
+++ b/apache/map.jinja
@@ -20,7 +20,6 @@
         'logdir': '/var/log/apache2',
         'logrotatedir': '/etc/logrotate.d/apache2',
         'wwwdir': '/var/www',
-        'use_require': False,
     },
     'Debian': {
         'server': 'apache2',
@@ -46,7 +45,6 @@
         'logdir': '/var/log/apache2',
         'logrotatedir': '/etc/logrotate.d/apache2',
         'wwwdir': '/srv',
-        'use_require': False,
     },
     'RedHat': {
         'server': 'httpd',
@@ -69,7 +67,6 @@
         'logrotatedir': '/etc/logrotate.d/httpd',
         'wwwdir': '/var/www',
         'default_charset': 'UTF-8',
-        'use_require': False,
     },
     'Suse': {
         'server': 'apache2',
@@ -89,7 +86,6 @@
         'default_site_ssl': 'vhost-ssl.template',
         'logdir': '/var/log/apache2',
         'wwwdir': '/srv/www',
-        'use_require': True,
     },
     'FreeBSD': {
         'server': 'apache22',
@@ -113,83 +109,70 @@
         'default_site_ssl': 'default-ssl',
         'logdir': '/var/log/',
         'wwwdir': '/usr/local/www/apache22/',
-        'use_require': False,
     },
 }, merge=salt['grains.filter_by']({
     'precise': {
         'confext': '',
         'default_site': 'default',
         'default_site_ssl': 'default-ssl',
-        'use_require': True,
     },
     'trusty': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'utopic': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'vivid': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'wily': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'xenial': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'yakkety': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'zesty': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'artful': {
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'jessie': {
         'wwwdir': '/var/www',
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'stretch': {
         'wwwdir': '/var/www',
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
     'buster': {
         'wwwdir': '/var/www',
         'confext': '.conf',
         'default_site': '000-default.conf',
         'default_site_ssl': 'default-ssl.conf',
-        'use_require': True,
     },
 }, grain='oscodename', merge=salt['grains.filter_by'](
     osfingermap
diff --git a/apache/vhosts/proxy.tmpl b/apache/vhosts/proxy.tmpl
index df1ef3b..af791a1 100644
--- a/apache/vhosts/proxy.tmpl
+++ b/apache/vhosts/proxy.tmpl
@@ -76,7 +76,7 @@
         'Dav': loc.get('Dav', False),
     } %}
     <Location "{{ path }}">
-      {% if apache.use_require %}
+      {% if apache.version == '2.4' %}
       {%- if lvals.get('Require') != False %}Require {{lvals.Require}}{% endif %}
       {% else %}
       {%- if lvals.get('Order') != False %}Order {{ lvals.Order }}{% endif %}
@@ -93,7 +93,7 @@
         'Dav': locmat.get('Dav', False),
     } %}
     <LocationMatch "{{ regpath }}">
-      {% if apache.use_require %}
+      {% if apache.version == '2.4' %}
       {%- if lmvals.get('Require') != False %}Require {{lmvals.Require}}{% endif %}
       {% else %}
       {%- if lmvals.get('Order') != False %}Order {{ lmvals.Order }}{% endif %}
diff --git a/apache/vhosts/standard.tmpl b/apache/vhosts/standard.tmpl
index 414d003..97bff80 100644
--- a/apache/vhosts/standard.tmpl
+++ b/apache/vhosts/standard.tmpl
@@ -88,7 +88,7 @@
 
     <Directory "{{ path }}">
         {% if dvals.get('Options') != False %}Options {{ dvals.Options }}{% endif %}
-        {% if map.use_require %}
+        {% if map.version == '2.4' %}
         {% if dvals.get('Require') != False %}Require {{dvals.Require}}{% endif %}
         {% else %}
         {% if dvals.get('Order') != False %}Order {{ dvals.Order }}{% endif %}
@@ -112,7 +112,7 @@
     } %}
 
     <Location "{{ path }}">
-        {% if map.use_require %}
+        {% if map.version == '2.4' %}
         {%- if lvals.get('Require') != False %}Require {{lvals.Require}}{% endif %}
         {% else %}
         {%- if lvals.get('Order') != False %}Order {{ lvals.Order }}{% endif %}