sneak/vaultik
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix snapshot list to fail on manifest errors

Browse Source 
- Remove error suppression for manifest decoding errors
- Manifest read/deserialize errors now fail immediately with clear error messages
- This ensures we catch format mismatches and other issues early
This commit is contained in:
Jeffrey Paul 2025-07-26 03:31:09 +02:00
parent a544fa80f2
commit 5c70405a85
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
internal/cli/snapshot.go
View File

@ -548,8 +548,7 @@ func (app *SnapshotApp) List(ctx context.Context, jsonOutput bool) error {
// Try to download manifest to get size // Try to download manifest to get size
totalSize, err := app.getManifestSize(ctx, snapshotID) totalSize, err := app.getManifestSize(ctx, snapshotID)
if err != nil { if err != nil {
log.Warn("Failed to get manifest size", "id", snapshotID, "error", err) return fmt.Errorf("failed to get manifest size for %s: %w", snapshotID, err)
totalSize = 0
} }
snapshots = append(snapshots, SnapshotInfo{ snapshots = append(snapshots, SnapshotInfo{