From f3270de24e74bee21f66e8d02d7140d617b69a23 Mon Sep 17 00:00:00 2001 From: Daniel Caillibaud Date: Mon, 20 May 2019 11:26:04 +0200 Subject: [PATCH] aj de routes manquantes + commentaire --- app/server/main/controllerMain.js | 11 ++++++++--- app/server/main/reactPage.js | 2 +- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/app/server/main/controllerMain.js b/app/server/main/controllerMain.js index 7460045a..595d4b31 100644 --- a/app/server/main/controllerMain.js +++ b/app/server/main/controllerMain.js @@ -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 // 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 // 10% qui restent. const reactRoutes = [ '/', '/autocomplete', + '/compte', '/mentionsLegales', '/ressource/ajouter', '/ressource/modifier/:oid', '/ressource/apercevoir/:oid', '/ressource/decrire/:oid', '/ressource/rechercher', - '/ressources' + '/groupe/ajouter', + '/groupe/editer/:groupe', + '/groupes/perso', + '/groupes/ouverts', + '/groupes/publics' ] reactRoutes.forEach(route => this.get(route, displayReactPage)) @@ -86,7 +91,7 @@ module.exports = function (mainComponent) { this.get('/navigateurObsolete', displayObsoletePage) // 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()) }) } diff --git a/app/server/main/reactPage.js b/app/server/main/reactPage.js index 616fd576..8f22e521 100644 --- a/app/server/main/reactPage.js +++ b/app/server/main/reactPage.js @@ -79,8 +79,8 @@ const getHtml = () => html * @param {string} [contentToAdd] Sera ajouté tel quel dans la page, après le div root (juste avant ), à priori du js… */ function displayReactPage (context) { - // sinon le content-type va imposer le transport html qui veut un template dust context.contentType = 'text/html' + // faut utiliser raw sinon le content-type va imposer le transport html qui veut un template dust context.raw(html, {}) // on peut fixer nos headers directement sur la réponse // context.response.append('Content-Length', reactPagelength) -- GitLab