correct htaccess caching parameters

Question

I have been reading up on the subject of htaccess caching to help increase the speed of my website. I put together the following based on the information from different website sources as I want to improve the speed of my cms.

Can someone tell me if this is the proper way to use the parameters?

### caching rules
<IfModule mod_mime.c>
    AddType application/x-javascript .js
    AddType text/css .css
</IfModule>
<IfModule mod_deflate.c>
    #AddOutputFilterByType DEFLATE text/text text/html text/plain text/xml text/css application/x-javascript application/javascript
    AddOutputFilterByType DEFLATE text/css application/x-javascript text/x-component text/html text/richtext image/svg+xml text/plain text/xsd text/xsl text/xml image/x-icon application/javascript
    <IfModule mod_setenvif.c>
    BrowserMatch ^Mozilla/4 gzip-only-text/html
    BrowserMatch ^Mozilla/4\.0[678] no-gzip
    BrowserMatch \bMSIE !no-gzip !gzip-only-text/html
    </IfModule>
    <IfModule mod_headers.c>
    Header append Vary User-Agent env=!dont-vary
    </IfModule>
</IfModule>

<IfModule mod_deflate.c>
    SetOutputFilter DEFLATE
    <IfModule mod_setenvif.c>
        SetEnvIfNoCase Request_URI \.(?:rar|zip)$ no-gzip dont-vary
        SetEnvIfNoCase Request_URI \.(?:gif|jpg|png)$ no-gzip dont-vary
        SetEnvIfNoCase Request_URI \.(?:avi|mov|mp4)$ no-gzip dont-vary
        SetEnvIfNoCase Request_URI \.mp3$ no-gzip dont-vary
    </IfModule>
</IfModule>

<IfModule mod_gzip.c>
    mod_gzip_on Yes
    mod_gzip_dechunk Yes
    mod_gzip_item_include file \.(html?|txt|css|js|php|pl)$
    mod_gzip_item_include handler ^cgi-script$
    mod_gzip_item_include mime ^text/.*
    mod_gzip_item_include mime ^application/x-javascript.*
    mod_gzip_item_exclude mime ^image/.*
    mod_gzip_item_exclude rspheader ^Content-Encoding:.*gzip.*
</IfModule>

Header unset Pragma
FileETag None
Header unset ETag

<IfModule mod_headers.c>
    Header unset ETag
</IfModule>
FileETag None

<FilesMatch "\.(ico|jpg|jpeg|png|gif|js|css|swf|pdf|flv|mp3)$">
<IfModule mod_expires.c>
    ExpiresActive on
    ExpiresDefault "access plus 14 days"
    Header set Cache-Control "public"
</IfModule>
</FilesMatch>
<FilesMatch "\.(html|htm|xml|txt|xsl)$">
    Header set Cache-Control "max-age=7200, must-revalidate"
</FilesMatch>

<IfModule mod_expires.c>
    ExpiresActive On
    #ExpiresDefault "access plus 1 week"
    ExpiresDefault "access plus 86400 seconds"
    ExpiresByType image/x-icon "modification plus 1 year"
    ExpiresByType image/gif "access plus 1 year"
    ExpiresByType image/png "access plus 1 year"
    ExpiresByType image/jpeg "access plus 1 year"
    ExpiresByType image/jpg "access plus 1 year"
    ExpiresByType text/js "access plus 1 month"
    ExpiresByType text/javascript "access plus 1 month"
    ExpiresByType application/javascript "access plus 1 month"
    ExpiresByType application/x-javascript "access plus 1 month"
    ExpiresByType application/x-shockwave-flash "access plus 1 month"
    ExpiresByType video/x-flv "access plus 1 month"
    ExpiresByType application/pdf "access plus 1 month"

    #ExpiresByType text/html "access plus 1 day"
    #ExpiresByType text/css "access plus 1 day"
    #ExpiresByType application/xhtml+xml "access plus 1 day"
    #ExpiresByType text/plain "access plus 1 month"

    #ExpiresByType text/javascript "access plus 604800 seconds"
    #ExpiresByType application/javascript "access plus 604800 seconds"
</IfModule>

Answer 1

The following line needs updating from:

SetEnvIfNoCase Request_URI \.(?:gif|jpg|png)$ no-gzip dont-vary

to

SetEnvIfNoCase Request_URI \.(?:gif|jpe?g|png)$ no-gzip dont-vary

You also seem to have some duplication here:

Header unset Pragma
FileETag None
Header unset ETag

<IfModule mod_headers.c>
    Header unset ETag
</IfModule>
FileETag None

And additionally, it'd be worth combining your <IfModule> directives, so all the directives for mod_expires are in one location and you should also remove the duplicate expires settings. E.g.

<FilesMatch "\.(ico|jpg|jpeg|png|gif|js|css|swf|pdf|flv|mp3)$">
  <IfModule mod_expires.c>
    ExpiresActive on
    ExpiresDefault "access plus 14 days"
    Header set Cache-Control "public"
  </IfModule>
</FilesMatch>

gets overwritten by

<IfModule mod_expires.c>
    ExpiresActive On
    ExpiresDefault "access plus 86400 seconds"
    ExpiresByType image/x-icon "modification plus 1 year"
    ExpiresByType image/gif "access plus 1 year"
    ExpiresByType image/png "access plus 1 year"
    ExpiresByType image/jpeg "access plus 1 year"
    ExpiresByType image/jpg "access plus 1 year"
    ExpiresByType text/js "access plus 1 month"
    ExpiresByType text/javascript "access plus 1 month"

Other than that, yes.