refactor: remove unnecessary next() calls in access token parsers

apps/app/src/server/middlewares/access-token-parser/access-token.ts

@@ -15,7 +15,7 @@ export const parserForAccessToken = (scopes: Scope[]) => {
 
     const accessToken = req.query.access_token ?? req.body.access_token;
     if (accessToken == null || typeof accessToken !== 'string') {
-      return next();
+      return;
     }
     if (scopes == null || scopes.length === 0) {
       logger.debug('scopes is empty');
@@ -43,7 +43,7 @@ export const parserForAccessToken = (scopes: Scope[]) => {
     }
 
     logger.debug('Access token parsed.');
-    return next();
+    return;
 
   };
 };

apps/app/src/server/middlewares/access-token-parser/api-token.ts

@@ -13,7 +13,7 @@ const logger = loggerFactory('growi:middleware:access-token-parser:api-token');
 export const parserForApiToken = async(req: AccessTokenParserReq, res: Response, next: NextFunction): Promise<void> => {
   const accessToken = req.query.access_token ?? req.body.access_token;
   if (accessToken == null || typeof accessToken !== 'string') {
-    return next();
+    return;
   }
 
   logger.debug('accessToken is', accessToken);
@@ -31,5 +31,5 @@ export const parserForApiToken = async(req: AccessTokenParserReq, res: Response,
   }
 
   logger.debug('Access token parsed.');
-  return next();
+  return;
 };