mirror of
https://github.com/freedbygrace/DynamoDNS.git
synced 2026-08-24 08:16:27 +00:00
Fix error 500 when loading webhook logs by adding error handling to database queries.
Replit-Commit-Author: Agent Replit-Commit-Session-Id: 9111ef36-26c8-4085-84ca-a35dc1fec1b5 Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/7083d608-d6d3-4a6a-9a27-6286c5109627/72f89b52-54a7-4cd6-a075-53a45d62be60.jpg
This commit is contained in:
@@ -272,8 +272,13 @@ export class DatabaseStorage implements IStorage {
|
||||
|
||||
// Webhook management
|
||||
async getWebhook(id: string): Promise<Webhook | undefined> {
|
||||
const [webhook] = await db.select().from(webhooks).where(eq(webhooks.id, id));
|
||||
return webhook;
|
||||
try {
|
||||
const [webhook] = await db.select().from(webhooks).where(eq(webhooks.id, id));
|
||||
return webhook;
|
||||
} catch (error) {
|
||||
console.error("Error fetching webhook:", error);
|
||||
return undefined;
|
||||
}
|
||||
}
|
||||
|
||||
async getWebhooksByOrganization(organizationId: string): Promise<Webhook[]> {
|
||||
@@ -372,15 +377,42 @@ export class DatabaseStorage implements IStorage {
|
||||
}
|
||||
|
||||
async getWebhookDeliveryLog(id: string): Promise<WebhookDeliveryLog | undefined> {
|
||||
const [log] = await db.select().from(webhookDeliveryLogs).where(eq(webhookDeliveryLogs.id, id));
|
||||
return log;
|
||||
try {
|
||||
const [log] = await db.select().from(webhookDeliveryLogs).where(eq(webhookDeliveryLogs.id, id));
|
||||
return log;
|
||||
} catch (error) {
|
||||
console.error("Error fetching webhook delivery log:", error);
|
||||
return undefined;
|
||||
}
|
||||
}
|
||||
|
||||
async getWebhookDeliveryLogsByWebhook(webhookId: string): Promise<WebhookDeliveryLog[]> {
|
||||
return await db.select()
|
||||
try {
|
||||
// Use a specific select list to avoid potential schema issues
|
||||
// Only select columns that we know exist in the database
|
||||
const logs = await db.select({
|
||||
id: webhookDeliveryLogs.id,
|
||||
webhookId: webhookDeliveryLogs.webhookId,
|
||||
status: webhookDeliveryLogs.status,
|
||||
statusCode: webhookDeliveryLogs.statusCode,
|
||||
message: webhookDeliveryLogs.message,
|
||||
payload: webhookDeliveryLogs.payload,
|
||||
responseBody: webhookDeliveryLogs.responseBody,
|
||||
retryCount: webhookDeliveryLogs.retryCount,
|
||||
createdAt: webhookDeliveryLogs.createdAt,
|
||||
signature: webhookDeliveryLogs.signature,
|
||||
// Use null for potentially missing fields
|
||||
event: sql`NULL as event`
|
||||
})
|
||||
.from(webhookDeliveryLogs)
|
||||
.where(eq(webhookDeliveryLogs.webhookId, webhookId))
|
||||
.orderBy(desc(webhookDeliveryLogs.createdAt));
|
||||
|
||||
return logs;
|
||||
} catch (error) {
|
||||
console.error("Error fetching webhook delivery logs:", error);
|
||||
return []; // Return empty array on error
|
||||
}
|
||||
}
|
||||
|
||||
// DNS Metrics
|
||||
|
||||
Reference in New Issue
Block a user