README is wrong for how to define context function

[cubabit]

The documentation says this is how you supply the function to define context:

export default startServerAndCreateLambdaHandler(
  server,
  handlers.createAPIGatewayProxyEventV2RequestHandler({
    context: async ({ event }) => {
      // Do some parsing on the event (parse JWT, cookie, auth header, etc.)
      return {
        isAuthenticated: true,
      };
    },
  }),
);

But handlers.createAPIGatewayProxyEventV2RequestHandler does not accept an argument. The correct syntax is:

export default startServerAndCreateLambdaHandler(
  server,
  handlers.createAPIGatewayProxyEventV2RequestHandler(),
  {
    context: async ({ event }) => {
      // Do some parsing on the event (parse JWT, cookie, auth header, etc.)
      return {
        isAuthenticated: true,
      };
    },
  }
);

[trevor-scheer]

Thanks @cubabit - do you want to submit a PR for the fix? I'm happy to otherwise.

[acbellini]

@trevor-scheer could you kindly fix this? I just wasted quite a bit of time on this problem... :)

[trevor-scheer]

@acbellini this is fixed now. This is a public repository, PRs to fix these kinds of issues are always welcome and preferred (and usually will get problems fixed in a much more timely manner). You can even edit files and open a PR directly in GitHub if you want to skip all of the repo setup / cloning.