Basic request log working

server/private/routers/auditLogs/exportActionAuditLog.ts

@@ -24,24 +24,8 @@ import { fromError } from "zod-validation-error";
 import { QueryActionAuditLogResponse } from "@server/routers/auditLogs/types";
 import response from "@server/lib/response";
 import logger from "@server/logger";
-import { queryAccessAuditLogsParams, queryAccessAuditLogsQuery, querySites } from "./queryActionAuditLog";
-
-function generateCSV(data: any[]): string {
-    if (data.length === 0) {
-        return "orgId,action,actorType,timestamp,actor\n";
-    }
-    
-    const headers = Object.keys(data[0]).join(",");
-    const rows = data.map(row => 
-        Object.values(row).map(value => 
-            typeof value === 'string' && value.includes(',') 
-                ? `"${value.replace(/"/g, '""')}"` 
-                : value
-        ).join(",")
-    );
-    
-    return [headers, ...rows].join("\n");
-}
+import { queryActionAuditLogsParams, queryActionAuditLogsQuery, querySites } from "./queryActionAuditLog";
+import { generateCSV } from "@server/routers/auditLogs/generateCSV";
 
 registry.registerPath({
     method: "get",
@@ -49,19 +33,19 @@ registry.registerPath({
     description: "Export the action audit log for an organization as CSV",
     tags: [OpenAPITags.Org],
     request: {
-        query: queryAccessAuditLogsQuery,
-        params: queryAccessAuditLogsParams
+        query: queryActionAuditLogsQuery,
+        params: queryActionAuditLogsParams
     },
     responses: {}
 });
 
-export async function exportAccessAuditLogs(
+export async function exportActionAuditLogs(
     req: Request,
     res: Response,
     next: NextFunction
 ): Promise<any> {
     try {
-        const parsedQuery = queryAccessAuditLogsQuery.safeParse(req.query);
+        const parsedQuery = queryActionAuditLogsQuery.safeParse(req.query);
         if (!parsedQuery.success) {
             return next(
                 createHttpError(
@@ -72,7 +56,7 @@ export async function exportAccessAuditLogs(
         }
         const { timeStart, timeEnd, limit, offset } = parsedQuery.data;
 
-        const parsedParams = queryAccessAuditLogsParams.safeParse(req.params);
+        const parsedParams = queryActionAuditLogsParams.safeParse(req.params);
         if (!parsedParams.success) {
             return next(
                 createHttpError(
@@ -90,7 +74,7 @@ export async function exportAccessAuditLogs(
         const csvData = generateCSV(log);
         
         res.setHeader('Content-Type', 'text/csv');
-        res.setHeader('Content-Disposition', `attachment; filename="audit-logs-${orgId}-${Date.now()}.csv"`);
+        res.setHeader('Content-Disposition', `attachment; filename="action-audit-logs-${orgId}-${Date.now()}.csv"`);
         
         return res.send(csvData);
     } catch (error) {

server/private/routers/auditLogs/queryActionAuditLog.ts

@@ -25,7 +25,7 @@ import { QueryActionAuditLogResponse } from "@server/routers/auditLogs/types";
 import response from "@server/lib/response";
 import logger from "@server/logger";
 
-export const queryAccessAuditLogsQuery = z.object({
+export const queryActionAuditLogsQuery = z.object({
     // iso string just validate its a parseable date
     timeStart: z
         .string()
@@ -55,7 +55,7 @@ export const queryAccessAuditLogsQuery = z.object({
         .pipe(z.number().int().nonnegative())
 });
 
-export const queryAccessAuditLogsParams = z.object({
+export const queryActionAuditLogsParams = z.object({
     orgId: z.string()
 });
 
@@ -100,19 +100,19 @@ registry.registerPath({
     description: "Query the action audit log for an organization",
     tags: [OpenAPITags.Org],
     request: {
-        query: queryAccessAuditLogsQuery,
-        params: queryAccessAuditLogsParams
+        query: queryActionAuditLogsQuery,
+        params: queryActionAuditLogsParams
     },
     responses: {}
 });
 
-export async function queryAccessAuditLogs(
+export async function queryActionAuditLogs(
     req: Request,
     res: Response,
     next: NextFunction
 ): Promise<any> {
     try {
-        const parsedQuery = queryAccessAuditLogsQuery.safeParse(req.query);
+        const parsedQuery = queryActionAuditLogsQuery.safeParse(req.query);
         if (!parsedQuery.success) {
             return next(
                 createHttpError(
@@ -123,7 +123,7 @@ export async function queryAccessAuditLogs(
         }
         const { timeStart, timeEnd, limit, offset } = parsedQuery.data;
 
-        const parsedParams = queryAccessAuditLogsParams.safeParse(req.params);
+        const parsedParams = queryActionAuditLogsParams.safeParse(req.params);
         if (!parsedParams.success) {
             return next(
                 createHttpError(

server/private/routers/external.ts

@@ -348,11 +348,11 @@ authenticated.post(
 
 authenticated.get(
     "/org/:orgId/logs/action",
-    logs.queryAccessAuditLogs 
+    logs.queryActionAuditLogs 
 )
 
 
 authenticated.get(
     "/org/:orgId/logs/action/export",
-    logs.exportAccessAuditLogs
+    logs.exportActionAuditLogs
 )