Skip to content

fix: cap looney-check function maxDuration at hobby limit #78

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
creatorcluster merged 2 commits into from
Aug 18, 2026
Merged

fix: cap looney-check function maxDuration at hobby limit #78

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c1fc744
fix: cap looney-check function duration at hobby limit
Coder-soft Aug 18, 2026
90a6e21
fix: bound upstream job creation by request-wide deadline
Coder-soft Aug 18, 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
13 changes: 11 additions & 2 deletions api/looney-check.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ import { isAllowedOrigin } from './cors.js';
const MAX_UPLOAD_BYTES = 50 * 1024 * 1024;
const DEFAULT_LOONEY_URL = 'https://looney.codersoft.xyz/check';
const DAILY_CHECK_LIMIT = 5;
// Hobby caps the whole invocation at 60s. Keep work inside a 50s budget so
// request parsing, the quota RPC, the upstream call, and releaseRateLimit
// cleanup all complete before the platform terminates the invocation.
const INVOCATION_BUDGET_MS = 50000;

function getHeader(request, name) {
if (request.headers?.get) return request.headers.get(name);
Expand Down Expand Up @@ -235,7 +239,7 @@ export const config = {
bodyParser: false,
sizeLimit: '52mb',
},
maxDuration: 300,
maxDuration: 60,
Comment thread
coderabbitai[bot] marked this conversation as resolved.
};

class ValidationError extends Error {}
Expand Down Expand Up @@ -271,6 +275,7 @@ export default async function handler(request) {
}

try {
const invocationDeadline = Date.now() + INVOCATION_BUDGET_MS;
const upstream = await buildUpstreamRequest(request);
const rateLimit = await consumeRateLimit(request);
if (!rateLimit.allowed) {
Expand All @@ -280,11 +285,15 @@ export default async function handler(request) {
}
let response;
try {
const remainingBudgetMs = Math.max(0, invocationDeadline - Date.now());
response = await fetch(`${getLooneyBaseUrl()}/jobs`, {
method: 'POST',
headers: upstreamHeaders(upstream.contentType),
body: upstream.body,
signal: AbortSignal.timeout(30000),
// Remote file checks may download and inspect the audio before returning a job ID.
// Bound the upstream call by the remaining request budget so the abort handler
// below can still release the rate-limit reservation before the 60s cap.
signal: AbortSignal.timeout(remainingBudgetMs),
});
} catch (error) {
try {
Expand Down
Loading