Skip to content

improvement(timeouts): files/base64 should use max timeouts + auth centralization #3179

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
icecrasher321 merged 8 commits into from
Feb 10, 2026
Merged

improvement(timeouts): files/base64 should use max timeouts + auth centralization #3179

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8902752
improvement(timeouts): files/base64 should use max timeouts
icecrasher321 Feb 10, 2026
0ff7e57
fix(billing): attribute cost to caller when info available"
icecrasher321 Feb 10, 2026
908f7ea
remove owner reference in billing
icecrasher321 Feb 10, 2026
b9171fa
fix auth
icecrasher321 Feb 10, 2026
c0b7ec0
update tests
icecrasher321 Feb 10, 2026
b7aa96c
fix return code
icecrasher321 Feb 10, 2026
c4d51ef
use shared types
icecrasher321 Feb 10, 2026
ab695c8
fix bugbot comment
icecrasher321 Feb 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove owner reference in billing
  • Loading branch information
@icecrasher321
icecrasher321 committed Feb 10, 2026
commit 908f7eac6b7e5fb7ac24ea9e7450816d90065a91
4 changes: 2 additions & 2 deletions apps/sim/lib/execution/preprocessing.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ export async function preprocessExecution(
}

if (!actorUserId) {
const fallbackUserId = userId || workflowRecord.userId || 'unknown'
const fallbackUserId = userId || 'unknown'
logger.warn(`[${requestId}] ${BILLING_ERROR_MESSAGES.BILLING_REQUIRED}`, {
workflowId,
workspaceId,
Expand Down Expand Up @@ -214,7 +214,7 @@ export async function preprocessExecution(
}
} catch (error) {
logger.error(`[${requestId}] Error resolving billing actor`, { error, workflowId })
const fallbackUserId = userId || workflowRecord.userId || 'unknown'
const fallbackUserId = userId || 'unknown'
await logPreprocessingError({
workflowId,
executionId,
Expand Down