feat: cors headers on error (#803)

<!--
Pull requests are squashed and merged using:
- their title as the commit message
- their description as the commit body

Having a good title and description is important for the users to get
readable changelog.
-->

<!-- 1. Explain WHAT the change is about -->

-

<!-- 2. Explain WHY the change cannot be made simpler -->

-

<!-- 3. Explain HOW users should update their code -->

#### Migration notes

...

- [ ] The change comes with new or modified tests
- [ ] Hard-to-understand functions have explanatory comments
- [ ] End-user documentation is updated to reflect the change

typegate/src/errors.ts

@@ -55,7 +55,10 @@ export class BaseError extends Error {
     return this;
   }
 
-  toResponse(graphqlFormat = true): Response {
+  toResponse(
+    headers: Headers = new Headers(),
+    graphqlFormat = true,
+  ): Response {
     const type = this.#type ?? this.constructor.name;
     logger.error(
       "{}[{}:{}]: {}",
@@ -98,7 +101,7 @@ export class BaseError extends Error {
 
     return new Response(JSON.stringify(responseObj), {
       status: this.code,
-      headers: { "Content-Type": "application/json" },
+      headers: { "Content-Type": "application/json", ...headers },
     });
   }
 }

typegate/src/services/graphql_service.ts

@@ -51,7 +51,7 @@ export async function handleGraphQL(
     content = await parseRequest(request);
   } catch (e) {
     if (e instanceof BaseError) {
-      return e.toResponse();
+      return e.toResponse(headers);
     }
     return badRequest(e.message);
   }
@@ -120,7 +120,7 @@ export async function handleGraphQL(
   } catch (e) {
     // throw e;
     if (e instanceof BaseError) {
-      return e.toResponse();
+      return e.toResponse(headers);
     }
     if (e instanceof ResolverError) {
       logger.error(`field err: ${e.message}`);

typegate/src/services/rest_service.ts

@@ -65,7 +65,7 @@ export async function handleRest(
         `query not found: ${name}`,
         404,
       ).withType("NotFound")
-        .toResponse(false);
+        .toResponse(headers, false);
     }
 
     const variables = req.method === "GET"
@@ -92,7 +92,7 @@ export async function handleRest(
   } catch (e) {
     headers.set("Content-Type", "application/json");
     if (e instanceof BaseError) {
-      return e.toResponse(false);
+      return e.toResponse(headers, false);
     }
     if (e instanceof ResolverError) {
       logger.error(`field err: ${e.message}`);