Is this correct code ?

I have included following code to set expiry header in .htaccess of my website. Please check and let me know if it is correct or anything additional to be added to improve website's working.

<IfModule mod_expires.c>
ExpiresActive on
# your document html
ExpiresByType text/html "access plus 10 minutes"
# favicon
ExpiresByType image/x-icon "access plus 1 month"
# media: images
ExpiresByType image/gif "access plus 1 month"
ExpiresByType image/png "access plus 1 month"
ExpiresByType image/jpg "access plus 1 month"
ExpiresByType image/jpeg "access plus 1 month"
# css and javascript
ExpiresByType text/css "access plus 1 week"
ExpiresByType text/javascript "access plus 1 week"

<IfModule mod_headers.c>
Header append Cache-Control "private"
<FilesMatch "\.(js|css|gz)$">
Header append Vary: Accept-Encoding
</FilesMatch>
</IfModule>
</IfModule>

I am good at SEO, I cant help you in coding bro...Sorry :)

Quote:

---

Originally Posted by Digitalpursuit

I am good at SEO, I cant help you in coding bro...Sorry :)

---

The code mentioned above is used in .htaccess file and it important w.r.t SEO . So I needed help in that.

Code:

---

<IfModule mod_expires.c>
    ExpiresActive on

    # Set expiration for HTML files
    ExpiresByType text/html "access plus 10 minutes"

    # Set expiration for favicon
    ExpiresByType image/x-icon "access plus 1 month"

    # Set expiration for image files
    ExpiresByType image/gif "access plus 1 month"
    ExpiresByType image/png "access plus 1 month"
    ExpiresByType image/jpg "access plus 1 month"
    ExpiresByType image/jpeg "access plus 1 month"

    # Set expiration for CSS and JavaScript files
    ExpiresByType text/css "access plus 1 week"
    ExpiresByType text/javascript "access plus 1 week"

    # Configure Cache-Control and Vary header
    <IfModule mod_headers.c>
        Header append Cache-Control "private"
        <FilesMatch "\.(js|css|gz)$">
            Header append Vary "Accept-Encoding"
        </FilesMatch>
    </IfModule>
</IfModule>

---