fix: bug in setting cacheTime for object uploads

supabase · Mar 17, 2021 · 9991eef · 9991eef
1 parent a6fbdee
commit 9991eef
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/src/app.ts b/src/app.ts
@@ -17,7 +17,7 @@ const build = (opts: buildOpts = {}): FastifyInstance => {
   const app = fastify(opts)
   app.register(fastifyMultipart, {
     limits: {
-      fields: 0,
+      fields: 10,
       fileSize: 50 * 1024 * 1024,
       files: 1,
     },

diff --git a/src/routes/object/createObject.ts b/src/routes/object/createObject.ts
@@ -50,7 +50,7 @@ export default async function routes(fastify: FastifyInstance) {
       // https://github.com/fastify/fastify-multipart/issues/162
       /* @ts-expect-error: https://github.com/aws/aws-sdk-js-v3/issues/2085 */
       const cacheTime = data.fields.cacheControl?.value
-      const cacheControl: string = `max-age=${cacheTime}` ?? 'no-cache'
+      const cacheControl: string = cacheTime ? `max-age=${cacheTime}` : 'no-cache'
 
       const { bucketName } = request.params
       const objectName = request.params['*']

diff --git a/src/utils/index.ts b/src/utils/index.ts
@@ -9,7 +9,6 @@ interface jwtInterface {
 }
 
 export function getPostgrestClient(jwt: string): PostgrestClient {
-  // @todo in kps, can we just ping localhost?
   const postgrest = new PostgrestClient(postgrestURL, {
     headers: {
       apiKey: anonKey,