mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2024-12-24 04:04:06 -08:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

fix(AwsS3 Node): Use location constrain (#8654)

Browse source
This commit is contained in:
Michael Kret 2024-02-16 15:13:26 +02:00 committed by GitHub
parent 8e9d3106a5
commit c73aeeec3d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
packages/nodes-base/credentials/Aws.credentials.ts
View file

@ -284,7 +284,13 @@ export class Aws implements ICredentialType {
let path = requestOptions.qs?.path;
const method = requestOptions.method;
let body = requestOptions.body;
let region = credentials.region;
if (requestOptions.qs?._region) {
region = requestOptions.qs._region as string;
delete requestOptions.qs._region;
}
let query = requestOptions.qs?.query as IDataObject;
// ! Workaround as we still use the IRequestOptions interface which uses uri instead of url
// ! To change when we replace the interface with IHttpRequestOptions
@ -324,10 +330,10 @@ export class Aws implements ICredentialType {
} else if (service === 'sqs' && credentials.sqsEndpoint) {
endpointString = credentials.sqsEndpoint as string;
} else if (service) {
endpointString = `https://${service}.${credentials.region}.amazonaws.com`;
endpointString = `https://${service}.${region}.amazonaws.com`;
}
endpoint = new URL(
endpointString!.replace('{region}', credentials.region as string) + (path as string),
endpointString!.replace('{region}', region as string) + (path as string),
);
} else {
// If no endpoint is set, we try to decompose the path and use the default endpoint

2
packages/nodes-base/nodes/Aws/S3/V2/GenericFunctions.ts
View file

@ -29,12 +29,14 @@ export async function awsApiRequest(
service,
path,
query,
_region,
},
method,
body,
url: '',
headers,
} as IHttpRequestOptions;
if (Object.keys(option).length !== 0) {
Object.assign(requestOptions, option);
}