fixed bug with minio.ts

- the provider can't be meaningfully initialized if this UploadProvider is not configured

Signed-off-by: Philip Molares <philip.molares@udo.edu>
Signed-off-by: David Mehren <dmehren1@gmail.com>

lib/web/imageRouter/minio.ts

@@ -1,22 +1,25 @@
 import path from 'path'
 import fs from 'fs'
-import Minio from 'minio'
+import { Client } from 'minio'
 
 import { config } from '../../config'
 import { getImageMimeType } from '../../utils'
 import { logger } from '../../logger'
 import { UploadProvider } from './index'
 
-const minioClient = new Minio.Client({
+let MinioUploadProvider: UploadProvider
+
+if (config.minio.endPoint !== undefined) {
+  const minioClient = new Client({
     endPoint: config.minio.endPoint,
     port: config.minio.port,
     useSSL: config.minio.secure,
     accessKey: config.minio.accessKey,
     secretKey: config.minio.secretKey
-})
+  })
 
-const MinioUploadProvider: UploadProvider = {
-  uploadImage: (imagePath, callback) => {
+  MinioUploadProvider = {
+    uploadImage: (imagePath, callback): void => {
       if (!imagePath) {
         callback(new Error('Image path is missing or wrong'), undefined)
         return
@@ -36,7 +39,11 @@ const MinioUploadProvider: UploadProvider = {
         const key = path.join('uploads', path.basename(imagePath))
         const protocol = config.minio.secure ? 'https' : 'http'
 
-      minioClient.putObject(config.s3bucket, key, buffer, buffer.length, getImageMimeType(imagePath), function (err, _) {
+        const metaData = {
+          ContentType: getImageMimeType(imagePath)
+        }
+
+        minioClient.putObject(config.s3bucket, key, buffer, buffer.length, metaData, function (err, _) {
           if (err) {
             callback(new Error(err.message), undefined)
             return
@@ -47,6 +54,7 @@ const MinioUploadProvider: UploadProvider = {
         })
       })
     }
+  }
 }
 
 export { MinioUploadProvider }