Skip to content
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

add toJSON implementation for R2Checksums #103

Merged
merged 1 commit into from
Oct 21, 2022
Merged

Conversation

Frederik-Baetens
Copy link
Contributor

This allows JSON.stringify() to work on the result of head/list again

@Frederik-Baetens Frederik-Baetens force-pushed the fbaetens/R2-checksums-json branch from ed586d1 to f263478 Compare October 19, 2022 21:01
@Frederik-Baetens Frederik-Baetens force-pushed the fbaetens/R2-checksums-json branch from f263478 to 451ce5e Compare October 19, 2022 21:07
@jasnell jasnell merged commit 33929a9 into main Oct 21, 2022
@jasnell jasnell deleted the fbaetens/R2-checksums-json branch October 21, 2022 23:15
mrbbot added a commit to cloudflare/miniflare that referenced this pull request Jan 17, 2023
* Coerce R2 keys to `string`s

The Workers runtime coerces arguments if the C++ type is a string.
It will only throw if the parameter isn't specified at all.

* Accept multiple keys in `R2Bucket#delete()`, closes #420

Ref: https://community.cloudflare.com/t/2022-9-16-workers-runtime-release-notes/420496

* Implement `r2_list_honor_include` compatibility flag

Ref: https://developers.cloudflare.com/workers/platform/compatibility-dates/#r2-bucket-list-respects-the-include-option

* Allow `R2Bucket#get()` range to be specified as a `Range` header

Miniflare already had code for parsing `Range` headers for the Cache
API, so this was moved to `@miniflare/shared` and reused here.

Ref: https://community.cloudflare.com/t/2022-8-12-workers-runtime-release-notes/410873

* Implement R2 `SHA-*` checksums and return from `R2Bucket#{get,head}`

Ref: https://community.cloudflare.com/t/2022-9-16-workers-runtime-release-notes/420496
Ref: cloudflare/workerd#103

* fixup! Implement R2 `SHA-*` checksums and return from `R2Bucket#{get,head}`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants