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

feat: cloudflare-r2-binding driver #235

Merged
merged 9 commits into from
Jul 4, 2023
Merged

feat: cloudflare-r2-binding driver #235

merged 9 commits into from
Jul 4, 2023

Conversation

pi0
Copy link
Member

@pi0 pi0 commented May 26, 2023

resolves #226

Support for cloudflare r2 (within workers). Support for HTTP / S3 compatible driver coming soon for other envs.

TODO:

  • Try to use miniflare v3 instead of mocks delayed
  • Test key normalization to convert : <> /

@nuxt-studio
Copy link

nuxt-studio bot commented May 26, 2023

Live Preview ready!

Name Edit Preview Latest Commit
unstorage Edit on Studio ↗︎ View Live Preview 6d6d739

@codecov
Copy link

codecov bot commented May 26, 2023

Codecov Report

Merging #235 (6d6d739) into main (34f14f8) will increase coverage by 0.03%.
The diff coverage is 77.57%.

@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
+ Coverage   76.48%   76.51%   +0.03%     
==========================================
  Files          25       26       +1     
  Lines        3057     3164     +107     
  Branches      449      473      +24     
==========================================
+ Hits         2338     2421      +83     
- Misses        718      742      +24     
  Partials        1        1              
Impacted Files Coverage Δ
src/drivers/cloudflare-r2-binding.ts 77.57% <77.57%> (ø)

@Hebilicious Hebilicious added enhancement New feature or request driver cloudflare and removed enhancement New feature or request labels Jun 30, 2023 — with Volta.net
@pi0 pi0 marked this pull request as ready for review July 4, 2023 15:47
@pi0
Copy link
Member Author

pi0 commented Jul 4, 2023

Moving forward as experimental driver

@pi0 pi0 merged commit 4386798 into main Jul 4, 2023
@pi0 pi0 deleted the feat/cloudflare-r2-binding branch July 4, 2023 15:51
so1ve pushed a commit to so1ve/unstorage that referenced this pull request Jul 8, 2023
* fix(prefixStorage): prefix `getItemRaw` and `setItemRaw` (unjs#232)

* fix(github): fetchFiles should return files (unjs#229)

* chore(deps): update all non-major dependencies (unjs#220)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: update eslint

* test: skip cloudflare-kv-http on node >= 18

* chore(release): v1.6.1

* docs: add social share image

* chore: update deps

* docs: fix typo (unjs#239)

Change `environemnt` to `environment`
Update cloudflare-kv-http.md

* chore: update dependencies

* feat: generic type support (unjs#237)

* refactor: fix issues with typescript strict (unjs#250)

* chore: add type check to ci

* ci: skip flaky azure tests

* chore(release): v1.7.0

* chore(deps): update all non-major dependencies

* docs: fix typo (unjs#252)

* chore(deps): update all non-major dependencies

* test: add test for `github` driver (unjs#259)

* feat: experimental operation batching (unjs#240)

Co-authored-by: Pooya Parsa <pooya@pi0.io>

* feat(cloudflare-kv): support `base` option for keys (unjs#261)

* feat: `cloudflare-r2-binding` driver (unjs#235)

* fix: add missing `cloudflareR2Binding` to the `builtinDrivers`

* chore: update dev dependencies

* chore(release): v1.8.0

* Fix typescript checks

* add typehint

* Install execa

* Write test code

* Remove not using import

---------

Co-authored-by: 魔王少年 <q267009886.work@gmail.com>
Co-authored-by: Andrei Dyldin <and@cesbo.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Pooya Parsa <pooya@pi0.io>
Co-authored-by: Sébastien Chopin <seb@nuxt.com>
Co-authored-by: Neelansh Mathur <53081208+neelansh15@users.noreply.github.com>
Co-authored-by: 魔王少年 <q267009886.tw@gmail.com>
Co-authored-by: Alex Duval <alexduval71@gmail.com>
Co-authored-by: Hebilicious <xsh4k3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add cloudflare R2 driver
2 participants