fix: switch to use mutation and perform logging

opengovsg · Aug 6, 2024 · de95e9b · de95e9b
1 parent c9016dd
commit de95e9b
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/apps/studio/src/server/modules/asset/asset.router.ts b/apps/studio/src/server/modules/asset/asset.router.ts
@@ -5,7 +5,7 @@ import { getFileKey, getPresignedPutUrl } from "./asset.service"
 export const assetRouter = router({
   getPresignedPutUrl: protectedProcedure
     .input(getPresignedPutUrlSchema)
-    .query(async ({ input: { siteId, fileName } }) => {
+    .mutation(async ({ ctx, input: { siteId, fileName } }) => {
       const fileKey = getFileKey({
         siteId,
         fileName,
@@ -15,6 +15,16 @@ export const assetRouter = router({
         key: fileKey,
       })
 
+      ctx.logger.info(
+        `Generated presigned PUT URL for ${fileKey} for site ${siteId}`,
+        {
+          userId: ctx.session?.userId,
+          siteId,
+          fileName,
+          fileKey,
+        },
+      )
+
       return {
         fileKey,
         presignedPutUrl,