-
-
Notifications
You must be signed in to change notification settings - Fork 1
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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
About the future of wrangler-proxy
since [email protected]
and up exposes getBindingsProxy
#23
Comments
Ohm, maybe I'll add some |
I recently attempted this approach. |
Yep I started building |
Oh I see! I'm only using it locally for tests on my side. This is interesting. |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Hello!
Since https://github.com/cloudflare/workers-sdk/releases/tag/wrangler%403.24.0 it is now possible to do the following:
The PR is cloudflare/workers-sdk#4523
@chientrm Maybe you already know about this, I wanted to mention it, in my context I first used
wrangler-proxy
, to query D1, which worked great! And now switched togetBindingsProxy
and it also works great! I like how it's an official feature now 🙌🏼So, I'm not sure what it means for this project here, if you see the need to continue or to recommend
getBindingsProxy
instead? Just curious about your thoughts.The text was updated successfully, but these errors were encountered: