chore(m04): decide safe-session Vault signer future before Vault sunset #609

Closed
opened 2026-05-29 19:01:07 +02:00 by codex · 0 comments
Collaborator

Context

#607 Phase 1 metadata-only Vault inventory found that live HashiCorp Vault on rs2000 does not expose a general KV secret/ engine.

Observed metadata-only shape:

secret engines: cubbyhole/, identity/, ssh/, sys/
policy: safe-session-signer
AppRole: safe-session-api
SSH role: safe-session

The repo also models safe-session-api as active with upstream dependency vault.

Scope

Decide what should happen to the safe-session subsystem before any Vault sunset continues.

Acceptance criteria

  • Decide whether safe-session-api and safe-session-web still provide value.
  • If keeping them, document Vault as a narrow internal SSH signer, not a general legacy secret manager.
  • If retiring them, create a separate runtime-gated retirement plan covering safe-session-api, safe-session-web, vault, and vault-bootstrap together.
  • Do not stop Vault from this issue.
  • Do not delete Vault volumes.
  • Do not rotate or expose Vault tokens.
  • Do not mirror anything to Infisical unless a later issue identifies actual KV secrets.

Out of scope

  • Vault shutdown.
  • Safe-session runtime changes.
  • Infisical writes.
  • Secret value reads.

Notes

Infisical remains the canonical platform secret backend. This issue exists because Vault is currently serving a different function: SSH certificate/signing support for safe-session.

## Context #607 Phase 1 metadata-only Vault inventory found that live HashiCorp Vault on `rs2000` does not expose a general KV `secret/` engine. Observed metadata-only shape: ```text secret engines: cubbyhole/, identity/, ssh/, sys/ policy: safe-session-signer AppRole: safe-session-api SSH role: safe-session ``` The repo also models `safe-session-api` as active with upstream dependency `vault`. ## Scope Decide what should happen to the `safe-session` subsystem before any Vault sunset continues. ## Acceptance criteria - Decide whether `safe-session-api` and `safe-session-web` still provide value. - If keeping them, document Vault as a narrow internal SSH signer, not a general legacy secret manager. - If retiring them, create a separate runtime-gated retirement plan covering `safe-session-api`, `safe-session-web`, `vault`, and `vault-bootstrap` together. - Do not stop Vault from this issue. - Do not delete Vault volumes. - Do not rotate or expose Vault tokens. - Do not mirror anything to Infisical unless a later issue identifies actual KV secrets. ## Out of scope - Vault shutdown. - Safe-session runtime changes. - Infisical writes. - Secret value reads. ## Notes Infisical remains the canonical platform secret backend. This issue exists because Vault is currently serving a different function: SSH certificate/signing support for `safe-session`.
codex added this to the 04 - Vault sunset milestone 2026-05-29 19:01:07 +02:00
Sign in to join this conversation.
No labels
W6d-automerge-calibration
agent/claude-code
agent/codex
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
automerge-candidate
class/security-sensitive
cutover-gate
dependency/blocked
dependency/blocks-others
dependency/cross-repo
dependency/needs-confirmation
domain:agents
domain:ci
domain:docs
domain:forgejo
domain:infra
domain:memory
domain:runtime
domain:signal
domain:ux
flow/architecture
flow/blocked
flow/deployed
flow/done
flow/implementation
flow/intake
flow/maintained
flow/observed
flow/ready
flow/refining
flow/retired
flow/review
iterating
judge/codex-candidate
judge/hermes-candidate
judge/low-confidence
judge/needs-refinement
judge/operator-needed
judge/p0
judge/p1
judge/p2
judge/p3
judge/park
judge/patchwarden-candidate
judge/stale-priority
kind/adr
kind/bug
kind/chore
kind/feature
kind/infra
kind/ops
kind/refactor
kind/research
large-impact
merge/auto
merge/manual
merge/manual-dependency-conflict
merge/manual-failing-tests
merge/manual-merge-conflict
merge/manual-missing-review
merge/manual-operator-preference
merge/manual-red-zone
merge/manual-security-sensitive
merge/manual-unclear-scope
merge/manual-unknown
meta
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
needs-operator-decision
needs-triage
not-ready
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
operator-emotional
owner-attention
phase/02
phase/03
priority:p0
priority:p1
priority:p2
priority:p3
proposed
ready-for-agent
ready-for-operator
recovery
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
risk/exposure
risk/process
risk/product
risk/runtime
safety:external-write
safety:no-prod-mutation
safety:prod-impact
safety:secret-touch
size/large
size/medium
size/small
size/tiny
size/unknown
source/adr
source/agent-generated
source/manual
source/operator-chat
source/voice-note
status:blocked
status:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:operator-needed
status:parked
tier/full
tier/lite
tier/stacked
tier:0-platform-substrate
tier:1-iskra-value-layer
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pdurlej/platform#609
No description provided.