[JCLOUDS-1422] LocalBlobStore.list ignores recursive flag when prefix set

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0, 2.2.0
• Fix Version/s: 2.2.0, 2.1.1
• Component/s: jclouds-blobstore
• Labels: filesystem transient
• Flags: Patch
• Assignee: Andrew Gaul
• Reporter: Jesse Glick
• Created: 23/May/18 00:04
• Updated: 07/Jun/18 05:52
• Resolved: 07/Jun/18 05:52

Description

While calling jclouds code to list a container I noticed that while the S3 implementation can handle both prefix and recursive on or off, when using the transient implementation the recursive flag seems to be ignored—the code always behaves as if it were on. I looked into LocalBlobStore.list and saw that indeed since JCLOUDS-930 the implementation just checks prefix before recursive and in that case calls filterPrefix which ignores recursive without calling extractCommonPrefixes like all other code branchs called when !recursive. It was tricky to see what exactly the desired behavior was (especially when the prefix does not end with the separator/delimiter), since the Javadoc of ListContainerOptions is sparse and vague, but I reasoned that the current behavior was wrong since

• the S3 implementation behaves differently
• the current implementation quietly fails to differentiate two option combinations, with no code in ListContainerOptions indicating that one implies the other

Issue Links

• relates to: JCLOUDS-930 Expose the prefix option when listing a container
• links to: PR 1212