Mirrors/audiobookshelf
Mirrors/audiobookshelf
1
0
Fork 0
mirror of https://github.com/advplyr/audiobookshelf.git synced 2025-11-10 01:19:37 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix codeQL error: Return json error object

Browse Source
This commit is contained in:
mikiher 2025-10-21 11:00:29 +03:00
parent fd593caafc
commit 57c7b123f0
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
server/controllers/SearchController.js
View File

@ -98,9 +98,9 @@ class SearchController {
} catch (error) { } catch (error) {
Logger.error(`[SearchController] findBooks: ${error.message}`) Logger.error(`[SearchController] findBooks: ${error.message}`)
if (error instanceof ValidationError || error instanceof NotFoundError) { if (error instanceof ValidationError || error instanceof NotFoundError) {
return res.status(error.status).send(error.message) return res.status(error.status).json({ error: error.message })
} }
return res.status(500).send('Internal server error') return res.status(500).json({ error: 'Internal server error' })
} }
} }
@ -125,9 +125,9 @@ class SearchController {
} catch (error) { } catch (error) {
Logger.error(`[SearchController] findCovers: ${error.message}`) Logger.error(`[SearchController] findCovers: ${error.message}`)
if (error instanceof ValidationError) { if (error instanceof ValidationError) {
return res.status(error.status).send(error.message) return res.status(error.status).json({ error: error.message })
} }
return res.status(500).send('Internal server error') return res.status(500).json({ error: 'Internal server error' })
} }
} }
@ -149,9 +149,9 @@ class SearchController {
} catch (error) { } catch (error) {
Logger.error(`[SearchController] findPodcasts: ${error.message}`) Logger.error(`[SearchController] findPodcasts: ${error.message}`)
if (error instanceof ValidationError) { if (error instanceof ValidationError) {
return res.status(error.status).send(error.message) return res.status(error.status).json({ error: error.message })
} }
return res.status(500).send('Internal server error') return res.status(500).json({ error: 'Internal server error' })
} }
} }
@ -171,9 +171,9 @@ class SearchController {
} catch (error) { } catch (error) {
Logger.error(`[SearchController] findAuthor: ${error.message}`) Logger.error(`[SearchController] findAuthor: ${error.message}`)
if (error instanceof ValidationError) { if (error instanceof ValidationError) {
return res.status(error.status).send(error.message) return res.status(error.status).json({ error: error.message })
} }
return res.status(500).send('Internal server error') return res.status(500).json({ error: 'Internal server error' })
} }
} }
@ -206,7 +206,7 @@ class SearchController {
return res.json({ error: 'Invalid region', stringKey: 'MessageInvalidRegion' }) return res.json({ error: 'Invalid region', stringKey: 'MessageInvalidRegion' })
} }
} }
return res.status(500).send('Internal server error') return res.status(500).json({ error: 'Internal server error' })
} }
} }