Merge pull request #145 from topcoder-platform/issue-144

Add support for allowAnonymous flag on endpoints

Author: jmgasper

package-lock.json

@@ -40,18 +40,20 @@ _.forEach(routes, (verbs, path) => {
       throw new Error(`${def.method} is undefined`);
     }
     const actions = [];
-    actions.push((req, res, next) => {
-      const v3jwt = _.get(req.cookies, constants.JWT_V3_NAME);
-      if (v3jwt) {
-        const decoded = jwtDecode(v3jwt);
-        req.currentUser = {
-          handle: decoded.handle.toLowerCase(),
-          roles: decoded.roles,
-        };
-      }
-      req.signature = `${def.controller}#${def.method}`;
-      next();
-    });
+    if (!def.allowAnonymous) {
+      actions.push((req, res, next) => {
+        const v3jwt = _.get(req.cookies, constants.JWT_V3_NAME);
+        if (v3jwt) {
+          const decoded = jwtDecode(v3jwt);
+          req.currentUser = {
+            handle: decoded.handle.toLowerCase(),
+            roles: decoded.roles,
+          };
+        }
+        req.signature = `${def.controller}#${def.method}`;
+        next();
+      });
+    }
     if (def.tcLogin) {
       // middleware to handle TC login
       actions.push((req, res, next) => {

src/app.js

@@ -43,13 +43,15 @@ module.exports = {
       method: 'addUserToTeam',
       allowNormalUser: true,
       tcLogin: true,
+      allowAnonymous: true,
     },
   },
   '/github/normaluser/callback': {
     get: {
       controller: 'GithubController',
       method: 'addUserToTeamCallback',
       allowNormalUser: true,
+      allowAnonymous: true,
     },
   },
 
@@ -84,13 +86,15 @@ module.exports = {
       method: 'addUserToGroup',
       allowNormalUser: true,
       tcLogin: true,
+      allowAnonymous: true,
     },
   },
   '/gitlab/normaluser/callback': {
     get: {
       controller: 'GitlabController',
       method: 'addUserToGroupCallback',
       allowNormalUser: true,
+      allowAnonymous: true,
     },
   },