search

MTES-MCT / dialog

Intégration de la réglementation de circulation dans les solutions numériques
https://dialog.beta.gouv.fr
GNU Affero General Public License v3.0
8 stars 0 forks source link

Active la compression gzip en local #850

Closed florimondmanca closed 5 days ago

florimondmanca commented 5 days ago

Scalingo applique bien une compression gzip en production https://github.com/MTES-MCT/dialog/pull/850#issuecomment-2191302851

Mais on ne l'avait pas encore en local

Cette PR l'active sur le Nginx local pour être iso avec la prod.

Le gzip réduit l'utilisation du réseau pour le transfert du HTML (pages) ou du XML (DATEX, CIFS...) d'un facteur 5 à 10. Par exemple en local l'arrêté JOP (#847) passe de 325 Ko de HTML transférés à 30 Ko !

codecov-commenter commented 5 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.10%. Comparing base (9840c8f) to head (8f7e326).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #850 +/- ## ========================================= Coverage 99.10% 99.10% Complexity 1170 1170 ========================================= Files 218 218 Lines 4939 4939 ========================================= Hits 4895 4895 Misses 44 44 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mmarchois commented 5 days ago

Après, je constate sur la prod que la compression est déjà activée (cf les headers) image

mmarchois commented 5 days ago

Idem sur les fichiers XML d'ailleurs https://dialog.beta.gouv.fr/api/regulations.xml image

florimondmanca commented 5 days ago

Ah oui je n'avais pas vérifié, c'était en local que ça m'a sauté aux yeux. Scalingo a l'air d'appliquer le gzip. Je vais modifier la PR pour activer le gzip dans notre default.conf alors (sinon ça ferait doublon avec ce que fait Scalingo)