Skip to content

fix(admin): extend request log export timeout - #5091

Merged
chrarnoldus merged 1 commit into
mainfrom
fix/api-request-log-export-timeout
Aug 6, 2026
Merged

fix(admin): extend request log export timeout#5091
chrarnoldus merged 1 commit into
mainfrom
fix/api-request-log-export-timeout

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • increase the API request log download route duration from 300 to 800 seconds
  • give large streamed archives more time to write their ZIP central directory before Vercel terminates the function

This is intentionally separate from #5089.

Verification

  • pnpm --filter web typecheck
  • pnpm exec oxlint --config .oxlintrc.json apps/web/src/app/admin/api/api-request-log/download/route.ts
  • pnpm exec oxfmt apps/web/src/app/admin/api/api-request-log/download/route.ts
  • git diff --check

Visual Changes

None. Server route configuration only.

@chrarnoldus chrarnoldus self-assigned this Aug 6, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Single-line Vercel maxDuration increase (300→800s) for the admin-only API request log ZIP export route; 800s is the established convention across ~10 other apps/web routes and matches the platform maximum, with no test, vercel.json, or memory-handling conflicts identified.

Files Reviewed (1 file)
  • apps/web/src/app/admin/api/api-request-log/download/route.ts

Reviewed by kimi-k3 · Input: 34.3K · Output: 5.3K · Cached: 238.9K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus enabled auto-merge (squash) August 6, 2026 10:51
@chrarnoldus
chrarnoldus merged commit 42f911f into main Aug 6, 2026
16 checks passed
@chrarnoldus
chrarnoldus deleted the fix/api-request-log-export-timeout branch August 6, 2026 10:57
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.

2 participants