Skip to content

security(integrations): harden credential lifecycle boundaries - #31

Merged
darwin67 merged 4 commits into
mainfrom
rfd-5-phase-1a-review-fixes
Sep 4, 2026
Merged

security(integrations): harden credential lifecycle boundaries#31
darwin67 merged 4 commits into
mainfrom
rfd-5-phase-1a-review-fixes

Conversation

@darwin67

@darwin67 darwin67 commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

  • reject raw OAuth installation through API-key connect and replacement operations
  • bound malformed top-level and nested credential payloads without exposing secret input in diagnostics
  • close provider-specific Phoenix and LiveView parameter-filtering gaps
  • normalize user-deletion races and align key-version validation with database limits
  • deterministically pause in-flight encryption to test competing writes, disconnect fencing, and deletion fencing

Verification

  • focused credential and lifecycle tests: 24 passed
  • mix precommit: 360 passed, 5 live-provider tests excluded
  • valid production key-ring runtime configuration check passed
  • final Oracle review found no remaining issues in the implemented Phase 1A foundation scope

Stack

Depends on #30, which depends on #29 and #28.

@darwin67
darwin67 force-pushed the rfd-5-phase-1a-lifecycle branch from aad7836 to f4e6668 Compare September 3, 2026 23:19
@darwin67
darwin67 force-pushed the rfd-5-phase-1a-review-fixes branch 3 times, most recently from ae90e62 to 458c13f Compare September 3, 2026 23:41
@darwin67
darwin67 force-pushed the rfd-5-phase-1a-lifecycle branch from ede6447 to b3769d0 Compare September 3, 2026 23:41
Base automatically changed from rfd-5-phase-1a-lifecycle to main September 3, 2026 23:46
@darwin67
darwin67 force-pushed the rfd-5-phase-1a-review-fixes branch from 458c13f to 080a4ba Compare September 3, 2026 23:47
@darwin67
darwin67 merged commit eb5e11a into main Sep 4, 2026
17 checks passed
@darwin67
darwin67 deleted the rfd-5-phase-1a-review-fixes branch September 4, 2026 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant