From d012419d1661f13db4c6555ab70aa92b11135685 Mon Sep 17 00:00:00 2001
From: Guillaume WANG <guillaume.wang@polytechnique.edu>
Date: Wed, 18 Apr 2018 00:43:12 +0200
Subject: [PATCH] same as previous commit, i.e. just adedd comments

---
 src/admin_view/admin_router.js |  9 ++++-----
 src/server.js                  | 18 +++++++++++++-----
 2 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/src/admin_view/admin_router.js b/src/admin_view/admin_router.js
index 3d61d71..c9e30ba 100644
--- a/src/admin_view/admin_router.js
+++ b/src/admin_view/admin_router.js
@@ -36,17 +36,16 @@ router.get('/admin',
     function (req, res) {
         console.log('Connecting to ' + req.url);
         let userName;
-        /*
-         * On ne veut pas déclencer d'erreur 500 si on ne peut pas lire l'utilisateur
-         * La personne n'est peut-être pas connectée, mais cela doit être géré autrement
-         * 
-         */
+        // Une erreur a ce stade peut etre triggered si req.user n'existe pas
+        // mais pour autant on est assures que la personne est bien authentifiee
+        // donc on laisse passer sans déclencher d'erreur 500
         try {
             let user = req.user;
             //let user = req.user;
             console.log('Welcome,',user.uid);
             userName = user.uid;
         } catch (err) {
+            console.log("Warning: in admin_router router.get('/admin')");
             console.log(err.message);
             userName = "No one";
         }
diff --git a/src/server.js b/src/server.js
index 70db881..9f43383 100644
--- a/src/server.js
+++ b/src/server.js
@@ -124,6 +124,8 @@ server.use(cors(corsOptions));
 // Charge le middleware express pour GraphQL
 server.use('/graphql', bodyParser.json(),
     graphqlExpress(req => {
+        // vary the options *on a per-request basis*
+        // (options are passed by using req object as argument)
         let uid;
         let password;
         
@@ -139,16 +141,22 @@ server.use('/graphql', bodyParser.json(),
 
         return {
             schema : schema,
-            context : {user : {uid : uid, password : password}}
+            context: { user: { uid: uid, password: password } } // accessible in every single resolver as the third argument
         };
     })
 );
 
-// GraphiQL est une console interactive pour faire des requêtes au schéma GraphQL
-server.use('/graphiql', /*ensureLoggedIn('/login'),*/ graphiqlExpress({endpointURL: '/graphql'}));
+// GraphiQL est une console interactive pour faire des requêtes GraphQL à la BDD
+server.use('/graphiql', 
+    /*ensureLoggedIn('/login'),*/ 
+    graphiqlExpress({endpointURL: '/graphql'})
+);
 
-// GraphQL voyager
-server.use('/voyager', graphqlVoyager({ endpointUrl: '/graphql' }));
+// GraphQL voyager affiche une représentation sous forme de graphe du schema GraphQL
+server.use('/voyager',
+    /*ensureLoggedIn('/login'),*/
+    graphqlVoyager({ endpointUrl: '/graphql' })
+);
 
 // connect-flash is middleware for flashing messages
 // used in sigma-back's admin interface (admin_view)
-- 
GitLab