Skip to content

Commit 7c7d785

Browse files
authored
Don't log waitpoint output when resolving (#3490)
Redact the `resolveWaitpoint` runtime log so it only emits `id` and `type` instead of the full completed waitpoint. Previously the log printed the entire waitpoint (including `output`) to stdout in production runs, which could leak sensitive payloads. The value returned by `wait.forToken()` is unchanged.
1 parent 1dfd595 commit 7c7d785

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@trigger.dev/core": patch
3+
---
4+
5+
Redact the `resolveWaitpoint` runtime log so it only emits `id` and `type` instead of the full completed waitpoint. Previously the log printed the entire waitpoint (including `output`) to stdout in production runs, which could leak sensitive payloads. The value returned by `wait.forToken()` is unchanged.

packages/core/src/v3/runtime/sharedRuntimeManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ export class SharedRuntimeManager implements RuntimeManager {
219219

220220
private resolveWaitpoint(waitpoint: CompletedWaitpoint, resolverId?: ResolverId | null): void {
221221
// This is spammy, don't make this a debug log
222-
this.log("resolveWaitpoint", waitpoint);
222+
this.log("resolveWaitpoint", { id: waitpoint.id, type: waitpoint.type });
223223

224224
if (waitpoint.type === "BATCH") {
225225
// We currently ignore these, they're not required to resume after a batch completes

0 commit comments

Comments
 (0)