feat: remove direct reqwest dependency from remote interface

denoland / denokv

A self-hosted backend for Deno KV

https://deno.com/kv

MIT License

500 stars 24 forks source link

feat: remove direct reqwest dependency from remote interface #73

Closed mmastrac closed 6 months ago

mmastrac commented 6 months ago

The denokv remote interface currently requires a specific, older version of reqwest that makes it difficult for consumers to integrate. This PR introduces a RemoteTransport and RemoteResponse trait that allow for embedders to implement this remote over any sort of HTTP transport.

mmastrac commented 6 months ago

Did you test this on a live database?

Yes, it successfully connects. I was able to send requests as well.