betagouv / rdv-service-public

Prise de RDV pour les services publics
https://rdv.anct.gouv.fr
GNU Affero General Public License v3.0
19 stars 3 forks source link

Webpack : les source maps ne fonctionnent pas en local #3096

Open francois-ferrandis opened 2 years ago

francois-ferrandis commented 2 years ago

Actuellement

image

Ce fonctionnement est peu performant et peu lisible.

Objectif

Technique

Les source maps semblent bien générés par webpack, mais on retrouve dans les logs

ActionController::RoutingError (No route matches [GET] "/assets/application_agent.js-52111dc76d7c5157026be3b71cb45626ff78ea68468c1cc3c5cfb90f02d1b008.map"):

Il y a quelque-chose de mal configuré entre sprockets et webpack. :thinking:

francois-ferrandis commented 5 months ago

@adipasquale Je vois que tu parles de faire fonctionner les source maps dans #4307 donc je me permets de référencer cette issue. :link:

adipasquale commented 5 months ago

merci @francois-ferrandis ! je vais regarder ça. J’ai l’impression que ça marche pour le bundle application.js

par ex sur https://demo.rdv-solidarites.fr/assets/application.js-8a44c055a7ebb66c15f47179c5273c3d9b8ca386ca6c6b201e6a1201abb1dd2e.map je trouve

//# sourceMappingURL=/assets/application.js-8a44c055a7ebb66c15f47179c5273c3d9b8ca386ca6c6b201e6a1201abb1dd2e.map

et cette URL de sourcemap est bien accessible

j’ai vérifié et c’est aussi le cas pour la source map du bundle application_agent

le souci décrit dans cette issue semble donc limité à l’env local où effectivement les source maps ne fonctionnent pas

le souci dans ma PR avec le DSFR est différent : ça ne marche pas en prod car je pense que la source map url est indiqué en dur dans le dsfr.min.js alors qu’on refait passer ce fichier par webpack. mon avis serait de ne plus passer par webpack pour ces assets déjà compilés, je vais bientôt proposer une PR

francois-ferrandis commented 5 months ago

@adipasquale Merci pour ces clarifications. :pray: