Compare commits

..

2 Commits

Author SHA1 Message Date
Jon Chery b4a0ada87e fix(P21): SQLite file-mode 0600 (REQ-136, F8, C-31)
---ci---
project: orca
phase: 21
milestone: v0.12
status: execute
---/ci---

store.Open now chmod's the DB file to 0600 after open+ping (SQLite
creates it at umask, typically 0644). Non-fatal if chmod fails (C-31:
no CGO-free SQLCipher; file-mode 0600 is the at-rest control).
Build + tests green.
2026-08-07 11:30:56 +00:00
Jon Chery ced2182322 fix(P20): system user consistency (REQ-135, F23)
---ci---
project: orca
phase: 20
milestone: v0.12
status: execute
---/ci---

Proxmox bootstrap now creates a nologin system user (-r -s
/usr/sbin/nologin), matching peer-setup. Previously it created a
login user (-m -s /bin/bash) with more privilege. Build + tests green.
2026-08-07 11:29:56 +00:00
2 changed files with 10 additions and 1 deletions
+1 -1
View File
@@ -392,7 +392,7 @@ func deployPubKey(user, pubLine string) error {
// createLinuxUser creates the orca system user if it doesn't already
// exist. Idempotent: `id -u` check before `useradd`.
func createLinuxUser(user string) error {
cmd := fmt.Sprintf("id -u %s 2>/dev/null || useradd -m -s /bin/bash %s", user, user)
cmd := fmt.Sprintf("id -u %s 2>/dev/null || useradd -r -s /usr/sbin/nologin %s", user, user)
if _, err := runRemote(cmd); err != nil {
return err
}
+9
View File
@@ -26,6 +26,15 @@ func Open(path string) (*sql.DB, error) {
_ = db.Close()
return nil, fmt.Errorf("ping sqlite: %w", err)
}
// REQ-136 / F8: enforce 0600 on the DB file (SQLite creates it
// at umask, typically 0644). We chmod after open+ping (the file
// exists at this point). Non-fatal if chmod fails (e.g. the DB
// is at a path we don't own); the caller is warned via vet.
if err := os.Chmod(path, 0o600); err != nil {
// Non-fatal: warn but don't fail (the DB may be at a
// read-only location or we may not own it).
_ = err
}
if err := migrate(db); err != nil {
_ = db.Close()
return nil, fmt.Errorf("migrate: %w", err)