Remove `range-parser` dependency

kotx / render

Cloudflare Worker to proxy and cache requests to R2

MIT License

384 stars 85 forks source link

Remove `range-parser` dependency #18

Open kotx opened 2 years ago

kotx commented 2 years ago

R2 API can process range requests now, it seems. We can remove the range-parser dependency and directly pass the header instead.

Erisa commented 1 year ago

It looks like the functionality to pass request headers to R2GetOptions is not documented outside of changelogs... nor does it actually work.

✘ [ERROR] Uncaught (in promise) Error: get: We encountered an internal error. Please try again. (10001)

As far as I can tell this bug is acknowledged but has existed for a while now without a fix. So this issue would be blocked waiting for that being fixed in future.

Frederik-Baetens commented 1 year ago

This should be fixed now. Let me know if you encounter any more issues surrounding this functionality.