Daft icon indicating copy to clipboard operation
Daft copied to clipboard

Disallow glob paths with `**` that is not a folder segment

Open jaychia opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe.

Daft currently only correctly understands the ** recursive wildcard when it is applied to a folder segment. I.e.

  • s3://bucket/**.csv is not valid
  • s3://bucket/**/*.csv is valid and conveys the intended behavior

We should provide an appropriate error message when we detect the former, or perhaps sanitize the input when we find instances of ** not enclosed by /.

jaychia avatar Jan 26 '24 19:01 jaychia

I would like to work on this

nemo-1999 avatar Jan 27 '24 04:01 nemo-1999

take

nemo-1999 avatar Jan 27 '24 04:01 nemo-1999

hey @jaychia, i'd like to give this a shot :)

conradsoon avatar Oct 22 '24 08:10 conradsoon