Proposal: a local front door to the Datum Cloud API (datumctl api proxy) - #246
Closed
scotwells wants to merge 10 commits into
Closed
Proposal: a local front door to the Datum Cloud API (datumctl api proxy)#246scotwells wants to merge 10 commits into
scotwells wants to merge 10 commits into
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
|
Moved to the enhancements repo, where proposal documents live: datum-cloud/enhancements#818. The companion implementation PR remains #247. |
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.
What this is
An enhancement proposal for
datumctl api proxy— a local server that gives every tool on your machine an authenticated front door to the Datum Cloud API. It forwards each request signed in as you, attaching credentials automatically and keeping them fresh, so the tools behind it never see a token, never store one, and never break when one expires:This PR contains only the design document (
docs/enhancements/api-proxy/README.md).Why it exists
.envfiles, a developer points the app's API base URL at the proxy once and datumctl owns authentication for the whole workday.curlagainst a local address instead of copy-pasting tokens that expire mid-investigation.Datum's own cloud portal is the first consumer, but the friction is the same for anything built against the platform.
Key design positions the doc takes
--project/--organizationserve a single project or organization for short URLs. Scope is never silently inherited from the active context.datumctl auth switchnever repoints a running proxy), the printed address doubles as a machine-readable readiness signal, and when the session dies the proxy answers with a clearly marked error that names the fix rather than imitating a platform rejection.Review focus
The decisions most worth reviewer eyes — a bare invocation never inherits the active context (production-parity wins), local-machine-only with no override, a same-OS-user socket mode as the first fast-follow, and never retrying a platform rejection on the client's behalf — are each stated with their rationale in the section they belong to.
Companion implementation PR: #247.
🤖 Generated with Claude Code