yujunjun/strapi
1
0
Fork 0
mirror of https://github.com/strapi/strapi.git synced 2025-09-16 20:10:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: make aws credentials optional

Browse Source
This commit is contained in:
Christian Capeans 2023-11-10 11:28:32 +01:00
parent d999ad0b00
commit 1d64d58bb8
2 changed files with 14 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
packages/providers/upload-aws-s3/src/__tests__/utils.test.ts
View File

@ -16,7 +16,7 @@ const defaultOptions = {
describe('Utils', () => {
describe('Extract credentials for V4 different aws provider configurations', () => {
test('[Legacy] Credentials directly in the options', async () => {
test('[Legacy] Credentials directly in the options', () => {
const options: InitOptions = {
accessKeyId,
secretAccessKey,
@ -30,7 +30,7 @@ describe('Utils', () => {
});
});
test('[Legacy] credentials directly in s3Options', async () => {
test('[Legacy] credentials directly in s3Options', () => {
const options: InitOptions = {
s3Options: {
accessKeyId,
@ -46,7 +46,7 @@ describe('Utils', () => {
});
});
test('Credentials in credentials object inside s3Options', async () => {
test('Credentials in credentials object inside s3Options', () => {
const options: InitOptions = {
s3Options: {
credentials: {
@ -63,5 +63,15 @@ describe('Utils', () => {
secretAccessKey,
});
});
test('Does not throw an error when are not present', () => {
const options: InitOptions = {
s3Options: {
...defaultOptions,
},
};
const credentials = extractCredentials(options);
expect(credentials).toEqual({});
});
});
});

3
packages/providers/upload-aws-s3/src/utils.ts
View File

@ -114,6 +114,5 @@ export const extractCredentials = (options: InitOptions): AwsCredentialIdentity
secretAccessKey: options.s3Options.credentials.secretAccessKey,
};
}
throw new Error("Couldn't find AWS credentials.");
return {};
};