import * as express from 'express'
import * as multer from 'multer'
+import { HttpStatusCode } from '../../shared/models/http/http-error-codes'
+import { CONFIG } from '../initializers/config'
import { REMOTE_SCHEME } from '../initializers/constants'
-import { logger } from './logger'
-import { deleteFileAsync, generateRandomString } from './utils'
-import { extname } from 'path'
+import { getLowercaseExtension } from './core-utils'
import { isArray } from './custom-validators/misc'
-import { CONFIG } from '../initializers/config'
+import { logger } from './logger'
+import { deleteFileAndCatch, generateRandomString } from './utils'
+import { getExtFromMimetype } from './video'
function buildNSFWFilter (res?: express.Response, paramNSFW?: string) {
if (paramNSFW === 'true') return true
return null
}
-function cleanUpReqFiles (req: { files: { [fieldname: string]: Express.Multer.File[] } | Express.Multer.File[] }) {
- const files = req.files
-
- if (!files) return
+function cleanUpReqFiles (req: express.Request) {
+ const filesObject = req.files
+ if (!filesObject) return
- if (isArray(files)) {
- (files as Express.Multer.File[]).forEach(f => deleteFileAsync(f.path))
+ if (isArray(filesObject)) {
+ filesObject.forEach(f => deleteFileAndCatch(f.path))
return
}
- for (const key of Object.keys(files)) {
- const file = files[key]
+ for (const key of Object.keys(filesObject)) {
+ const files = filesObject[key]
- if (isArray(file)) file.forEach(f => deleteFileAsync(f.path))
- else deleteFileAsync(file.path)
+ files.forEach(f => deleteFileAndCatch(f.path))
}
}
}
function badRequest (req: express.Request, res: express.Response) {
- return res.type('json').status(400).end()
+ return res.type('json')
+ .status(HttpStatusCode.BAD_REQUEST_400)
+ .end()
}
function createReqFiles (
fieldNames: string[],
- mimeTypes: { [id: string]: string },
+ mimeTypes: { [id: string]: string | string[] },
destinations: { [fieldName: string]: string }
) {
const storage = multer.diskStorage({
filename: async (req, file, cb) => {
let extension: string
- const fileExtension = extname(file.originalname)
- const extensionFromMimetype = mimeTypes[file.mimetype]
+ const fileExtension = getLowercaseExtension(file.originalname)
+ const extensionFromMimetype = getExtFromMimetype(mimeTypes, file.mimetype)
// Take the file extension if we don't understand the mime type
- // We have the OGG/OGV exception too because firefox sends a bad mime type when sending an OGG file
- if (fileExtension === '.ogg' || fileExtension === '.ogv' || !extensionFromMimetype) {
+ if (!extensionFromMimetype) {
extension = fileExtension
} else {
+ // Take the first available extension for this mimetype
extension = extensionFromMimetype
}