Skip to content

Bugfix/mgallien/windows clean up logs - #11080

Open
mgallien wants to merge 6 commits into
masterfrom
bugfix/mgallien/windowsCleanUpLogs
Open

mgallien wants to merge 6 commits into
masterfrom
bugfix/mgallien/windowsCleanUpLogs

Conversation

@mgallien

Copy link
Copy Markdown
Collaborator

Resolves

#

Summary

TODO

  • ...

Checklist

AI (if applicable)

a folder that has not been populated even if modified on the server
should not become populated

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
will ensure teh client has proper tracking of which items are virtual
and which items are hydrated or populated

should avoid teh cleint deciding to populate a folder that just got a
change on server but was never visited by the user

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
add more logs around any item type modifications that are only compiled
when NEXTCLOUD_DEV option is active

might be needed for extreme cases when item type is not clear

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
tune logs to limit their size to limit their impact

possibly not hapenning in productiond default settings but some may be
activating debug logs

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
add more information about the process triggering a folder population
via the CfApi callback

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

Copy link
Copy Markdown
Collaborator Author

includes the changes in #11078

@mgallien

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0 please

@mgallien

Copy link
Copy Markdown
Collaborator Author

/backport to stable-35.0 please

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
79.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant