Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configurer le expires sur les assets #380

Open
agallou opened this issue Mar 6, 2022 · 3 comments
Open

Configurer le expires sur les assets #380

agallou opened this issue Mar 6, 2022 · 3 comments

Comments

@agallou
Copy link
Member

agallou commented Mar 6, 2022

On avait cette configuration sur les assets du barometre quand il était hébergé sur le dédié.

Avec la migration sur Clever Cloud on a perdu cette configuration.

    <Directory /home/sources/barometre/current/web/geofla>
        SetOutputFilter DEFLATE
    </Directory>
    <Directory /home/sources/barometre/current/web/assets/js>
        ExpiresActive On
        ExpiresDefault "access plus 12 month"
    </Directory>
    <Directory /home/sources/barometre/current/web/assets/css>
        ExpiresActive On
        ExpiresDefault "access plus 12 month"
    </Directory>
    <Directory /home/sources/barometre/current/web/assets/logos>
        ExpiresActive On
        ExpiresDefault "access plus 12 month"
    </Directory>

Il serait utile de reprendre cette configuration dans le fichier .htaccess, ici : https://github.com/afup/barometre/blob/master/public/.htaccess.

Cela améliorerais la webperf à nouveau (cf https://webpagetest.org/performance_optimization.php?test=220117_AiDc97_704eb281a973cbf496b1e500828199ba&run=3#cache_static_content)

@klnjmm
Copy link
Contributor

klnjmm commented Mar 6, 2022

Je vais voir si on ne peut pas mettre comme pour les css / js un code après l’URL des assets pour qu’ils soient mise en cache automatiquement par le navigateur mais qu’ils soient remplacer lors de nouveau build. (Et si c’est pas le cas pour le js et css faire pareil).

@klnjmm
Copy link
Contributor

klnjmm commented Mar 6, 2022

Ah mais il y a déjà sur les fichiers images (et pas sur les js et css apparemment)

082E9DF8-408F-4DBC-892E-2A608E9B05FE

Quel est l’intérêt du max-age si on gère ça par « build » ?

En tout cas je vais regarder ça pour les css et js (version par build)

@agallou
Copy link
Member Author

agallou commented Mar 7, 2022

Ça va permettre de ne charger le js/css qu'une seule fois (et c'est aussi pour ça qu'on génère un nom de fichier, en fonction de son son contenu : pour pourvoir ajouter cette configuration)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

2 participants