changed the RequestContext request property to the shortened req

Author: sdangol

packages/event-handler/src/rest/Router.ts

@@ -166,7 +166,7 @@ class Router {
    * ```typescript
    * const authMiddleware: Middleware = async ({params, reqCtx, next}) => {
    *   // Authentication logic
-   *   if (!isAuthenticated(reqCtx.request)) {
+   *   if (!isAuthenticated(reqCtx.req)) {
    *     return new Response('Unauthorized', { status: 401 });
    *   }
    *   await next();
@@ -215,19 +215,19 @@ class Router {
       };
     }
 
-    const request = proxyEventToWebRequest(event);
+    const req = proxyEventToWebRequest(event);
 
     const requestContext: RequestContext = {
       event,
       context,
-      request,
+      req,
       // this response should be overwritten by the handler, if it isn't
       // it means something went wrong with the middleware chain
       res: new Response('', { status: 500 }),
     };
 
     try {
-      const path = new URL(request.url).pathname as Path;
+      const path = new URL(req.url).pathname as Path;
 
       const route = this.routeRegistry.resolve(method, path);
 

packages/event-handler/src/rest/middleware/compress.ts

@@ -70,9 +70,7 @@ const compress = (options?: CompressionOptions): Middleware => {
   return async ({ reqCtx, next }) => {
     await next();
 
-    if (
-      !shouldCompress(reqCtx.request, reqCtx.res, preferredEncoding, threshold)
-    ) {
+    if (!shouldCompress(reqCtx.req, reqCtx.res, preferredEncoding, threshold)) {
       return;
     }
 

packages/event-handler/src/rest/middleware/cors.ts

@@ -97,15 +97,15 @@ export const cors = (options?: CorsOptions): Middleware => {
   };
 
   return async ({ reqCtx, next }) => {
-    const requestOrigin = reqCtx.request.headers.get('Origin');
+    const requestOrigin = reqCtx.req.headers.get('Origin');
     if (!isOriginAllowed(requestOrigin)) {
       await next();
       return;
     }
 
     // Handle preflight OPTIONS request
-    if (reqCtx.request.method === HttpVerbs.OPTIONS) {
-      if (!isValidPreflightRequest(reqCtx.request.headers)) {
+    if (reqCtx.req.method === HttpVerbs.OPTIONS) {
+      if (!isValidPreflightRequest(reqCtx.req.headers)) {
         await next();
         return;
       }

packages/event-handler/src/types/rest.ts

@@ -15,7 +15,7 @@ type ErrorResponse = {
 };
 
 type RequestContext = {
-  request: Request;
+  req: Request;
   event: APIGatewayProxyEvent;
   context: Context;
   res: Response;

packages/event-handler/tests/unit/rest/Router/basic-routing.test.ts

@@ -91,7 +91,7 @@ describe('Class: Router - Basic Routing', () => {
 
     app.get('/test', async (_params, reqCtx) => {
       return {
-        hasRequest: reqCtx.request instanceof Request,
+        hasRequest: reqCtx.req instanceof Request,
         hasEvent: reqCtx.event === testEvent,
         hasContext: reqCtx.context === context,
       };

packages/event-handler/tests/unit/rest/Router/decorators.test.ts

@@ -400,7 +400,7 @@ describe('Class: Router - Decorators', () => {
         @app.get('/test')
         public async getTest(_params: any, reqCtx: any) {
           return {
-            hasRequest: reqCtx.request instanceof Request,
+            hasRequest: reqCtx.req instanceof Request,
             hasEvent: reqCtx.event === testEvent,
             hasContext: reqCtx.context === context,
           };
@@ -435,7 +435,7 @@ describe('Class: Router - Decorators', () => {
             statusCode: HttpErrorCodes.BAD_REQUEST,
             error: 'Bad Request',
             message: error.message,
-            hasRequest: reqCtx.request instanceof Request,
+            hasRequest: reqCtx.req instanceof Request,
             hasEvent: reqCtx.event === testEvent,
             hasContext: reqCtx.context === context,
           };

packages/event-handler/tests/unit/rest/Router/error-handling.test.ts

@@ -375,7 +375,7 @@ describe('Class: Router - Error Handling', () => {
       statusCode: HttpErrorCodes.BAD_REQUEST,
       error: 'Bad Request',
       message: error.message,
-      hasRequest: reqCtx.request instanceof Request,
+      hasRequest: reqCtx.req instanceof Request,
       hasEvent: reqCtx.event === testEvent,
       hasContext: reqCtx.context === context,
     }));

packages/event-handler/tests/unit/rest/Router/middleware.test.ts

@@ -154,7 +154,7 @@ describe('Class: Router - Middleware', () => {
       expect(middlewareParams).toEqual({ id: '123' });
       expect(middlewareOptions?.event).toBe(testEvent);
       expect(middlewareOptions?.context).toBe(context);
-      expect(middlewareOptions?.request).toBeInstanceOf(Request);
+      expect(middlewareOptions?.req).toBeInstanceOf(Request);
     });
 
     it('returns error response when next() is called multiple times', async () => {