In snapshot.goListSnapshots(), listing snapshots silently deletes local snapshot records that are not found in remote storage:
// Remove local snapshots that don't exist remotelyfor_,snapshot:=rangelocalSnapshots{if!remoteSnapshots[snapshotIDStr]{// ... deletes from local DB}}
A list/read operation should not have destructive side effects. This sync behavior should be a separate explicit operation (e.g. vaultik sync), or at minimum clearly documented and logged prominently.
In `snapshot.go` `ListSnapshots()`, listing snapshots silently deletes local snapshot records that are not found in remote storage:
```go
// Remove local snapshots that don't exist remotely
for _, snapshot := range localSnapshots {
if !remoteSnapshots[snapshotIDStr] {
// ... deletes from local DB
}
}
```
A list/read operation should not have destructive side effects. This sync behavior should be a separate explicit operation (e.g. `vaultik sync`), or at minimum clearly documented and logged prominently.
Ref: parent issue #1
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
In
snapshot.goListSnapshots(), listing snapshots silently deletes local snapshot records that are not found in remote storage:A list/read operation should not have destructive side effects. This sync behavior should be a separate explicit operation (e.g.
vaultik sync), or at minimum clearly documented and logged prominently.Ref: parent issue #1