mirror of
https://github.com/s3fs-fuse/s3fs-fuse.git
synced 2026-04-25 05:16:00 +03:00
[PR #753] [MERGED] fix xpath selector in bucket listing #1624
Labels
No labels
bug
bug
dataloss
duplicate
enhancement
feature request
help wanted
invalid
need info
performance
pull-request
question
question
testing
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/s3fs-fuse#1624
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/753
Author: @cfz
Created: 4/23/2018
Status: ✅ Merged
Merged: 5/2/2018
Merged by: @ggtakec
Base:
master← Head:master📝 Commits (1)
04493defix xpath selector in bucket listing📊 Changes
1 file changed (+5 additions, -2 deletions)
View changed files
📝
src/s3fs.cpp(+5 -2)📄 Description
the original implementation in get_base_exp() depends on the order of xml return from the server.
patriotically, when listing a directory with sub directory(s), the xml document response contains more than 2 nodes(some of them are in node).
the source code arbitrarily select the first one in the documents (nodes->nodeTab[0]->xmlChildrenNode), which is OK for aws.
however, some s3 compatible service return the list-bucket result in different order, leading the s3fs to a wrong behavior
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.