Merge pull request #479 from brc-dd/master-1

fix: enable cors for api routes

src/functions/api.ts

@@ -3,7 +3,8 @@ import { getPRInfo } from "../queries/pr-query";
 const headers = {
     "Content-Type": "text/json",
     "Access-Control-Allow-Methods": "GET",
-    "Access-Control-Allow-Origin": "*",
+    "Access-Control-Allow-Origin": "https://www.typescriptlang.org",
+    "Vary": "Origin"
 };
 const notFound = (reason: string) => ({
     headers,
@@ -27,7 +28,7 @@ export async function httpTrigger(request: HttpRequest, context: InvocationConte
     // Extract the JSON from the comment
     const jsonText = welcomeComment.body.replace(/^[^]*```json\n([^]*)\n```[^]*$/, "$1");
     const response = { title: prInfo.title, ...JSON.parse(jsonText) };
-    return { status: 200, body: JSON.stringify(response) };
+    return { headers, status: 200, body: JSON.stringify(response) };
 }
 // Allow all others to access this, we can
 // tighten this down to the TS URLs if the route is abused

src/run.ts

@@ -94,7 +94,7 @@ const start = async function () {
             console.error(`  No PR with this number exists, (${JSON.stringify(info)})`);
             continue;
         }
-        let state: BotResult | undefined
+        let state: BotResult | undefined;
         try {
             state = await deriveStateForPR(prInfo);
         }