-
Notifications
You must be signed in to change notification settings - Fork 10.3k
Remove R2 compatibility notes on sdk clients #19890
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.
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
Remove R2 compatibility notes on sdk clients #19890
Conversation
|
Congratulations @smellercf, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cm70q8mjc00320cl154yryi6u This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
|
With It appears the problem hasn't been fixed in this case. Adding: to each storage definition in config/storage.yml` did resolve the issue but I think what was implied by this PR is that R2 was updated to work with the new S3 library and it doesn't appear to have been, or at least not the servers that I use (Sydney). |
Same here. Adding the above settings fixed the issue for me. If I remove them, I get the same error. |
I've raised it in the Discord and they're looking into it :) https://discord.com/channels/595317990191398933/1339407687049412608 |
Summary
Removes outdated compatibility notes for SDK clients now that R2 is compatible for most clients. Updates the CLI documentation to specify the requirement for the CRC32 checksum flag.
Screenshots (optional)
Documentation checklist