From 3e0b9947fa58087cce8cd5a33d89c3209d56a8b0 Mon Sep 17 00:00:00 2001
From: Alex Andru <alex007d@gmail.com>
Date: Sat, 29 Mar 2025 02:32:49 +0100
Subject: [PATCH] refactor: update CORS configuration type in
 HttpStreamTransport

---
 src/transports/http/server.ts |  3 ++-
 src/transports/http/types.ts  | 10 ++--------
 2 files changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/transports/http/server.ts b/src/transports/http/server.ts
index f8f6ff8..b3ea2b8 100644
--- a/src/transports/http/server.ts
+++ b/src/transports/http/server.ts
@@ -20,6 +20,7 @@ import { AuthResult, DEFAULT_AUTH_ERROR } from "../../auth/types.js";
 import { logger } from "../../core/Logger.js";
 import { getRequestHeader, setResponseHeaders } from "../../utils/headers.js";
 import { DEFAULT_CORS_CONFIG } from "../sse/types.js";
+import { CORSConfig } from "../sse/types.js";
 
 function isRequest(msg: JsonRpcMessage): msg is JsonRpcRequest {
   return msg && 
@@ -79,7 +80,7 @@ export class HttpStreamTransport extends AbstractTransport {
       ...config,
       session: { ...DEFAULT_HTTP_STREAM_CONFIG.session, ...config.session },
       resumability: { ...DEFAULT_HTTP_STREAM_CONFIG.resumability, ...config.resumability },
-      cors: { ...DEFAULT_CORS_CONFIG, ...(config.cors || {}) } as Required<NonNullable<HttpStreamTransportConfig['cors']>>,
+      cors: { ...DEFAULT_CORS_CONFIG, ...(config.cors || {}) } as Required<NonNullable<CORSConfig>>,
       auth: config.auth ?? DEFAULT_HTTP_STREAM_CONFIG.auth,
       headers: config.headers ?? DEFAULT_HTTP_STREAM_CONFIG.headers,
       enableGetSse: config.enableGetSse ?? DEFAULT_HTTP_STREAM_CONFIG.enableGetSse,
diff --git a/src/transports/http/types.ts b/src/transports/http/types.ts
index a3c6dbd..aa8a2cd 100644
--- a/src/transports/http/types.ts
+++ b/src/transports/http/types.ts
@@ -99,13 +99,7 @@ export interface HttpStreamTransportConfig {
   /**
    * CORS configuration
    */
-  cors?: {
-    allowOrigin?: string;
-    allowMethods?: string;
-    allowHeaders?: string;
-    exposeHeaders?: string;
-    maxAge?: string;
-  };
+  cors?: CORSConfig;
   
   /**
    * Authentication configuration
@@ -158,7 +152,7 @@ export interface HttpStreamTransportConfig {
  * Internal configuration type with required fields
  */
 export type HttpStreamTransportConfigInternal = Required<Omit<HttpStreamTransportConfig, 'headers' | 'auth' | 'cors' | 'session' | 'resumability'>> & {
-  cors: Required<NonNullable<HttpStreamTransportConfig['cors']>>;
+  cors: Required<NonNullable<CORSConfig>>;
   session: Required<NonNullable<HttpStreamTransportConfig['session']>>;
   resumability: Required<NonNullable<HttpStreamTransportConfig['resumability']>>;
   headers?: Record<string, string>;