feat(integration): compose hosted Vault plugin runtime - #244
Draft
seonghobae wants to merge 27 commits into
Draft
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This was referenced Sep 4, 2026
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.
Buyer/security outcome
Advances #130 beyond application-only Vault composition by adding the hosted Integration runtime seam. This slice owns one Integration PostgreSQL pool boundary for installation lifecycle, credential metadata and replay; composes the authenticated Vault-backed operator before listener start; and ties runtime shutdown to the service lifecycle without making another bounded context the persistence owner.
Current stack
Parent #243 is now
d03b886f67552e6bb2b53352087d2041dedba07c. Current exact headf0b91869f306b1b7d0d5759307acf347e413aa06is a non-force two-parent descendant of prior #244 headc552fbbd94e1aaeca66d65ae30187386aea906deand current #243. Fresh compare from the parent is ahead-only (behind_by=0) with the same nine Integration/ADR/architecture paths. No predecessor check/review evidence transfers.The preserved #244 RED→repair lineage covers exact
INTEGRATION_DATABASE_URL, one Integration-owned SQL pool, bounded malformed-envelope cleanup, pre-listener operator composition, captured lifecycle authority and concretePluginOperatorApplicationacceptance. Proposed ADR 0005 remains Proposed until the concrete driver and real PostgreSQL/Vault acceptance exist.Concrete successor #245 is now exact
45b5ad55b1575a677c9edf65fc350e55e3c68f8f, ahead-only from this #244 with ten Integration-owned paths. It retains self-contained PostgreSQL URI authority, URI query/fragment denial, verified TLS source policy, finite Pool/statement/query lifecycle, bounded idle-error evidence, acquisition cleanup, credential-free process startup failure, lazy-Pool readiness, hostile readiness-result handling, and one-time capture of both query and shutdown method authority. Latest shutdown-authority RED is95c23186b09114e4ae01b6a6eaf1c8e4aec126ba→ causal repair0c6065378a57c99574694e203463b5111847238c, with edge coverage at exact successor head.The frozen-lock RED remains: Integration manifest declares
pg/@types/pg, while the root Integration importer lacks both direct entries. The accepted next repair is importer-only; prior broad lock regeneration is not accepted evidence.Keep Draft until frozen-lock reproducibility and protected-lineage PostgreSQL + Vault/TLS acceptance exist. Only then advance #130 to connect-time SSRF/DNS-rebinding-safe outbound HTTPS, redirect/proxy policy, signing/idempotency, durable retry/dead-letter/recovery, revocation fencing and operator recovery.
Refs #130, #205, #235, #241, #242, #243, #245.