Skip to content

Commit 04a288c

Browse files
authored
[opencode] VULN-117372 Fix SQL injection vulnerability in durable-chat-template (#958)
1 parent 30d1642 commit 04a288c

File tree

1 file changed

+8
-7
lines changed
  • durable-chat-template/src/server

1 file changed

+8
-7
lines changed

durable-chat-template/src/server/index.ts

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -54,14 +54,15 @@ export class Chat extends Server<Env> {
5454
this.messages.push(message);
5555
}
5656

57+
// Use parameterized queries to prevent SQL injection
5758
this.ctx.storage.sql.exec(
58-
`INSERT INTO messages (id, user, role, content) VALUES ('${
59-
message.id
60-
}', '${message.user}', '${message.role}', ${JSON.stringify(
61-
message.content,
62-
)}) ON CONFLICT (id) DO UPDATE SET content = ${JSON.stringify(
63-
message.content,
64-
)}`,
59+
`INSERT INTO messages (id, user, role, content) VALUES (?, ?, ?, ?)
60+
ON CONFLICT (id) DO UPDATE SET content = ?`,
61+
message.id,
62+
message.user,
63+
message.role,
64+
message.content,
65+
message.content,
6566
);
6667
}
6768

0 commit comments

Comments
 (0)