fix: pattern exclude when no request url

added support for request-less invocations (Eg event handler controllers, kafka) (#11)

Author: itaywol

src/interceptors/request.interceptor.spec.ts

@@ -64,6 +64,22 @@ describe('RequestInterceptor', () => {
       );
     });
 
+    it('should handle request with empty request context', async () => {
+      const emptyRequestContext = { };
+      (mockContext.switchToHttp().getRequest as jest.Mock).mockReturnValue(emptyRequestContext);
+      
+      const observable = await interceptor.intercept(mockContext, mockCallHandler);
+      await lastValueFrom(observable);
+
+      expect(ContextLogger.updateContext).toHaveBeenCalledWith(
+        expect.objectContaining({
+          requestMethod: undefined,
+          requestUrl: undefined
+        })
+      );
+      expect(mockLogger.debug).toHaveBeenCalled();
+    });
+
     it('should skip context and logging for excluded routes', async () => {
       mockRequest.url = '/health';
 

src/interceptors/request.interceptor.ts

@@ -24,10 +24,16 @@ export class RequestInterceptor implements NestInterceptor {
     next: CallHandler
   ): Promise<Observable<any>> {
     const request = context.switchToHttp().getRequest();
-    if (this.options.exclude?.some(pattern => request.url.indexOf(pattern) === 0)) {
+
+    if (
+      request.url &&
+      this.options.exclude?.some(
+        (pattern) => request.url.indexOf(pattern) === 0
+      )
+    ) {
       return next.handle();
     }
-    
+
     const startTime = new Date();
 
     // Base context