Add FileETag

Author: kuchosauronad0

manifests/init.pp

@@ -76,6 +76,7 @@
   $server_signature       = 'On',
   $trace_enable           = 'On',
   $allow_encoded_slashes  = undef,
+  $file_e_tag             = undef,
   $package_ensure         = 'installed',
   $use_optional_includes  = $::apache::params::use_optional_includes,
   $use_systemd            = $::apache::params::use_systemd,

spec/acceptance/apache_parameters_spec.rb

@@ -483,6 +483,22 @@ class { 'apache':
     end
   end
 
+  describe 'file_e_tag' do
+    it 'applys cleanly' do
+      pp = <<-EOS
+        class { 'apache':
+          file_e_tag  => 'None',
+        }
+      EOS
+      apply_manifest(pp, :catch_failures => true)
+    end
+
+    describe file($conf_file) do
+      it { is_expected.to be_file }
+      it { is_expected.to contain 'FileETag None' }
+    end
+  end
+
   describe 'package_ensure' do
     it 'applys cleanly' do
       pp = <<-EOS

spec/classes/apache_spec.rb

@@ -110,6 +110,14 @@
       it { is_expected.to contain_file("/etc/apache2/apache2.conf").with_content %r{^AllowEncodedSlashes nodecode$} }
     end
 
+    context "when specifying fileETag behaviour" do
+      let :params do
+        { :file_e_tag => 'None' }
+      end
+
+      it { is_expected.to contain_file("/etc/apache2/apache2.conf").with_content %r{^FileETag None$} }
+    end
+
     context "when specifying default character set" do
       let :params do
         { :default_charset => 'none' }

templates/httpd.conf.erb

@@ -61,6 +61,9 @@ EnableSendfile <%= @sendfile %>
 <%- if @allow_encoded_slashes -%>
 AllowEncodedSlashes <%= @allow_encoded_slashes %>
 <%- end -%>
+<%- if @file_e_tag -%>
+FileETag <%= @file_e_tag %>
+<%- end -%>
 
 #Listen 80