Skip to content

fix(filesystembase): log correct ACL errors#8863

Merged
nilsding merged 1 commit intomasterfrom
bugfix/noid/acl-error-logging
Oct 9, 2025
Merged

fix(filesystembase): log correct ACL errors#8863
nilsding merged 1 commit intomasterfrom
bugfix/noid/acl-error-logging

Conversation

@nilsding
Copy link
Copy Markdown
Member

@nilsding nilsding commented Oct 9, 2025

In #8860 I noticed that the "insufficient memory error" message is logged if the last error was anything but ERROR_INSUFFICIENT_BUFFER

  • fix comparison operator
  • format Windows error codes with a hex code and message

In #8860 I noticed that the "insufficient memory error" message is
logged if the last error was anything but `ERROR_INSUFFICIENT_BUFFER`

- fix comparison operator
- format Windows error codes with a hex code and message

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding nilsding added this to the 4.0.0 milestone Oct 9, 2025
@nilsding nilsding self-assigned this Oct 9, 2025
@nilsding
Copy link
Copy Markdown
Member Author

nilsding commented Oct 9, 2025

/backport to stable-4.0

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 9, 2025

Artifact containing the AppImage: nextcloud-appimage-pr-8863.zip

Digest: sha256:c1ad79f034a0dc7080a986db3f394be1baf1e11b8fdae7f5771a5aaff1cd43d7

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Oct 9, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
Image C Maintainability Rating on New Code (required ≥ A)
Image 6 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Image Catch issues before they fail your Quality Gate with our IDE extension Image SonarQube for IDE

@nilsding nilsding merged commit c0bd552 into master Oct 9, 2025
21 of 22 checks passed
@nilsding nilsding deleted the bugfix/noid/acl-error-logging branch October 9, 2025 09:44
@nilsding nilsding modified the milestones: 4.0.0, 4.1.0 Oct 13, 2025
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.

2 participants