groove/audiobookshelf
1
0
Fork 0
mirror of https://github.com/advplyr/audiobookshelf.git synced 2026-07-07 01:41:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix keep-alive timeout and remove fs.stat() overhead from downloads

Browse source
This commit is contained in:
Trevor Renshaw 2026-05-16 09:19:01 -06:00
parent cf0a432027
commit e74dbf5f40
2 changed files with 13 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

5
server/Server.js
View file

@ -301,6 +301,11 @@ class Server {
app.disable('x-powered-by')
this.server = http.createServer(app)
// Keep connections alive for 120s so multi-file downloads don't pay a new
// TCP handshake per file. Default Node.js keepAliveTimeout is only 5s,
// which causes connections to drop mid-download for files > 5s.
this.server.keepAliveTimeout = 120000
this.server.headersTimeout = 125000
router.use(
fileUpload({

21
server/controllers/LibraryItemController.js
View file

@ -184,12 +184,10 @@ class LibraryItemController {
if (audioMimeType) {
res.setHeader('Content-Type', audioMimeType)
}
// Set Content-Length so clients can show accurate progress bars.
const fileStat = await fs.stat(libraryItemPath).catch(() => null)
if (fileStat) {
res.setHeader('Content-Length', fileStat.size)
}
await new Promise((resolve, reject) => res.download(libraryItemPath, req.libraryItem.relPath, (error) => (error ? reject(error) : resolve())))
// Use sendFile so the `send` module sets Content-Length, Accept-Ranges,
// and handles range requests natively — no extra stat() call needed.
res.setHeader('Content-Disposition', `attachment; filename="${encodeURIComponent(Path.basename(req.libraryItem.relPath))}"`)
await new Promise((resolve, reject) => res.sendFile(libraryItemPath, { dotfiles: 'allow' }, (error) => (error ? reject(error) : resolve())))
} else {
const filename = `${itemTitle}.zip`
await zipHelpers.zipDirectoryPipe(libraryItemPath, filename, res)
@ -1102,13 +1100,10 @@ class LibraryItemController {
}
try {
// Set Content-Length so clients (including the Android app) can show accurate
// download progress and avoid chunked-transfer encoding when the file size is known.
const fileStat = await fs.stat(libraryFile.metadata.path).catch(() => null)
if (fileStat) {
res.setHeader('Content-Length', fileStat.size)
}
await new Promise((resolve, reject) => res.download(libraryFile.metadata.path, libraryFile.metadata.filename, (error) => (error ? reject(error) : resolve())))
// Use sendFile so the `send` module sets Content-Length, Accept-Ranges,
// and handles range requests natively — no extra stat() call needed.
res.setHeader('Content-Disposition', `attachment; filename="${encodeURIComponent(libraryFile.metadata.filename)}"`)
await new Promise((resolve, reject) => res.sendFile(libraryFile.metadata.path, { dotfiles: 'allow' }, (error) => (error ? reject(error) : resolve())))
Logger.info(`[LibraryItemController] Downloaded file "${libraryFile.metadata.path}"`)
} catch (error) {
Logger.error(`[LibraryItemController] Failed to download file "${libraryFile.metadata.path}"`, error)