mirror of
https://github.com/yusufipk/OpenFrame.git
synced 2026-09-11 17:46:06 +00:00
A reservation id was never a secret and could not have been one. An upload token is base64url(payload) followed by its signature, so a client can read every claim out of its own token, and the two R2 init routes hand their reservation ids to the client outright. The asset route takes a reservation id from the request body and deleted it on the strength of that id and the billed user alone, and every hold an account owns is billed to the same user. So a caller could start a Bunny upload, read the id out of the token they were just given, quote it while attaching a one byte image or even a bare YouTube link, and have the quota handed back while the upload carried on. Repeat and a trial worth three gigabytes uploads as much as it likes for as long as Bunny takes to report a figure of its own. Signing the id rather than handing it over bought nothing, because signing is not hiding. A hold now records what it was opened for and is only ever consumed by that flow, so naming one is no longer enough to drop it. Guests hold against the workspace owner's quota rather than their own and had no way to give it back: the release was gated on being signed in. Declaring a size and walking away cost the guest nothing and cost the owner their whole remaining allowance for two hours. The guest grant now carries the reservation and the declared size, bound to the Bunny video as well as to ours, so cancelling gives the quota back and costs them the upload it stood for. What a guest can hold without cancelling lapses in half an hour rather than two hours. The in-transaction fallback check counted the account's Bunny storage as zero on a Bunny upload, because the figure was only prefetched for R2 providers and that branch was unreachable for Bunny until this PR made it reachable. On an account whose storage is all Bunny that was a check that could not fail. It is prefetched for every provider that can reach the fallback now.
210 lines
6.9 KiB
TypeScript
210 lines
6.9 KiB
TypeScript
import { NextRequest } from 'next/server';
|
|
import { auth, checkProjectAccess } from '@/lib/auth';
|
|
import { db } from '@/lib/db';
|
|
import { r2Client, R2_BUCKET_NAME } from '@/lib/r2';
|
|
import { PutObjectCommand } from '@aws-sdk/client-s3';
|
|
import { randomUUID } from 'crypto';
|
|
import { rateLimit } from '@/lib/rate-limit';
|
|
import { validateShareLinkAccess } from '@/lib/share-links';
|
|
import { getShareSessionFromRequest } from '@/lib/share-session';
|
|
import { apiErrors, successResponse, withCacheControl } from '@/lib/api-response';
|
|
import {
|
|
detectImageMime,
|
|
getImageExtension,
|
|
isAllowedImageType,
|
|
normalizeImageMime,
|
|
} from '@/lib/image-upload-validation';
|
|
import {
|
|
deriveGuestUploadContext,
|
|
enforceGuestUploadQuota,
|
|
verifyGuestUploadToken,
|
|
} from '@/lib/guest-upload-token';
|
|
import { logError } from '@/lib/logger';
|
|
import {
|
|
reserveStorageQuota,
|
|
releaseStorageReservation,
|
|
UPLOAD_RESERVATION_PURPOSES,
|
|
} from '@/lib/storage-quota';
|
|
|
|
const MAX_FILE_SIZE = 10 * 1024 * 1024; // 10MB
|
|
const MAX_MULTIPART_BODY_SIZE = MAX_FILE_SIZE + 512 * 1024; // file + multipart overhead
|
|
|
|
export async function POST(request: NextRequest) {
|
|
try {
|
|
// Check Content-Length header BEFORE loading the file
|
|
const contentLength = request.headers.get('content-length');
|
|
if (!contentLength) {
|
|
return apiErrors.badRequest('Missing Content-Length header');
|
|
}
|
|
const bodySize = parseInt(contentLength, 10);
|
|
if (isNaN(bodySize) || bodySize <= 0) {
|
|
return apiErrors.badRequest('Invalid Content-Length header');
|
|
}
|
|
if (bodySize > MAX_MULTIPART_BODY_SIZE) {
|
|
return apiErrors.badRequest('File too large. Maximum size is 10MB.');
|
|
}
|
|
|
|
// Rate limit
|
|
const limited = await rateLimit(request, 'image-upload');
|
|
if (limited) return limited;
|
|
|
|
const session = await auth();
|
|
|
|
const formData = await request.formData();
|
|
const files = formData.getAll('image');
|
|
if (files.length !== 1) {
|
|
return apiErrors.badRequest('No image file provided');
|
|
}
|
|
const file = files[0];
|
|
const videoId = formData.get('videoId');
|
|
const uploadToken = formData.get('uploadToken');
|
|
|
|
if (!(file instanceof File)) {
|
|
return apiErrors.badRequest('No image file provided');
|
|
}
|
|
if (typeof videoId !== 'string' || !videoId.trim()) {
|
|
return apiErrors.badRequest('videoId is required');
|
|
}
|
|
|
|
const safeVideoId = videoId.trim();
|
|
const video = await db.video.findUnique({
|
|
where: { id: safeVideoId },
|
|
include: {
|
|
project: {
|
|
include: { workspace: { select: { ownerId: true } } },
|
|
},
|
|
},
|
|
});
|
|
if (!video) {
|
|
return apiErrors.notFound('Video');
|
|
}
|
|
|
|
const access = await checkProjectAccess(video.project, session?.user?.id);
|
|
const shareSession = getShareSessionFromRequest(request, safeVideoId);
|
|
const shareAccess = shareSession
|
|
? await validateShareLinkAccess({
|
|
token: shareSession.token,
|
|
projectId: video.projectId,
|
|
videoId: safeVideoId,
|
|
requiredPermission: 'COMMENT',
|
|
passwordVerified: shareSession.passwordVerified,
|
|
})
|
|
: {
|
|
hasAccess: false,
|
|
canComment: false,
|
|
canDownload: false,
|
|
allowGuests: false,
|
|
requiresPassword: false,
|
|
};
|
|
const canCommentWithMembership = !!session?.user?.id && access.hasAccess;
|
|
const canCommentWithShareLink =
|
|
shareAccess.canComment && (session?.user?.id ? true : shareAccess.allowGuests);
|
|
if (!canCommentWithMembership && !canCommentWithShareLink) {
|
|
return apiErrors.forbidden('Access denied');
|
|
}
|
|
|
|
if (!session?.user?.id) {
|
|
if (typeof uploadToken !== 'string' || !uploadToken.trim()) {
|
|
return apiErrors.badRequest('uploadToken is required for guest uploads');
|
|
}
|
|
|
|
const expectedContext = deriveGuestUploadContext(request, shareSession?.token ?? null);
|
|
if (!expectedContext) {
|
|
return apiErrors.forbidden('Missing trusted client IP header');
|
|
}
|
|
|
|
const isValidUploadToken = verifyGuestUploadToken(uploadToken.trim(), {
|
|
projectId: video.projectId,
|
|
videoId: safeVideoId,
|
|
intent: 'image',
|
|
context: expectedContext,
|
|
});
|
|
if (!isValidUploadToken) {
|
|
return apiErrors.forbidden('Invalid upload token');
|
|
}
|
|
|
|
const quotaError = await enforceGuestUploadQuota(
|
|
request,
|
|
safeVideoId,
|
|
'image',
|
|
shareSession?.token ?? null
|
|
);
|
|
if (quotaError) return quotaError;
|
|
}
|
|
|
|
// Double-check file size (defense in depth - Content-Length can be spoofed)
|
|
if (file.size > MAX_FILE_SIZE) {
|
|
return apiErrors.badRequest('File too large. Maximum size is 10MB.');
|
|
}
|
|
|
|
// Enforce per-user storage quota before uploading.
|
|
// All paths use the advisory-locked reservation so concurrent uploads always
|
|
// see each other's in-flight sizes, eliminating the TOCTOU race.
|
|
const workspaceOwnerId = video.project.workspace.ownerId;
|
|
const reserveResult = await reserveStorageQuota(
|
|
workspaceOwnerId,
|
|
BigInt(file.size),
|
|
UPLOAD_RESERVATION_PURPOSES.IMAGE
|
|
);
|
|
if ('error' in reserveResult) return reserveResult.error;
|
|
const reservationId = reserveResult.reservationId;
|
|
|
|
// Check content type
|
|
const normalizedMime = normalizeImageMime(file.type);
|
|
if (normalizedMime && !isAllowedImageType(normalizedMime)) {
|
|
await releaseStorageReservation(
|
|
reservationId,
|
|
workspaceOwnerId,
|
|
UPLOAD_RESERVATION_PURPOSES.IMAGE
|
|
);
|
|
return apiErrors.badRequest(`Unsupported image format: ${file.type}`);
|
|
}
|
|
|
|
// Convert to buffer
|
|
const arrayBuffer = await file.arrayBuffer();
|
|
const buffer = Buffer.from(arrayBuffer);
|
|
const detectedMime = detectImageMime(buffer);
|
|
if (!detectedMime) {
|
|
await releaseStorageReservation(
|
|
reservationId,
|
|
workspaceOwnerId,
|
|
UPLOAD_RESERVATION_PURPOSES.IMAGE
|
|
);
|
|
return apiErrors.badRequest('Uploaded file content does not match an allowed image type');
|
|
}
|
|
|
|
// Generate unique filename
|
|
const ext = getImageExtension(detectedMime);
|
|
const filename = `${randomUUID()}.${ext}`;
|
|
const key = `images/${filename}`;
|
|
|
|
try {
|
|
// Upload to R2
|
|
await r2Client.send(
|
|
new PutObjectCommand({
|
|
Bucket: R2_BUCKET_NAME,
|
|
Key: key,
|
|
Body: buffer,
|
|
ContentType: detectedMime,
|
|
})
|
|
);
|
|
} catch (uploadError) {
|
|
await releaseStorageReservation(
|
|
reservationId,
|
|
workspaceOwnerId,
|
|
UPLOAD_RESERVATION_PURPOSES.IMAGE
|
|
);
|
|
throw uploadError;
|
|
}
|
|
|
|
// Return the URL through our proxy endpoint
|
|
const imageUrl = `/api/upload/image/${filename}`;
|
|
|
|
const response = successResponse({ url: imageUrl, reservationId }, 201);
|
|
return withCacheControl(response, 'private, no-store');
|
|
} catch (error) {
|
|
logError('Error uploading image:', error);
|
|
return apiErrors.internalError('Failed to upload image');
|
|
}
|
|
}
|