Fix bugs discovered after real-world use.

spatie / flysystem-dropbox

A flysystem driver for Dropbox that uses the v2 API

MIT License

342 stars 50 forks source link

Closed BusterNeece closed 3 years ago

BusterNeece commented 3 years ago

Two classes of bugs I found:

Some of the calls are using the original path, not the filtered path (so prefixed folders don't work quite right)
When returning listContents values, Dropbox prepends everything with a leading slash, which doesn't line up with how files are listed in other Flysystem libraries.

Both are fixed in this revision.

freekmurze commented 3 years ago

Thank you, could you make sure all the tests pass in this package?

BusterNeece commented 3 years ago

@freekmurze Sorry about that, one miscall there. Should be green now.

freekmurze commented 3 years ago

Thanks!