Issue 2081 / path_to_filesystem

[pbiering]

• Fix: broken storage/mtime granularity detection on vfat
• Improve: path_to_filesystem() by pre-detection of collision-free file system
• Adjustment: MKCOL/MKCALENDAR return now CONFLICT instead of BADREQUEST of file name collision

Relates to

• pathutils/path_to_filesystem() need review #2081
• Improve performance of path_to_filesystem() #2002

[pbiering]

@flesueur - I've extended test cases and found that your implementation is also breaking softlinks supported on Windows with same issue found originally on Linux during PROPFIND request, see e.g. this particular workflow run: https://github.com/Kozea/Radicale/actions/runs/24395680534/job/71253826211#step:5:731

Having your change from #2002 reverted by 94746cf it is working again and passing the test on Windows https://github.com/Kozea/Radicale/actions/runs/24409725117

Can you please investigate further on, if there is no proper solution then the performance impact would be only on filesystems detected during startup as not collision-free.

[flesueur]

Sorry it was much more complex than what I expected ! I guess this part is still WIP but, anyway, I did not find any other way to solve it.
As soon as we use realpath() symlinks are problematic on every OS and surprisingly, there does not seem to be a realpath() alternative suited to this need. Anyway, could not find such a workaround.

Even going back to the original motivation I don't spontaneously find another good alternative :( .

Thanks !

[pbiering]

Sorry it was much more complex than what I expected ! I guess this part is still WIP

I think there is no other way then predetecting and making it faster by skip on unproblematic file system while keeping the old code for others.

If you agree the block below can be removed again, anyhow forced to be not active at the moment:

Radicale/radicale/pathutils.py

Line 295 in 7d07962

if sys.platform == "win32" and False: # temporary for testing

[flesueur]

Yes I think so, it is now useless and can be removed.