Skip to content

Commit

Permalink
fix async log auth check (#1916)
Browse files Browse the repository at this point in the history
  • Loading branch information
chitalian authored May 14, 2024
1 parent 83a0c27 commit c16e6b7
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion worker/src/lib/managers/AsyncLogManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,16 @@ import { S3Client } from "../clients/S3Client";
import { RequestResponseManager } from "./RequestResponseManager";
import { KafkaProducer } from "../clients/KafkaProducer";

function mergeHeaders(x: Headers, y: Headers) {
const merged = new Headers();
for (const [key, value] of x.entries()) {
merged.set(key, value);
}
for (const [key, value] of y.entries()) {
merged.set(key, value);
}
return merged;
}
export async function logAsync(
requestWrapper: RequestWrapper,
env: Env,
Expand All @@ -37,7 +47,9 @@ export async function logAsync(

const requestHeaders = new Headers(asyncLogModel.providerRequest.meta);
const responseHeaders = new Headers(asyncLogModel.providerResponse.headers);
const heliconeHeaders = new HeliconeHeaders(requestHeaders);
const heliconeHeaders = new HeliconeHeaders(
mergeHeaders(requestHeaders, requestWrapper.headers)
);

const loggable = await dbLoggableRequestFromAsyncLogModel({
requestWrapper,
Expand Down

0 comments on commit c16e6b7

Please sign in to comment.