rdrop2 icon indicating copy to clipboard operation
rdrop2 copied to clipboard

drop_exists() is case-sensitive

Open dholstius opened this issue 4 years ago • 1 comments

drop_exists() is case-sensitive. This means that it will incorrectly return FALSE in the following scenario:

  1. Alice creates (in her Dropbox) a file named /Foo/bar/baz.txt.
  2. Alice shares /Foo/ with Bob.
  3. Bob renames (on his filesystem) the subfolder bar/ to Bar/.
  4. Bob invokes drop_exists("/Foo/Bar/baz.txt"), expecting the result to be TRUE.

Following up with a pull request.

dholstius avatar Jan 06 '20 16:01 dholstius

Wouldn't case insensitive mean that the following scenario will be incorrect?

On a case-sensitive system (i.e., Linux):

  1. Alice creates in her Dropbox) a file named /Foo/bar/baz.txt
  2. Bob checks if /Foo/bar/BAZ.txt exists. After your fix, the return will be TRUE.
  3. Bob tries to read /Foo/Bar/BAZ.txt using R (such as with readLines().
  4. Operation fails, Bob is bamboozled.

J-Moravec avatar Jan 11 '24 20:01 J-Moravec