unable to upload screenshots/videos to S3

[sorry-cypress]

unable to upload screenshots/videos to S3

[dineshk-qa]

@agoldis actually our S3 bucket was hosted in ap-southeast-2 and we were not setting the region and it was taking the default value, so we had that issue.
It just coincided with your changes :)
All working fine now after the above fix, I can see images/videos in the dashboard.
Thanks for so quick response & fix.

[danielvianna]

I have the exact issue before then disappeared after 2 days - no clue what is causing on a specific machine, and on another one, it never happened (both are windows)
If you upload a file via postman (or similar tool) does the Amazon S3 API works fine?

[agoldis]

@dineshk-qa thanks for reporting the issue! Would be useful to run the failed tests locally with similar settings with DEBUG=cypress:* env variable to see if there are more details from cypress agent.

[dineshk-qa]

Here is Cypress in DEBUG mode
cypress:server:record failed to upload artifact { file: 'source/cypress-poc/cypress/screenshots/2_gift_cards_spec.js/Gift Cards Page Tests -- iframe demo - buy gift cards (failed).png', stack: 'StatusCodeError: 400 - "<?xml version=\\"1.0\\" encoding=\\"UTF-8\\"?>\\n<Error><Code>InvalidArgument</Code><Message></Message><ArgumentName>x-amz-acl</ArgumentName><ArgumentValue>ap-southeast-2</ArgumentValue><RequestId>xxxxx</RequestId> <HostId>xxxxxxxxx</HostId> </Error>

Getting 400 due to Invalid argument > x-amz-acl: ap-southeast-2

@danielvianna - Yes uploading through postman is working fine, but I have checked using my user not that role

[EddyQiu]

I've found something interesting in the docker.io/agoldis/sorry-cypress-director image in the path /app/src/screenshots/s3

# cat config.ts export const S3_BUCKET = process.env.S3_BUCKET || 'sorry-cypress'; export const S3_REGION = process.env.S3_REGION || 'us-east-1'; export const S3_ACL = process.env.S3_REGION || 'public-read'; export const S3_READ_URL_PREFIX = process.env.S3_READ_URL_PREFIX || null;

Does the S3_ACL need to be set to a region?
Because when I set the S3_REGION to ap-southeast-2 it will also change the S3_ACL to ap-southeast-2 when we want it to be 'public-read'

Thanks,
Eddy

[EddyQiu]

I believe the fix for this would be just to change the line
export const S3_ACL = process.env.S3_REGION || 'public-read';
to be:
export const S3_ACL = process.env.S3_ACL || 'public-read';

[agoldis]

There was a bug introduced in the latests release. Should be fixed in https://github.com/agoldis/sorry-cypress/releases/tag/v0.2.5.

@dineshk-qa Please tell me if that somehow resolves the issue, because the issue was opened before the bug was introduced 😃