aboutsummaryrefslogtreecommitdiff
path: root/weed/operation/grpc_client.go
diff options
context:
space:
mode:
authorChris Lu <chrislusf@users.noreply.github.com>2025-12-12 23:21:51 -0800
committerGitHub <noreply@github.com>2025-12-12 23:21:51 -0800
commit72853a3bbfce27c55e6c316ee446ad35e6786260 (patch)
tree91d272c6022f0557e89d2e4058c5b6b7ccad3b4e /weed/operation/grpc_client.go
parent662a6ac8ee45caede35b82f4c5527cc30c0de0c2 (diff)
downloadseaweedfs-72853a3bbfce27c55e6c316ee446ad35e6786260.tar.xz
seaweedfs-72853a3bbfce27c55e6c316ee446ad35e6786260.zip
fix: prevent path doubling in versioned object listing (#7729)
* fix: prevent path doubling in versioned object listing Fix path doubling bug in getLatestVersionEntryForListOperation that caused Velero/Kopia backups to fail when using S3 bucket versioning. The issue was that when creating logical entries for versioned object listing, the entry.Name was set to the full object path (e.g., 'kopia/logpaste/kopia.blobcfg') instead of just the base filename ('kopia.blobcfg'). When this entry was used in the list callback which combines dir + entry.Name, the paths got doubled: '/buckets/velero/kopia/logpaste/kopia/logpaste/kopia.blobcfg' This caused Kopia to fail loading pack indexes with the error: 'unable to load pack indexes despite 10 retries' The fix uses path.Base(object) to extract only the filename portion, matching how regular (non-versioned) entries work in the listing callback. Fixes: GitHub discussion #7573 * refactor: use path.Base directly in test instead of reimplementing Address code review feedback to simplify the test by using the standard library path.Base function directly instead of reimplementing it. * remove test: unit test for path.Base doesn't add much value
Diffstat (limited to 'weed/operation/grpc_client.go')
0 files changed, 0 insertions, 0 deletions