BaryoDev / BaryoDev/barakoCMS

Credentials: the startup encryption migration still reads partitions directly under database enforcement

Open
#944 0 comments 0 reactions 0 assignees View on GitHub
bug core security
Dominant language
C#
Stars
6
Forks
7
Avg merge
4h 42m
Merged PRs (30d)
307

Description

#943 moves the workflow runner, both retention sweeps and the job queue worker onto a per-tenant listing when `Tenancy:DatabaseEnforcement` is on, and closes #877. The startup credential migration (the one #880 extends with the `enc:v1:` prefix) still lists tenants the old way, so with enforcement on it fails and stored credentials stay unencrypted.

### Done when
- The credential migration uses the same tenant listing as #943.
- A test with enforcement on fails before the change and shows a named tenant's credential encrypted after it.

Follow-up to #877 and #880.

Contributor guide

Open the contributing guide

Research direction

Start at the startup credential migration and compare its tenant-listing path with the per-tenant listing introduced by #943. Add the enforcement-on regression test described in the issue, then verify that a named tenant’s stored credential is encrypted after the migration.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
backend, databases, security
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.