Skip to content

fix flatgeobuf params by getRequestConfigurationByUrl#12200

Merged
allyoucanmap merged 1 commit intogeosolutions-it:masterfrom
stefanocudini:fix-12162-cogs-flatgeobuf-from-protected-storage_patch
Mar 31, 2026
Merged

fix flatgeobuf params by getRequestConfigurationByUrl#12200
allyoucanmap merged 1 commit intogeosolutions-it:masterfrom
stefanocudini:fix-12162-cogs-flatgeobuf-from-protected-storage_patch

Conversation

@stefanocudini
Copy link
Copy Markdown
Member

@stefanocudini stefanocudini commented Mar 31, 2026

Fix #12162

  • fix cog catalog sugin security params
  • new function setSecurityParams
  • added unit test for setSecurityParams
  • secure params in 3D layers
  • fix tests SecurityUtils
  • This PR includes missing security params in the FlatGeoBufLayer requests

Description

In short, in cases of datasets using custom rules in the requestsConfigurationRules section of localConfig.json, the URL parameters for resource addresses corresponding to COG and FlatGeobuf files were not being passed. See the issue for an example of rules to use for testing.

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix

Issue

What is the current behavior?

#12162

What is the new behavior?
Ensure the security params are included inside FlatGeoBufLayer in openlayers

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • No

Other useful information

See the linked issue for bug reproducing details.
Geonode-client porting is required

@stefanocudini stefanocudini added this to the 2026.01.01 milestone Mar 31, 2026
@cla-bot cla-bot bot added the CLA Ready label Mar 31, 2026
@stefanocudini stefanocudini self-assigned this Mar 31, 2026
@stefanocudini stefanocudini linked an issue Mar 31, 2026 that may be closed by this pull request
1 task
@allyoucanmap allyoucanmap merged commit 06c0454 into geosolutions-it:master Mar 31, 2026
13 checks passed
@allyoucanmap
Copy link
Copy Markdown
Contributor

@ElenaGallo please test this fix on dev, thanks

@offtherailz
Copy link
Copy Markdown
Member

Created backport PR for 2026.01.xx:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-12200-to-2026.01.xx
git worktree add --checkout .worktree/backport-12200-to-2026.01.xx backport-12200-to-2026.01.xx
cd .worktree/backport-12200-to-2026.01.xx
git reset --hard HEAD^
git cherry-pick -x 06c0454619cb2ead15ce8d7e6800ca9fd25f3545
git push --force-with-lease

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.

COG and FlatGeobuf layers support params for protected storage

4 participants