mirror of
https://github.com/overleaf/overleaf.git
synced 2024-11-29 06:03:42 -05:00
0db4a17a14
the fs-extra method has a HUGE overhead of JS code and also syscalls for no particular benefit in this case: just copy the tiny.pdf file. Here is an overview of the major operations: paths are relative to https://github.com/jprichardson/node-fs-extra/blob/1.0.0 We start in /lib/copy/copy.js - sys: check that the source file exists - sys: check that the source file has an existing parent directory?! Continue in /lib/copy/ncp.js - sys: more stat calls on both source and dest to determine permissions - read/write streams to pipe the file content through the process - sys: chmod on the destination to match the source permissions What we actually need is a call to the binding and let node/the os figure out the best way to copy the contents. Signed-off-by: Jakob Ackermann <das7pad@outlook.com>
69 lines
1.7 KiB
JavaScript
69 lines
1.7 KiB
JavaScript
const fs = require('fs')
|
|
const path = require('path')
|
|
const Settings = require('settings-sharelatex')
|
|
const streamBuffers = require('stream-buffers')
|
|
const { promisify } = require('util')
|
|
const Stream = require('stream')
|
|
|
|
const pipeline = promisify(Stream.pipeline)
|
|
const fsCopy = promisify(fs.copyFile)
|
|
const fsUnlink = promisify(fs.unlink)
|
|
|
|
const { HealthCheckError } = require('./Errors')
|
|
const FileConverter = require('./FileConverter').promises
|
|
const FileHandler = require('./FileHandler').promises
|
|
|
|
async function checkCanGetFiles() {
|
|
if (!Settings.health_check) {
|
|
return
|
|
}
|
|
|
|
const projectId = Settings.health_check.project_id
|
|
const fileId = Settings.health_check.file_id
|
|
const key = `${projectId}/${fileId}`
|
|
const bucket = Settings.filestore.stores.user_files
|
|
|
|
const buffer = new streamBuffers.WritableStreamBuffer({
|
|
initialSize: 100
|
|
})
|
|
|
|
const sourceStream = await FileHandler.getFile(bucket, key, {})
|
|
try {
|
|
await pipeline(sourceStream, buffer)
|
|
} catch (err) {
|
|
throw new HealthCheckError('failed to get health-check file').withCause(err)
|
|
}
|
|
|
|
if (!buffer.size()) {
|
|
throw new HealthCheckError('no bytes written to download stream')
|
|
}
|
|
}
|
|
|
|
async function checkFileConvert() {
|
|
if (!Settings.enableConversions) {
|
|
return
|
|
}
|
|
|
|
const imgPath = path.join(Settings.path.uploadFolder, '/tiny.pdf')
|
|
|
|
let resultPath
|
|
try {
|
|
await fsCopy('./tiny.pdf', imgPath)
|
|
resultPath = await FileConverter.thumbnail(imgPath)
|
|
} finally {
|
|
if (resultPath) {
|
|
await fsUnlink(resultPath)
|
|
}
|
|
await fsUnlink(imgPath)
|
|
}
|
|
}
|
|
|
|
module.exports = {
|
|
check(req, res, next) {
|
|
Promise.all([checkCanGetFiles(), checkFileConvert()])
|
|
.then(() => res.sendStatus(200))
|
|
.catch(err => {
|
|
next(err)
|
|
})
|
|
}
|
|
}
|