search

nextcloud-gmbh / mtime_fixer_tool_kit

Tool kit to fix the mtime issue on the server state
15 stars 7 forks source link

add an extra arg to possibly also scan the corrected files #7

Closed mgallien closed 2 years ago

mgallien commented 2 years ago

Signed-off-by: Matthieu Gallien matthieu.gallien@nextcloud.com

artonge commented 2 years ago

I thought about this, but I think that most cases if not all cases will not benefit from it as it will slow down the script way too much. So not a good idea to offer this feature. Do you have a request for it ?

mgallien commented 2 years ago

I thought about this, but I think that most cases if not all cases will not benefit from it as it will slow down the script way too much. So not a good idea to offer this feature. Do you have a request for it ?

Imagine a case where the server has way many files but only a few ones are affected by the wrong date bug. I guess in that case we would benefit from this. So I would like to be able to suggest that in cases where only a few files are affected.

mgallien commented 2 years ago

@artonge, if it is off by default I guess it will be better to be able to use it for cases where storage has many files but only 0.1% are affected

mgallien commented 2 years ago

@artonge is it good now ? I will need it today (if you agree)