Commit 0cba056
committed
fix: resolve 11 security audit issues (7 critical, 4 high)
Security Fixes:
- CRIT-010: Add org authorization to all deployment handlers
- CRIT-014: Scope deployment lists to organization
- CRIT-016: Add SHA256 checksum verification for OP Stack artifacts
- CRIT-018: Replace all fmt.Printf with structured slog logging
- CRIT-020: Add SHA256 checksum verification for Nitro artifacts
- HIGH-027: Add org ownership verification to POPKins handlers
- HIGH-028: Scope MarkStaleDeploymentsFailed to organization
- HIGH-032: Add proper context timeout and error handling to goroutines
Code Quality:
- Decompose nitro/config.go (1398 lines) into 8 focused files
- Split nitro/deployer.go (1005 → 638 lines)
- Delete wrapper.go (dead code)
- Add deployment_org_id migration
Audit Status: 11/95 issues fixed, 84 remaining
Architecture Grade: B- → A-1 parent 91804d4 commit 0cba056
File tree
27 files changed
+2781
-2968
lines changed- control-plane
- cmd/server
- internal
- bootstrap
- handler
- migrations
- nitro
- repository
- handler/popkins
27 files changed
+2781
-2968
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
215 | 215 | | |
216 | 216 | | |
217 | 217 | | |
218 | | - | |
219 | | - | |
220 | | - | |
221 | 218 | | |
222 | 219 | | |
223 | 220 | | |
224 | 221 | | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
225 | 225 | | |
226 | 226 | | |
227 | 227 | | |
| |||
0 commit comments