Fix/fix ipfs uploading error (#39)

* fix: fix ipfs image uploading error * refactor: Refactor ipfs connector upload input structure
cubiclearn · Nov 8, 2023 · e33327e · e33327e
1 parent 79d295b
commit e33327e
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 30 deletions.
diff --git a/apps/web/pages/api/v0/course/credential.ts b/apps/web/pages/api/v0/course/credential.ts
@@ -115,21 +115,21 @@ const handlePostRequest = async (
 
     const credentialImageBuffer = fs.readFileSync(filepath)
 
-    const ipfsCredentialImageData = await IpfsConnector.upload(
-      credentialImageBuffer,
-      mimetype ?? '',
-      originalFilename ?? '',
-    )
+    const ipfsCredentialImageData = await IpfsConnector.upload({
+      fileContent: credentialImageBuffer,
+      fileName: originalFilename ?? '',
+      mimeType: mimetype ?? '',
+    })
 
-    const ipfsCredentialMetadata = await IpfsConnector.upload(
-      {
+    const ipfsCredentialMetadata = await IpfsConnector.upload({
+      fileContent: {
         name: name,
         description: description,
         image: ipfsCredentialImageData.url,
       },
-      '',
-      'data/json',
-    )
+      fileName: '',
+      mimeType: 'data/json',
+    })
 
     const credential = await prisma.credential.create({
       data: {

diff --git a/apps/web/pages/api/v0/course/metadata.ts b/apps/web/pages/api/v0/course/metadata.ts
@@ -32,26 +32,27 @@ const handlePostRequest = async (
     }
 
     const { mimetype, filepath, originalFilename } = imageFile
+    console.log(originalFilename)
     const courseImageBuffer = fs.readFileSync(filepath)
 
-    const ipfsCourseImageData = await IpfsConnector.upload(
-      courseImageBuffer,
-      mimetype ?? '',
-      originalFilename ?? '',
-    )
+    const ipfsCourseImageData = await IpfsConnector.upload({
+      fileContent: courseImageBuffer,
+      fileName: originalFilename ?? '',
+      mimeType: mimetype ?? '',
+    })
 
-    const ipfsCourseMetadata = await IpfsConnector.upload(
-      {
+    const ipfsCourseMetadata = await IpfsConnector.upload({
+      fileContent: {
         image: ipfsCourseImageData.url,
         name: name,
         description: description,
         website: website,
         'snapshot-ens': snapshotEns,
         'media-channel': mediaChannel,
       },
-      '',
-      'data/json',
-    )
+      fileName: '',
+      mimeType: 'data/json',
+    })
 
     res.status(200).json({
       status: ApiResponseStatus.success,

diff --git a/packages/ipfs/src/connectors/infura.ts b/packages/ipfs/src/connectors/infura.ts
@@ -15,11 +15,15 @@ export class InfuraIpfsConnector implements IpfsConnector {
     this.ipfsGateway = ipfsGateway
   }
 
-  public async upload(
-    fileContent: Buffer | Record<string, any>,
-    fileName: string,
-    mimeType: string,
-  ): Promise<IpfsUploadResult> {
+  public async upload({
+    fileContent,
+    fileName,
+    mimeType,
+  }: {
+    fileContent: Buffer | Record<string, any>
+    fileName: string
+    mimeType: string
+  }): Promise<IpfsUploadResult> {
     const formData = new FormData()
 
     if (fileContent instanceof Buffer) {

diff --git a/packages/ipfs/src/index.ts b/packages/ipfs/src/index.ts
@@ -9,9 +9,13 @@ export type IpfsUploadResult = {
 }
 
 export interface IpfsConnector {
-  upload(
-    fileContent: Buffer | Record<string, any>,
-    mimeType?: string,
-    fileName?: string,
-  ): Promise<IpfsUploadResult>
+  upload({
+    fileContent,
+    mimeType,
+    fileName,
+  }: {
+    fileContent: Buffer | Record<string, any>
+    mimeType?: string
+    fileName?: string
+  }): Promise<IpfsUploadResult>
 }