Server/OAPIF: fix unreported issue with access control layer permissions

[elpaso]

When 'canRead' property for a published WFS layer was set to False from an access control filter plugin the whole /collections request was returning a 404 instead of returning the list of published layers (minus the filtered ones).

Funded by: Gis3W

[github-actions]

🪟 Windows builds ready!

Windows builds of this PR are available for testing here. Debug symbols for this build are available here.

(Built from commit 937e2e8)

[nyalldawson]

@elpaso this needs a merge conflict fix now

[qgis-bot]

The backport to queued_ltr_backports failed:

The process '/usr/bin/git' failed with exit code 1

stderr

error: could not apply f9688deeeff... Server/OAPIF: fix unreported issue with access control layer permissions
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

stdout

Auto-merging src/server/services/wfs3/qgswfs3handlers.cpp
CONFLICT (content): Merge conflict in src/server/services/wfs3/qgswfs3handlers.cpp
Auto-merging tests/src/python/test_qgsserver_api.py

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-queued_ltr_backports queued_ltr_backports
# Navigate to the new working tree
cd .worktrees/backport-queued_ltr_backports
# Create a new branch
git switch --create backport-57046-to-queued_ltr_backports
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick f9688deeeff9bfd352b84e888af4df64af1f98b2
# Push it to GitHub
git push --set-upstream origin backport-57046-to-queued_ltr_backports
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-queued_ltr_backports

Then, create a pull request where the base branch is queued_ltr_backports and the compare/head branch is backport-57046-to-queued_ltr_backports.

[qgis-bot]

The backport to release-3_36 failed:

The process '/usr/bin/git' failed with exit code 1

stderr

error: could not apply f9688deeeff... Server/OAPIF: fix unreported issue with access control layer permissions
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".

stdout

Auto-merging src/server/services/wfs3/qgswfs3handlers.cpp
CONFLICT (content): Merge conflict in src/server/services/wfs3/qgswfs3handlers.cpp

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-3_36 release-3_36
# Navigate to the new working tree
cd .worktrees/backport-release-3_36
# Create a new branch
git switch --create backport-57046-to-release-3_36
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick f9688deeeff9bfd352b84e888af4df64af1f98b2
# Push it to GitHub
git push --set-upstream origin backport-57046-to-release-3_36
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-3_36

Then, create a pull request where the base branch is release-3_36 and the compare/head branch is backport-57046-to-release-3_36.