Skip to content

fix(client): apply timeout to score ingestion - #887

Draft
sidsri14 wants to merge 1 commit into
langfuse:mainfrom
sidsri14:fix/score-ingestion-timeout
Draft

fix(client): apply timeout to score ingestion#887
sidsri14 wants to merge 1 commit into
langfuse:mainfrom
sidsri14:fix/score-ingestion-timeout

Conversation

@sidsri14

Copy link
Copy Markdown

Problem

Score ingestion ignored LANGFUSE_TIMEOUT and LangfuseClient({ timeout }) because ScoreManager did not receive the client's resolved timeout. Generated ingestion requests therefore used their 60-second fallback during flush and shutdown.

Fixes #883.

Changes

  • Pass the resolved client timeout into ScoreManager.
  • Apply it to score ingestion batch request options.
  • Preserve the existing environment/default fallback for direct ScoreManager construction.
  • Add regression coverage for the LANGFUSE_TIMEOUT path.

Verification

  • pnpm lint
  • pnpm typecheck
  • pnpm test (the repository-wide command includes server-backed E2E)
  • pnpm test:integration
  • pnpm test:e2e (requires the Docker-based Langfuse server stack; delegated to CI)
  • pnpm format:check
  • pnpm test:unit
  • pnpm build

Release info

Bump level

  • Major
  • Minor
  • Patch

Libraries affected

  • All of them
  • @langfuse/core
  • @langfuse/client
  • @langfuse/tracing
  • @langfuse/otel
  • @langfuse/openai
  • @langfuse/langchain

Changelog notes

  • Apply configured request timeouts to score ingestion batches.

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

@sidsri14 is attempting to deploy a commit to the langfuse Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LANGFUSE_TIMEOUT is not applied to ScoreManager ingestion requests

2 participants