mirrors/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-11-15 20:52:01 +03:00
Code Issues Packages Projects Releases Wiki Activity

chore(core): redact params in database query errors (#55621)

Browse Source
This commit is contained in:
Aiden Cline
2026-10-15 01:04:07 -05:00
committed by GitHub
Unverified
parent 7b5f1bd9e1
commit a8fc567c5c
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/core/src/database/drizzle/sqlite-core/effect/session.ts
+1 -1
View File
@@ -277,7 +277,7 @@ export class SQLiteEffectPreparedQuery<
}
assertUnreachable(cacheStrat)
}).pipe(Effect.mapError((e) => new EffectDrizzleQueryError({ query: queryString, params, cause: Cause.fail(e) })))
}).pipe(Effect.mapError((e) => new EffectDrizzleQueryError({ query: queryString, params: params.map(() => "[redacted]"), cause: Cause.fail(e) })))
}
getQuery(): Query {
packages/core/test/database-drizzle.test.ts
+2 -2
View File
@@ -48,7 +48,7 @@ test("selects rows through Effect-yieldable query builders", async () => {
)
})
test("maps query failures with query, params, and cause", async () => {
test("maps query failures with query, redacted params, and cause", async () => {
await run(
Effect.gen(function* () {
const db = yield* EffectDrizzleSqlite.makeWithDefaults()
@@ -56,7 +56,7 @@ test("maps query failures with query, params, and cause", async () => {
expect(error).toBeInstanceOf(EffectDrizzleQueryError)
expect(error.query).toBe("select * from missing_table where id = ?")
expect(error.params).toEqual([42])
expect(error.params).toEqual(["[redacted]"])
expect(Cause.isCause(error.cause)).toBe(true)
if (!Cause.isCause(error.cause)) return
expect(error.cause.reasons[0]?._tag).toBe("Fail")