core/go-infra
8
0
Fork 0
Code Issues 4 Pull requests 1 Projects Releases Packages Wiki Activity Actions

Fix CodeRabbit findings #8

Merged
Snider merged 2 commits from agent/fix-coderabbit-findings--verify-each-aga into dev 2026-03-24 11:43:34 +00:00
Conversation 0 Commits 2 Files changed 2 +28 -8

2 commits

Author SHA1 Message Date
Snider
1cecf00148 Merge origin/dev into agent/fix-coderabbit-findings--verify-each-aga 
Resolve conflict in hetzner_test.go: keep PR's CodeRabbit fixes
alongside dev's new test functions (load balancer, snapshot, GetServer).

Co-Authored-By: Virgil <virgil@lethean.io>
 2026-03-24 11:41:09 +00:00
Snider
ec1a591b71 fix(coderabbit): address review findings 
- Truncate response body in error messages to 256 bytes to prevent
  sensitive data leakage from upstream APIs (client.go)
- Replace unchecked strings.Split index with safe repoShortName helper
  to prevent potential panic (cmd/monitor)
- Fix test assertions to match coreerr.E error format after prior refactor

Co-Authored-By: Virgil <virgil@lethean.io>
 2026-03-17 16:24:37 +00:00