Bugfix/path traversal check on chatId (#5428)
* path traversal check on chatId * Update packages/server/src/utils/createAttachment.ts Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
This commit is contained in:
parent
ec1762b10f
commit
03c1750d73
|
|
@ -27,15 +27,15 @@ export const createFileAttachment = async (req: Request) => {
|
||||||
const appServer = getRunningExpressApp()
|
const appServer = getRunningExpressApp()
|
||||||
|
|
||||||
const chatflowid = req.params.chatflowId
|
const chatflowid = req.params.chatflowId
|
||||||
|
const chatId = req.params.chatId
|
||||||
|
|
||||||
if (!chatflowid || !isValidUUID(chatflowid)) {
|
if (!chatflowid || !isValidUUID(chatflowid)) {
|
||||||
throw new InternalFlowiseError(StatusCodes.BAD_REQUEST, 'Invalid chatflowId format - must be a valid UUID')
|
throw new InternalFlowiseError(StatusCodes.BAD_REQUEST, 'Invalid chatflowId format - must be a valid UUID')
|
||||||
}
|
}
|
||||||
if (isPathTraversal(chatflowid)) {
|
if (isPathTraversal(chatflowid) || (chatId && isPathTraversal(chatId))) {
|
||||||
throw new InternalFlowiseError(StatusCodes.BAD_REQUEST, 'Invalid path characters detected')
|
throw new InternalFlowiseError(StatusCodes.BAD_REQUEST, 'Invalid path characters detected')
|
||||||
}
|
}
|
||||||
|
|
||||||
const chatId = req.params.chatId
|
|
||||||
|
|
||||||
// Validate chatflow exists and check API key
|
// Validate chatflow exists and check API key
|
||||||
const chatflow = await appServer.AppDataSource.getRepository(ChatFlow).findOneBy({
|
const chatflow = await appServer.AppDataSource.getRepository(ChatFlow).findOneBy({
|
||||||
id: chatflowid
|
id: chatflowid
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue