mirror of
https://github.com/Significant-Gravitas/AutoGPT.git
synced 2026-01-08 22:58:01 -05:00
feat(frontend): increase file upload size limit to 256MB (#11634)
- Updated Next.js configuration to set body size limits for server
actions and API routes.
- Enhanced error handling in the API client to provide user-friendly
messages for file size errors.
- Added user-friendly error messages for 413 Payload Too Large responses
in API error parsing.
These changes ensure that file uploads are consistent with backend
limits and improve user experience during uploads.
### Checklist 📋
#### For code changes:
- [x] I have clearly listed my changes in the PR description
- [x] I have made a test plan
- [x] I have tested my changes according to the test plan:
<!-- Put your test plan here: -->
- [x] Upload a file bigger than 10MB and it works
- [X] Upload a file bigger than 256MB and you see a official error
stating the max file size is 256MB
This commit is contained in:
@@ -3,6 +3,14 @@ import { withSentryConfig } from "@sentry/nextjs";
|
||||
/** @type {import('next').NextConfig} */
|
||||
const nextConfig = {
|
||||
productionBrowserSourceMaps: true,
|
||||
experimental: {
|
||||
serverActions: {
|
||||
bodySizeLimit: "256mb",
|
||||
},
|
||||
// Increase body size limit for API routes (file uploads) - 256MB to match backend limit
|
||||
proxyClientMaxBodySize: "256mb",
|
||||
middlewareClientMaxBodySize: "256mb",
|
||||
},
|
||||
images: {
|
||||
domains: [
|
||||
// We dont need to maintain alphabetical order here
|
||||
|
||||
@@ -6,6 +6,10 @@ import {
|
||||
import { environment } from "@/services/environment";
|
||||
import { NextRequest, NextResponse } from "next/server";
|
||||
|
||||
// Increase body size limit to 256MB to match backend file upload limit
|
||||
export const maxDuration = 300; // 5 minutes timeout for large uploads
|
||||
export const dynamic = "force-dynamic";
|
||||
|
||||
function buildBackendUrl(path: string[], queryString: string): string {
|
||||
const backendPath = path.join("/");
|
||||
return `${environment.getAGPTServerBaseUrl()}/${backendPath}${queryString}`;
|
||||
|
||||
@@ -910,7 +910,37 @@ export default class BackendAPI {
|
||||
reject(new Error("Invalid JSON response"));
|
||||
}
|
||||
} else {
|
||||
reject(new Error(`HTTP ${xhr.status}: ${xhr.statusText}`));
|
||||
// Handle file size errors with user-friendly message
|
||||
if (xhr.status === 413) {
|
||||
reject(new Error("File is too large — max size is 256MB"));
|
||||
return;
|
||||
}
|
||||
|
||||
// Try to parse error response for better messages
|
||||
let errorMessage = `Upload failed (${xhr.status})`;
|
||||
try {
|
||||
const errorData = JSON.parse(xhr.responseText);
|
||||
if (errorData.detail) {
|
||||
if (
|
||||
typeof errorData.detail === "string" &&
|
||||
errorData.detail.includes("exceeds the maximum")
|
||||
) {
|
||||
const match = errorData.detail.match(
|
||||
/maximum allowed size of (\d+)MB/,
|
||||
);
|
||||
const maxSize = match ? match[1] : "256";
|
||||
errorMessage = `File is too large — max size is ${maxSize}MB`;
|
||||
} else if (typeof errorData.detail === "string") {
|
||||
errorMessage = errorData.detail;
|
||||
}
|
||||
} else if (errorData.error) {
|
||||
errorMessage = errorData.error;
|
||||
}
|
||||
} catch {
|
||||
// Keep default message if parsing fails
|
||||
}
|
||||
|
||||
reject(new Error(errorMessage));
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -184,6 +184,11 @@ export function serializeRequestBody(
|
||||
}
|
||||
|
||||
export async function parseApiError(response: Response): Promise<string> {
|
||||
// Handle 413 Payload Too Large with user-friendly message
|
||||
if (response.status === 413) {
|
||||
return "File is too large — max size is 256MB";
|
||||
}
|
||||
|
||||
try {
|
||||
const errorData = await response.clone().json();
|
||||
|
||||
@@ -205,6 +210,16 @@ export async function parseApiError(response: Response): Promise<string> {
|
||||
return response.statusText; // Fallback to status text if no message
|
||||
}
|
||||
|
||||
// Check for file size error from backend
|
||||
if (
|
||||
typeof errorData.detail === "string" &&
|
||||
errorData.detail.includes("exceeds the maximum")
|
||||
) {
|
||||
const match = errorData.detail.match(/maximum allowed size of (\d+)MB/);
|
||||
const maxSize = match ? match[1] : "256";
|
||||
return `File is too large — max size is ${maxSize}MB`;
|
||||
}
|
||||
|
||||
return errorData.detail || errorData.error || response.statusText;
|
||||
} catch {
|
||||
return response.statusText;
|
||||
|
||||
Reference in New Issue
Block a user