Include the Outerbase SDK for external database connections#39
Merged
Conversation
JayJamieson
reviewed
Nov 24, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Connecting to external data sources through your Starbase instance can help enhance your external database, whether it's by caching query data, serving as a data transformation layer (either before reaching the database, or before returning back to the client), acting as an extension to your database for development, or more.
This pull request makes it possible for the Worker to forward requests directly to your database via native drivers utilizing the Outerbase SDK. You can still opt to pass in an Outerbase API key instead and the request will be routed to Outerbase, but this will perhaps remove an extra hop/step in the flow by connecting to your database directly.
This also fixes issue #40 where the REST endpoint does not work for DELETE calls.
Tasks
Verify
afterQueryhook gets called on these new requestsEXTERNAL_DB_DEFAULT_SCHEMAdeclaration inwrangler.tomlBefore
After