Skip to content

[R2] Update Presigned URL docs #21975

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
May 12, 2025
Merged

[R2] Update Presigned URL docs #21975

merged 8 commits into from
May 12, 2025

Conversation

@harshil1712
Copy link
Contributor

@harshil1712 harshil1712 commented Apr 25, 2025

Summary

Clarifies Presigned URLs in R2. Fixes #19190

Documentation checklist

@github-actions github-actions bot added product:r2 R2 object storage: https://developers.cloudflare.com/r2 size/s labels Apr 25, 2025
@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Apr 25, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.

PR Change Summary

Updated documentation for Presigned URLs in R2 to clarify usage and examples.

  • Clarified the generation and usage of presigned URLs in R2 documentation.
  • Added an example of generating presigned URLs using the aws4fetch package.
  • Updated the explanation of differences between presigned URLs and R2 bindings.

Modified Files

  • src/content/docs/r2/api/s3/presigned-urls.mdx
How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 25, 2025 •
edited
Loading

Preview URL: https://f57dc42d.preview.developers.cloudflare.com
Preview Branch URL: https://r2-fix-presigned-url.preview.developers.cloudflare.com

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/r2/api/s3/presigned-urls/ https://r2-fix-presigned-url.preview.developers.cloudflare.com/r2/api/s3/presigned-urls/

Oxyjun
Oxyjun reviewed Apr 25, 2025
View reviewed changes
Oxyjun
Oxyjun reviewed Apr 25, 2025
View reviewed changes
Oxyjun
Oxyjun reviewed Apr 25, 2025
View reviewed changes
Oxyjun
Oxyjun reviewed Apr 25, 2025
View reviewed changes
Oxyjun
Oxyjun reviewed Apr 25, 2025
View reviewed changes
Oxyjun
Oxyjun reviewed Apr 25, 2025
View reviewed changes
@github-actions github-actions bot added size/m and removed size/s labels Apr 25, 2025
@aninibread
Copy link
Contributor

aninibread commented May 12, 2025

that looks good to me, thank you for the suggestions!

@Oxyjun Oxyjun merged commit 8b002df into production May 12, 2025
13 checks passed
@Oxyjun Oxyjun deleted the r2-fix-presigned-url branch May 12, 2025 14:42
@Oxyjun Oxyjun mentioned this pull request May 12, 2025
Closed
daisyfaithauma pushed a commit that referenced this pull request May 13, 2025
@harshil1712 @Oxyjun @daisyfaithauma
[R2] Update Presigned URL docs (#21975)
4aa433f 
* update Presigned URL vs Binding

* Apply suggestions from code review

* update limitations

* Apply suggestions from code review

* Cleaning up structure of the doc with tabs.

* Small fix

* Further language clarification

---------

Co-authored-by: Jun Lee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Oxyjun Oxyjun Oxyjun approved these changes

@elithrar elithrar Awaiting requested review from elithrar elithrar is a code owner

@jonesphillip jonesphillip Awaiting requested review from jonesphillip jonesphillip is a code owner

Assignees

@elithrar elithrar

@jonesphillip jonesphillip

@Oxyjun Oxyjun

Labels

product:r2 R2 object storage: https://developers.cloudflare.com/r2 size/m

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[R2] Clarify Presigned URL docs

5 participants

@harshil1712 @aninibread @Oxyjun @elithrar @jonesphillip