Merge pull request #1013 from constructive-io/devin/1776595653-fix-upload-requests-size

fix: remove non-existent 'size' column from upload_requests INSERTs

Author: pyramation

graphile/graphile-presigned-url-plugin/src/plugin.ts

@@ -318,9 +318,9 @@ export function createPresignedUrlPlugin(
                   // Track the dedup request
                   await txClient.query({
                     text: `INSERT INTO ${storageConfig.uploadRequestsQualifiedName}
-                     (file_id, bucket_id, key, content_type, content_hash, size, status, expires_at)
-                     VALUES ($1, $2, $3, $4, $5, $6, 'confirmed', NOW())`,
-                    values: [existingFile.id, bucket.id, s3Key, contentType, contentHash, size],
+                     (file_id, bucket_id, key, content_type, content_hash, status, expires_at)
+                     VALUES ($1, $2, $3, $4, $5, 'confirmed', NOW())`,
+                    values: [existingFile.id, bucket.id, s3Key, contentType, contentHash],
                   });
 
                   return {
@@ -385,9 +385,9 @@ export function createPresignedUrlPlugin(
                 // --- Track the upload request ---
                 await txClient.query({
                   text: `INSERT INTO ${storageConfig.uploadRequestsQualifiedName}
-                   (file_id, bucket_id, key, content_type, content_hash, size, status, expires_at)
-                   VALUES ($1, $2, $3, $4, $5, $6, 'issued', $7)`,
-                  values: [fileId, bucket.id, s3Key, contentType, contentHash, size, expiresAt],
+                   (file_id, bucket_id, key, content_type, content_hash, status, expires_at)
+                   VALUES ($1, $2, $3, $4, $5, 'issued', $6)`,
+                  values: [fileId, bucket.id, s3Key, contentType, contentHash, expiresAt],
                 });
 
                 return {

graphql/server-test/__fixtures__/seed/simple-seed-storage/schema.sql

@@ -56,14 +56,16 @@ COMMENT ON TABLE "simple-storage-public".files IS E'@storageFiles\nStorage files
 CREATE TABLE IF NOT EXISTS "simple-storage-public".upload_requests (
   id uuid PRIMARY KEY DEFAULT uuid_generate_v4(),
   file_id uuid NOT NULL REFERENCES "simple-storage-public".files(id),
+  actor_id uuid,
   bucket_id uuid NOT NULL REFERENCES "simple-storage-public".buckets(id),
   key text NOT NULL,
   content_type text NOT NULL,
   content_hash text,
-  size bigint,
   status text NOT NULL DEFAULT 'issued',
   expires_at timestamptz,
   confirmed_at timestamptz,
+  ip_address inet,
+  user_agent text,
   created_at timestamptz DEFAULT now(),
   updated_at timestamptz DEFAULT now()
 );