Andreas/LibreChat
1
0
Fork 0
mirror of https://github.com/danny-avila/LibreChat.git synced 2025-12-16 16:30:15 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

Include the path length when substring-ing the url when using forcePathStyle

Browse source
This commit is contained in:
David Newman 2025-11-13 11:36:28 +10:00
parent 75487ca12a
commit e28e9969fa
1 changed files with 15 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

15
api/server/services/Files/S3/crud.js
View file

@ -11,8 +11,10 @@ const {
DeleteObjectCommand,
} = require('@aws-sdk/client-s3');
const endpoint = process.env.AWS_ENDPOINT_URL;
const bucketName = process.env.AWS_BUCKET_NAME;
const defaultBasePath = 'images';
const forcePathStyle = ['1', 'true', 'yes'].includes(process.env.AWS_FORCE_PATH_STYLE?.toLowerCase());
let s3UrlExpirySeconds = 2 * 60; // 2 minutes
let s3RefreshExpiryMs = null;
@ -250,6 +252,13 @@ function extractKeyFromS3Url(fileUrlOrKey) {
try {
const url = new URL(fileUrlOrKey);
if (endpoint?.trim() && forcePathStyle) {
const endpointUrl = new URL(endpoint)
const startPos = endpointUrl.pathname.length + (endpointUrl.pathname.endsWith('/') ? 2 : 1) + bucketName.length + 1;
return url.pathname.substring(startPos);
}
return url.pathname.substring(1);
} catch (error) {
const parts = fileUrlOrKey.split('/');
@ -258,6 +267,12 @@ function extractKeyFromS3Url(fileUrlOrKey) {
return fileUrlOrKey;
}
if (endpoint?.trim() && forcePathStyle) {
const endpointUrl = new URL(endpoint)
const startPos = endpointUrl.pathname.length + (endpointUrl.pathname.endsWith('/') ? 2 : 1) + bucketName.length + 1;
return fileUrlOrKey.substring(startPos);
}
return fileUrlOrKey.startsWith('/') ? fileUrlOrKey.substring(1) : fileUrlOrKey;
}
}