fix(blob): handle pdf type correctly in ensureBlob (#392)

Yizack · web-flow · commit c6bddbb0baca · 2024-12-12T10:20:50.000+01:00
diff --git a/src/runtime/blob/server/utils/blob.ts b/src/runtime/blob/server/utils/blob.ts
@@ -717,7 +717,11 @@ export function ensureBlob(blob: Blob, options: BlobEnsureOptions = {}) {
     }
   }
   const blobShortType = blob.type.split('/')[0]
-  if (options.types?.length && !options.types.includes(blob.type as BlobType) && !options.types.includes(blobShortType as BlobType)) {
+  if (options.types?.length
+    && !options.types.includes(blob.type as BlobType)
+    && !options.types.includes(blobShortType as BlobType)
+    && !(options.types.includes('pdf' as BlobType) && blob.type === 'application/pdf')
+  ) {
     throw createError({
       statusCode: 400,
       message: `File type is invalid, must be: ${options.types.join(', ')}`

Original file line number	Diff line number	Diff line change
`@@ -717,7 +717,11 @@ export function ensureBlob(blob: Blob, options: BlobEnsureOptions = {}) {`
`717`	`717`	`}`
`718`	`718`	`}`
`719`	`719`	`const blobShortType = blob.type.split('/')[0]`
`720`		`- if (options.types?.length && !options.types.includes(blob.type as BlobType) && !options.types.includes(blobShortType as BlobType)) {`
	`720`	`+ if (options.types?.length`
	`721`	`+ && !options.types.includes(blob.type as BlobType)`
	`722`	`+ && !options.types.includes(blobShortType as BlobType)`
	`723`	`+ && !(options.types.includes('pdf' as BlobType) && blob.type === 'application/pdf')`
	`724`	`+ ) {`
`721`	`725`	`throw createError({`
`722`	`726`	`statusCode: 400,`
`723`	`727`	message: `File type is invalid, must be: ${options.types.join(', ')}`