chore(api): disable request timeout

[jakubno]

Note

Medium Risk
Removing per-request context deadlines can allow handlers to block indefinitely (e.g., on DB pool acquisition), increasing the risk of goroutine buildup and degraded availability under load. It also shifts timeout behavior to upstream/proxy and server write deadlines, which may be less predictable for long-running requests.

Overview
Disables the API’s per-request context timeout by commenting out the requestTimeout constant and the customMiddleware.RequestTimeout(...) Gin middleware, so requests no longer get an enforced 60s cancellation/408 behavior and may run until other server or upstream timeouts intervene.

^{Written by Cursor Bugbot for commit 2144d21. This will update automatically on new commits. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Removing request timeout risks indefinite handler blocking
  • Restored the RequestTimeout middleware by uncommenting both the requestTimeout constant and the middleware registration, preventing indefinite handler blocking during DB/gRPC calls.

Or push these changes by commenting:

@cursor push 0920ca2bd3

Preview (0920ca2bd3)

diff --git a/packages/api/main.go b/packages/api/main.go
--- a/packages/api/main.go
+++ b/packages/api/main.go
@@ -59,7 +59,7 @@
 	// Must be less than maxWriteTimeout so the context cancels before the
 	// server's write deadline kills the connection (WriteTimeout does NOT
 	// cancel r.Context(); see https://github.com/golang/go/issues/59602).
-	// requestTimeout = 60 * time.Second
+	requestTimeout = 60 * time.Second
 
 	// This timeout should be > 600 (GCP LB upstream idle timeout) to prevent race condition
 	// https://cloud.google.com/load-balancing/docs/https#timeouts_and_retries%23:~:text=The%20load%20balancer%27s%20backend%20keepalive,is%20greater%20than%20600%20seconds
@@ -129,7 +129,7 @@
 			},
 		}),
 		gin.Recovery(),
-		// customMiddleware.RequestTimeout(requestTimeout), //nolint:contextcheck // Gin middleware sets context via c.Request.WithContext
+		customMiddleware.RequestTimeout(requestTimeout), //nolint:contextcheck // Gin middleware sets context via c.Request.WithContext
 	)
 
 	corsConfig := cors.DefaultConfig()

_{This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2144d21c85

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".