Skip to content

Commit

Permalink
pre-validate sessionID type for embed chats
Browse files Browse the repository at this point in the history
  • Loading branch information
timothycarambat committed Aug 27, 2024
1 parent 548da9a commit 334fd9c
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 6 deletions.
10 changes: 5 additions & 5 deletions server/models/embedChats.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ const EmbedChats = {
embed_id: Number(embedId),
response: JSON.stringify(response),
connection_information: JSON.stringify(connection_information),
session_id: sessionId,
session_id: String(sessionId),
},
});
return { chat, message: null };
Expand All @@ -36,8 +36,8 @@ const EmbedChats = {
try {
const chats = await prisma.embed_chats.findMany({
where: {
embed_id: embedId,
session_id: sessionId,
embed_id: Number(embedId),
session_id: String(sessionId),
include: true,
},
...(limit !== null ? { take: limit } : {}),
Expand All @@ -56,8 +56,8 @@ const EmbedChats = {
try {
await prisma.embed_chats.updateMany({
where: {
embed_id: embedId,
session_id: sessionId,
embed_id: Number(embedId),
session_id: String(sessionId),
},
data: {
include: false,
Expand Down
13 changes: 12 additions & 1 deletion server/utils/middleware/embedMiddleware.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
const { v4: uuidv4 } = require("uuid");
const { v4: uuidv4, validate } = require("uuid");
const { VALID_CHAT_MODE } = require("../chats/stream");
const { EmbedChats } = require("../../models/embedChats");
const { EmbedConfig } = require("../../models/embedConfig");
Expand Down Expand Up @@ -78,6 +78,17 @@ async function canRespond(request, response, next) {
}

const { sessionId, message } = reqBody(request);
if (typeof sessionId !== "string" || !validate(String(sessionId))) {
response.status(404).json({
id: uuidv4(),
type: "abort",
textResponse: null,
sources: [],
close: true,
error: "Invalid session ID.",
});
return;
}

if (!message?.length || !VALID_CHAT_MODE.includes(embed.chat_mode)) {
response.status(400).json({
Expand Down

0 comments on commit 334fd9c

Please sign in to comment.