Add 30-second global request timeout middleware to Express app

pook commented

2026-04-10 05:24:07 -04:00

Owner

Add a global request timeout to prevent hung OpenAI API calls or slow database queries from blocking the Express server indefinitely.

Implementation:

Add middleware before route handlers that sets req.setTimeout(30000) and attaches a response timeout
Use res.setTimeout(30000, () => { res.status(504).json({ error: 'Request timeout' }) })
Or use connect-timeout middleware with 30s limit
Add integration test: mock a slow handler (>30s), verify 504 response
Ensure timeout does NOT interfere with Stripe webhook async processing (exclude /webhooks/* from timeout)

Acceptance: Requests exceeding 30s get 504, webhook routes exempt, test passes.

Generated by CEO Planner (priority: 3)

Add a global request timeout to prevent hung OpenAI API calls or slow database queries from blocking the Express server indefinitely. **Implementation:** - Add middleware before route handlers that sets `req.setTimeout(30000)` and attaches a response timeout - Use `res.setTimeout(30000, () => { res.status(504).json({ error: 'Request timeout' }) })` - Or use connect-timeout middleware with 30s limit - Add integration test: mock a slow handler (>30s), verify 504 response - Ensure timeout does NOT interfere with Stripe webhook async processing (exclude /webhooks/* from timeout) **Acceptance:** Requests exceeding 30s get 504, webhook routes exempt, test passes. --- *Generated by CEO Planner (priority: 3)*

pook added the

agent-task

label

2026-04-10 05:24:07 -04:00

pook commented

2026-04-10 11:30:45 -04:00

Author

Owner

⚠️ Stale Task Alert — This task has been open for 6h with no associated PR.

Possible causes:

Worker failed to execute (check logs)
Claude CLI produced no changes
Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 6h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*

pook commented

2026-04-10 11:46:15 -04:00

Author

Owner

⚠️ Stale Task Alert — This task has been open for 6h with no associated PR.

Possible causes:

Worker failed to execute (check logs)
Claude CLI produced no changes
Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 6h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*

pook commented

2026-04-10 12:02:18 -04:00

Author

Owner

⚠️ Stale Task Alert — This task has been open for 7h with no associated PR.

Possible causes:

Worker failed to execute (check logs)
Claude CLI produced no changes
Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 7h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*

pook referenced this issue

2026-04-10 12:05:52 -04:00

Replace stale #304 — add 30-second global request timeout to Express app #354

agent-bot referenced this issue from a commit

2026-04-10 12:23:55 -04:00

Add 30s global request timeout middleware

pook commented

2026-04-10 12:24:23 -04:00

Author

Owner

⚠️ Stale Task Alert — This task has been open for 7h with no associated PR.

Possible causes:

Worker failed to execute (check logs)
Claude CLI produced no changes
Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 7h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*

pook commented

2026-04-10 12:30:03 -04:00

Author

Owner

⚠️ Stale Task Alert — This task has been open for 7h with no associated PR.

Possible causes:

Worker failed to execute (check logs)
Claude CLI produced no changes
Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 7h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*

pook commented

2026-04-10 12:42:55 -04:00

Author

Owner

⚠️ Stale Task Alert — This task has been open for 7h with no associated PR.

Possible causes:

Worker failed to execute (check logs)
Claude CLI produced no changes
Task may be too complex for single-session execution

The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped.

— CEO Planner (automated)

⚠️ **Stale Task Alert** — This task has been open for 7h with no associated PR. Possible causes: - Worker failed to execute (check logs) - Claude CLI produced no changes - Task may be too complex for single-session execution The CEO planner will re-evaluate this task. If it remains stale for 24h+, it will be closed and re-scoped. *— CEO Planner (automated)*

pook referenced this issue

2026-04-10 12:47:06 -04:00

Close 5 stale issues with exact replacement issues open #360

pook commented

2026-04-10 12:49:20 -04:00

Author

Owner

Closing stale — replaced by #354

pook closed this issue

2026-04-10 12:49:20 -04:00

pook referenced this issue

2026-04-10 12:49:35 -04:00