Commit f3270de2 authored by Daniel Caillibaud's avatar Daniel Caillibaud

aj de routes manquantes + commentaire

parent 6bc50f3d
...@@ -66,19 +66,24 @@ module.exports = function (mainComponent) { ...@@ -66,19 +66,24 @@ module.exports = function (mainComponent) {
// On continue à passer ici par un contrôleur pour toutes les pages statiques, même si à première // On continue à passer ici par un contrôleur pour toutes les pages statiques, même si à première
// vue ce serait plus intelligent de construire ça au build avec html-webpack-plugin et le servir // vue ce serait plus intelligent de construire ça au build avec html-webpack-plugin et le servir
// en statique (via le serve qui précède), car les 3 routes dynamiques demandent un contrôleur et // en statique (via le serve qui précède), car les routes dynamiques demandent un contrôleur et
// représentent > 90% des requêtes, pas la peine de doublonner du code pour optimiser un peu les // représentent > 90% des requêtes, pas la peine de doublonner du code pour optimiser un peu les
// 10% qui restent. // 10% qui restent.
const reactRoutes = [ const reactRoutes = [
'/', '/',
'/autocomplete', '/autocomplete',
'/compte',
'/mentionsLegales', '/mentionsLegales',
'/ressource/ajouter', '/ressource/ajouter',
'/ressource/modifier/:oid', '/ressource/modifier/:oid',
'/ressource/apercevoir/:oid', '/ressource/apercevoir/:oid',
'/ressource/decrire/:oid', '/ressource/decrire/:oid',
'/ressource/rechercher', '/ressource/rechercher',
'/ressources' '/groupe/ajouter',
'/groupe/editer/:groupe',
'/groupes/perso',
'/groupes/ouverts',
'/groupes/publics'
] ]
reactRoutes.forEach(route => this.get(route, displayReactPage)) reactRoutes.forEach(route => this.get(route, displayReactPage))
...@@ -86,7 +91,7 @@ module.exports = function (mainComponent) { ...@@ -86,7 +91,7 @@ module.exports = function (mainComponent) {
this.get('/navigateurObsolete', displayObsoletePage) this.get('/navigateurObsolete', displayObsoletePage)
// lassi ne gère pas les requêtes head. nginx en frontal le fait pour nous, // lassi ne gère pas les requêtes head. nginx en frontal le fait pour nous,
// mais on veut répondre sur / pour le monitoring local (avec monit, 'protocol http' => head) // mais on veut répondre sur / pour le monitoring local (avec monit le 'protocol http' donne du head)
app.head('/', (req, res) => res.send()) app.head('/', (req, res) => res.send())
}) })
} }
...@@ -79,8 +79,8 @@ const getHtml = () => html ...@@ -79,8 +79,8 @@ const getHtml = () => html
* @param {string} [contentToAdd] Sera ajouté tel quel dans la page, après le div root (juste avant </body>), à priori du js… * @param {string} [contentToAdd] Sera ajouté tel quel dans la page, après le div root (juste avant </body>), à priori du js…
*/ */
function displayReactPage (context) { function displayReactPage (context) {
// sinon le content-type va imposer le transport html qui veut un template dust
context.contentType = 'text/html' context.contentType = 'text/html'
// faut utiliser raw sinon le content-type va imposer le transport html qui veut un template dust
context.raw(html, {}) context.raw(html, {})
// on peut fixer nos headers directement sur la réponse // on peut fixer nos headers directement sur la réponse
// context.response.append('Content-Length', reactPagelength) // context.response.append('Content-Length', reactPagelength)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment