Skip to content

Comments

WIP: win32 readonly ACLs#9491

Draft
nilsding wants to merge 2 commits intomasterfrom
bugfix/noid/acl-errors-again
Draft

WIP: win32 readonly ACLs#9491
nilsding wants to merge 2 commits intomasterfrom
bugfix/noid/acl-errors-again

Conversation

@nilsding
Copy link
Member

@nilsding nilsding commented Feb 23, 2026

No description provided.

@nilsding nilsding added this to the 33.0.0 milestone Feb 23, 2026
@nilsding nilsding marked this pull request as draft February 23, 2026 14:30
@nilsding nilsding force-pushed the bugfix/noid/acl-errors-again branch from 1244106 to 4381607 Compare February 25, 2026 09:41
@github-actions
Copy link

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

Digest: sha256:0d87ed5a50856e6bcefcb8b4957acf3250206432aefc71d0d9cec6c9e938d457

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

Quality Gate Failed Quality Gate failed

Failed conditions
D Maintainability Rating on New Code (required ≥ A)
30 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

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

Noticed that GetLastError() could return 0 (error code for a successful
operation) -- probably due to win32 calls performed during logging

--> Store the result in a variable before writing log messages, just in
case ...

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
Also added some extra logging when InitializeAcl fails, and a regression
test.

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@nilsding nilsding force-pushed the bugfix/noid/acl-errors-again branch from 4381607 to 41caca0 Compare February 25, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant