cryptomator / cryptofs

Java Filesystem Provider with integrated encryption
GNU Affero General Public License v3.0
93 stars 35 forks source link

Feature/shortened names check #110

Closed infeo closed 3 years ago

infeo commented 3 years ago

This PR adds a new check to the health check API: ShortenedNamesCheck

For all directories ending with c9s it checks, if the file name.c9s exists in its hashed & encoded content matches the directory name.

It produces 4 results:

For the future this check can be refinded by providing the directory id of the encrypted dir containing the shortened resource, but for this PR it is out of scope and only noted via inline comments.

infeo commented 3 years ago

This feature is feature-complete, so remove those comments

Should I only remove the "TODO" marker or the whole comment?

overheadhunter commented 3 years ago

Should I only remove the "TODO" marker or the whole comment?

The whole comment. Treat the current implementation as final, document future changes in an issue.