Merge pull request #89 from karthik2804/fix/resp_headers_ts

make headers optional for http response

Author: karthik2804

crates/spin-js-engine/src/js_sdk/modules/spinSdk.ts

@@ -19,7 +19,7 @@ interface HttpRequest extends BaseHttpRequest {
 
 interface HttpResponse {
     status: number
-    headers: Record<string, string>
+    headers?: Record<string, string>
     body?: ArrayBuffer
 }
 
@@ -74,7 +74,7 @@ function fetch(uri: string, options?: FetchOptions) {
     return Promise.resolve({
         status,
         headers: {
-            entries: () => Object.entries(headers)
+            entries: () => Object.entries(headers || {})
         },
         arrayBuffer: () => Promise.resolve(body),
         ok: (status > 199 && status < 300),

types/lib/modules/spinSdk.d.ts

@@ -13,7 +13,7 @@ interface HttpRequest extends BaseHttpRequest {
 }
 interface HttpResponse {
     status: number;
-    headers: Record<string, string>;
+    headers?: Record<string, string>;
     body?: ArrayBuffer;
 }
 declare type HandleRequest = (request: HttpRequest) => Promise<HttpResponse>;