In `snapshot.go` `RemoveAllSnapshots()`, duplicate snapshot IDs are detected with a linear scan:
```go
found := false
for _, id := range snapshotIDs {
if id == sid {
found = true
break
}
}
```
This is O(n²) overall. Should use a `map[string]bool` for O(1) lookups. Same pattern exists in `PruneBlobs()`.
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.goRemoveAllSnapshots(), duplicate snapshot IDs are detected with a linear scan:This is O(n²) overall. Should use a
map[string]boolfor O(1) lookups. Same pattern exists inPruneBlobs().Ref: parent issue #1