old_contents: 'use strict'
  const fs = require('fs')
  const path = require('path')
  
  const config = require('../../config')
  const {getImageMimeType} = require('../../utils')
  const logger = require('../../logger')
  
  const Minio = require('minio')
  const minioClient = new Minio.Client({
    endPoint: config.minio.endPoint,
    port: config.minio.port,
    secure: config.minio.secure,
    accessKey: config.minio.accessKey,
    secretKey: config.minio.secretKey
  })
  
  exports.uploadImage = function (imagePath, callback) {
    if (!imagePath || typeof imagePath !== 'string') {
      callback(new Error('Image path is missing or wrong'), null)
      return
    }
  
    if (!callback || typeof callback !== 'function') {
      logger.error('Callback has to be a function')
      return
    }
  
    fs.readFile(imagePath, function (err, buffer) {
      if (err) {
        callback(new Error(err), null)
        return
      }
  
      let key = path.join('uploads', path.basename(imagePath))
      let protocol = config.minio.secure ? 'https' : 'http'
  
      minioClient.putObject(config.s3bucket, key, buffer, buffer.size, getImageMimeType(imagePath), function (err, data) {
        if (err) {
          callback(new Error(err), null)
          return
        }
        let hidePort = (protocol === 'http' && config.minio.port === 80) || (protocol === 'https' && config.minio.port === 443)
        let urlPort = hidePort ? '' : `:${config.minio.port}`
        callback(null, `${protocol}://${config.minio.endPoint}${urlPort}/${config.s3bucket}/${key}`)
      })
    })
  }
  
new_contents: 'use strict'
  const fs = require('fs')
  const path = require('path')
  
  const config = require('../../config')
  const {getImageMimeType} = require('../../utils')
  const logger = require('../../logger')
  
  const Minio = require('minio')
  const minioClient = new Minio.Client({
    endPoint: config.minio.endPoint,
    port: config.minio.port,
    secure: config.minio.secure,
    accessKey: config.minio.accessKey,
    secretKey: config.minio.secretKey
  })
  
  exports.uploadImage = function (imagePath, callback) {
    if (!imagePath || typeof imagePath !== 'string') {
      callback(new Error('Image path is missing or wrong'), null)
      return
    }
  
    if (!callback || typeof callback !== 'function') {
      logger.error('Callback has to be a function')
      return
    }
  
    fs.readFile(imagePath, function (err, buffer) {
      if (err) {
        callback(new Error(err), null)
        return
      }
  
      let key = path.join('uploads', path.basename(imagePath))
      let protocol = config.minio.secure ? 'https' : 'http'
  
      minioClient.putObject(config.s3bucket, key, buffer, buffer.size, getImageMimeType(imagePath), function (err, data) {
        if (err) {
          callback(new Error(err), null)
          return
        }
        let hidePort = [80, 443].includes(config.minio.port)
        let urlPort = hidePort ? '' : `:${config.minio.port}`
        callback(null, `${protocol}://${config.minio.endPoint}${urlPort}/${config.s3bucket}/${key}`)
      })
    })
  }
  
subject: Remove checks for protocol before removing port