Skip to content

add documentation about S3 signature validation - #861

Merged
quetzalliwrites merged 4 commits into
aws-docs-release-augustfrom
s3-signature-validation
Aug 25, 2026
Merged

add documentation about S3 signature validation#861
quetzalliwrites merged 4 commits into
aws-docs-release-augustfrom
s3-signature-validation

Conversation

@bentsku

@bentsku bentsku commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

In S3, we have special behavior for Signature Validation, something we don't do for other services.

We have supported pre-signed URLs signature validation for a long time now, but we didn't have a lot of documentation in how to best use it, which credentials to use and what are the limitations.

We recently also added signature validation for all requests, not only pre-signed for a prospect (Canva).
This felt like the right time to add documentation about both feature together as they are closely related and share some logic about which credentials you should use.

AWS Documentation on pre-signed URLs: https://docs.aws.amazon.com/AmazonS3/latest/userguide/using-presigned-url.html
For all and regular request signing: https://docs.aws.amazon.com/AmazonS3/latest/developerguide/sig-v4-authenticating-requests.html

Related

towards AWS-1256
closes DOC-401

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 10, 2026

Copy link
Copy Markdown

Deploying localstack-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 607442e
Status:⚡️  Build in progress...

View logs

@bentsku bentsku added the aws label Aug 11, 2026
@bentsku
bentsku marked this pull request as ready for review August 11, 2026 17:33
@quetzalliwrites

quetzalliwrites commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

hey @aidehn @k-a-il Ben needs an ENG review for his PR before we merge, ty :)

@bentsku

bentsku commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

I wrote this in the Linear ticket but might be worth sharing it here too: I think we're good on the engineering review side, if the writing looks good, I wrote the feature and we are in the middle of revisiting the ownership of services (Aiden is in another squad now and Anastasia on leave), so if it looks good to you Quetzalli, I think we can merge it 👍

I cannot remove their review request because they are still marked as codeowners. If you really want a second ENG review, maybe @viren-nadkarni can have a look, but I don't think it is necessary

@viren-nadkarni
viren-nadkarni self-requested a review August 25, 2026 09:53

@viren-nadkarni viren-nadkarni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

Comment thread src/content/docs/aws/services/s3.mdx Outdated
@bentsku

bentsku commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

@quetzalliwrites anything else we need to merge this before the release tomorrow?

Add "by default" per review feedback, since IAM policy enforcement
can be enabled to add the authorization check this sentence
otherwise reads as permanently absent.
@quetzalliwrites

quetzalliwrites commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

@bentsku Pushed 607442e0 addressing Viren's "by default" wording suggestion. That was the only open feedback — this will be good to merge from my side tmw after the release.

@quetzalliwrites
quetzalliwrites merged commit 40c9588 into aws-docs-release-august Aug 25, 2026
1 check was pending
@quetzalliwrites
quetzalliwrites deleted the s3-signature-validation branch August 25, 2026 18:45
quetzalliwrites added a commit that referenced this pull request Aug 26, 2026
Co-authored-by: Quetzalli <alejandra.olvera.novack@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants