Skip to content

Commit 7d75507

Browse files
committed
Better console.errors everywhere
1 parent 0dcf529 commit 7d75507

File tree

8 files changed

+11
-7
lines changed

8 files changed

+11
-7
lines changed

src/handlers/batchesHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ function batchesHandler(endpoint: endpointStrings, method: 'POST' | 'GET') {
2323

2424
return tryTargetsResponse;
2525
} catch (err: any) {
26-
console.error({ message: `${endpoint} error ${err.message}` });
26+
console.error('batchesHandler error: ', err);
2727
return new Response(
2828
JSON.stringify({
2929
status: 'failure',

src/handlers/filesHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ function filesHandler(
2929

3030
return tryTargetsResponse;
3131
} catch (err: any) {
32-
console.error({ message: `${endpoint} error ${err.message}` });
32+
console.error('filesHandler error: ', err);
3333
return new Response(
3434
JSON.stringify({
3535
status: 'failure',

src/handlers/finetuneHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ async function finetuneHandler(c: Context) {
5050

5151
return tryTargetsResponse;
5252
} catch (err: any) {
53-
console.error({ message: `${endpoint} error ${err.message}` });
53+
console.error('finetuneHandler error: ', err);
5454
return new Response(
5555
JSON.stringify({
5656
status: 'failure',

src/handlers/handlerUtils.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -805,6 +805,7 @@ export async function tryTargetsRecursively(
805805
method
806806
);
807807
} catch (error: any) {
808+
console.error('tryTargetsRecursively error: ', error);
808809
// tryPost always returns a Response.
809810
// TypeError will check for all unhandled exceptions.
810811
// GatewayError will check for all handled exceptions which cannot allow the request to proceed.

src/handlers/modelResponsesHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ function modelResponsesHandler(
2626

2727
return tryTargetsResponse;
2828
} catch (err: any) {
29-
console.error({ message: `${endpoint} error ${err.message}` });
29+
console.error('modelResponsesHandler error: ', err);
3030
return new Response(
3131
JSON.stringify({
3232
status: 'failure',

src/handlers/responseHandlers.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,7 +291,7 @@ export async function afterRequestHookHandler(
291291

292292
return createHookResponse(response, responseData, hooksResult);
293293
} catch (err) {
294-
console.error(err);
294+
console.error('afterRequestHookHandler error: ', err);
295295
return response;
296296
}
297297
}

src/index.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ app.notFound((c) => c.json({ message: 'Not Found', ok: false }, 404));
109109
* Otherwise, logs the error and returns a JSON response with status code 500.
110110
*/
111111
app.onError((err, c) => {
112+
console.error('Global Error Handler: ', err);
112113
if (err instanceof HTTPException) {
113114
return err.getResponse();
114115
}

src/providers/bedrock/utils.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ export async function getAssumedRoleCredentials(
276276

277277
if (!response.ok) {
278278
const resp = await response.text();
279-
console.error({ message: resp });
279+
console.error('getAssumedRoleCredentials error: ', { message: resp });
280280
throw new Error(`HTTP error! status: ${response.status}`);
281281
}
282282

@@ -286,7 +286,9 @@ export async function getAssumedRoleCredentials(
286286
await putInCacheWithValue(env(c), cacheKey, credentials, 300); //5 minutes
287287
}
288288
} catch (error) {
289-
console.error({ message: `Error assuming role:, ${error}` });
289+
console.error('getAssumedRoleCredentials error: ', {
290+
message: `Error assuming role:, ${error}`,
291+
});
290292
}
291293
return credentials;
292294
}

0 commit comments

Comments
 (0)