dohertj2/scadaproj
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

[Secrets] CLI appsettings SqlitePath is relative (fails validation); plain IConfigurationRoot reload gap; OtOpcUa starter now redundant #49

New Issue
Closed
opened 2026-11-09 15:01:05 -05:00 by dohertj2 · 1 comment
dohertj2 commented 2026-11-09 15:01:05 -05:00
Owner
Copy Link
Copy Source

Follow-ups from the Secrets 0.6.6 fixes (#30–#35):

  • The CLI's own src/ZB.MOM.WW.Secrets.Cli/appsettings.json sets "SqlitePath": "secrets.db", which the store-path validator rejects as relative — the headless CLI fails at startup unless an env var overrides it. Ship a valid default (absolute, outside the content root, e.g. under LocalApplicationData) or no default with a clear message. (Secrets-009 covers only the README sample.)
  • A plain IConfigurationRoot that reloads gets only the weaker re-apply-after-reload behaviour from #32 (callback order not guaranteed) — documented, not fully fixed.
  • Once OtOpcUa adopts 0.6.6 it can drop its own SecretReplicationStarter (#33).
Follow-ups from the Secrets 0.6.6 fixes (#30–#35): - The CLI's own `src/ZB.MOM.WW.Secrets.Cli/appsettings.json` sets `"SqlitePath": "secrets.db"`, which the store-path validator rejects as relative — the headless CLI fails at startup unless an env var overrides it. Ship a valid default (absolute, outside the content root, e.g. under LocalApplicationData) or no default with a clear message. (Secrets-009 covers only the README sample.) - A plain `IConfigurationRoot` that reloads gets only the weaker re-apply-after-reload behaviour from #32 (callback order not guaranteed) — documented, not fully fixed. - Once OtOpcUa adopts 0.6.6 it can drop its own `SecretReplicationStarter` (#33).
dohertj2 added the type:followuppriority:p3 labels 2026-11-09 15:01:05 -05:00
dohertj2 commented 2026-11-09 15:33:26 -05:00
Author
Owner
Copy Link
Copy Source

Done on main 8235ee6 / 2619166: the CLI's relative SqlitePath removed — the per-user default applies; the plain-IConfigurationRoot reload gap is documented (fixing it needs .NET configuration internals; no family app is exposed); OtOpcUa's SecretReplicationStarter noted as redundant in clustered-secrets.md.

Done on main 8235ee6 / 2619166: the CLI's relative SqlitePath removed — the per-user default applies; the plain-IConfigurationRoot reload gap is documented (fixing it needs .NET configuration internals; no family app is exposed); OtOpcUa's SecretReplicationStarter noted as redundant in clustered-secrets.md.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
blocked
Blocked by dependency or external hold
 cross-repo
Needs coordinated change in another family repo (wire contract etc.)
 migrated
Migrated from a markdown backlog (2026-11 standardization)
 priority:p0
Must-have / blocking
 priority:p1
High priority
 priority:p2
Normal priority
 priority:p3
Low priority / nice to have
 type:bug
Defect in shipped behaviour
 type:debt
Technical debt / hygiene
 type:deploy-gate
Rollout/deploy/migration step not yet done on an environment
 type:docs
Documentation work
 type:feature
Feature implementation
 type:followup
Follow-up deferred from a plan, PR or gate
 type:infra
Infrastructure and scaffolding
 type:review-finding
Unfixed finding from an architecture/code review
 type:test
Test work
No Label priority:p3 type:followup
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dohertj2 (dohertj2)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dohertj2/scadaproj#49
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?