fix: swagger info now gets stored on wrap (not on execution

Author: Lukas Siemon

lib/api.js

@@ -7,27 +7,24 @@ const swagger = require("./swagger");
 
 const endpoints = {};
 
-const Parser = (request, params) => {
-  endpoints[request] = params;
-  return (event) => {
-    const expectedRequestMethod = request.split(" ")[0];
-    const receivedRequestMethod = get(event, 'httpMethod');
-    if (receivedRequestMethod !== expectedRequestMethod) {
-      throw response.ApiError(`Request Method "${expectedRequestMethod}" expected.`, 400, 99004, {
-        value: receivedRequestMethod
-      });
-    }
-    let body;
-    try {
-      body = JSON.parse(get(event, 'body', '{}'));
-    } catch (e) {
-      throw response.ApiError("Invalid Json Body detected.", 400, 99001, {
-        value: get(event, 'body')
-      });
-    }
-    const eventParsed = Object.assign({}, event, { body });
-    return Promise.resolve(params.map(p => p.get(eventParsed)));
-  };
+const parse = (request, params, event) => {
+  const expectedRequestMethod = request.split(" ")[0];
+  const receivedRequestMethod = get(event, 'httpMethod');
+  if (receivedRequestMethod !== expectedRequestMethod) {
+    throw response.ApiError(`Request Method "${expectedRequestMethod}" expected.`, 400, 99004, {
+      value: receivedRequestMethod
+    });
+  }
+  let body;
+  try {
+    body = JSON.parse(get(event, 'body', '{}'));
+  } catch (e) {
+    throw response.ApiError("Invalid Json Body detected.", 400, 99001, {
+      value: get(event, 'body')
+    });
+  }
+  const eventParsed = Object.assign({}, event, { body });
+  return Promise.resolve(params.map(p => p.get(eventParsed)));
 };
 
 
@@ -62,14 +59,14 @@ module.exports = (options = {}) => {
     if (params.filter(p => p.position === 'path').some(p => request.indexOf(`{${p.name}}`) === -1)) {
       throw new Error("Path Parameter not defined in given path.");
     }
-    const parser = Parser(request, params);
+    endpoints[request] = params;
     return rollbar
       .wrap((event, context, callback, rb) => limiter
         .check(limit, get(event, 'requestContext.identity.sourceIp'))
         .catch(() => {
           throw response.ApiError("Rate limit exceeded.", 429);
         })
-        .then(() => parser(event))
+        .then(() => parse(request, params, event))
         .then(paramsOut => handler(paramsOut, context, callback, rb))
         .then(payload => generateResponse(null, payload, callback, rb))
         .catch(err => generateResponse(err, null, callback, rb)));